From e89915d06bd862bfb14da4f0d08581c2a1ef45f8 Mon Sep 17 00:00:00 2001
From: Sean Lilley <lilleyse@gmail.com>
Date: Mon, 12 Dec 2016 14:23:41 -0500
Subject: [PATCH] Add TODO list in spec - temp

---
 Specs/Scene/ModelSpec.js | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/Specs/Scene/ModelSpec.js b/Specs/Scene/ModelSpec.js
index 5c484eee2b8d..d388c2d41888 100644
--- a/Specs/Scene/ModelSpec.js
+++ b/Specs/Scene/ModelSpec.js
@@ -1934,6 +1934,32 @@ defineSuite([
         });
     });
 
+    it('renders with a silhouette', function() {
+        return loadModel(boxUrl).then(function(model) {
+            model.show = true;
+            model.zoomTo();
+            model.silhouetteColor = Color.GREEN;
+            model.silhouetteSize = 1.0;
+
+            var color = scene.renderForSpecs();
+
+            // Make the silhouette size big-ish
+            // Make the silhouette size = 0. Expect no commands to render
+            // Expect no commands when silhouette alpha is 0
+            // How to test - doesn't work when there is no normal attribute
+            // When color alpha is 0 and sihoutte size is 0, expect no commands
+            // When color alpha is 0 and silhouette exists, should render just the silhouette
+            // Renders two models at the same time ... how to test this? Maybe check the alpha blending...
+            // Opaque vs. translucent silhouette
+            // Works with and without OIT
+            // Works in 2d... test this for model coloring too....
+            // Check Model.silhouetteSupported
+            // Change silhouette color
+            // Change silhouette size
+            // Uses translucent silhouettes when model has translucent commands or is translucent
+        });
+    });
+
     describe('height referenced model', function() {
         function createMockGlobe() {
             var globe = {