Browse Source

EAGLESIX-2892 cleaned up DepsTracker tests, todos

Austin Meagher 11 years ago
parent
commit
d21ec422c5
2 changed files with 3 additions and 9 deletions
  1. 1 0
      lib/pipeline/DepsTracker.js
  2. 2 9
      test/lib/pipeline/DepsTracker.js

+ 1 - 0
lib/pipeline/DepsTracker.js

@@ -98,6 +98,7 @@ proto.toParsedDeps = function toParsedDeps() {
         }
 
         last = it + ".";
+        // TODO: set nested field to true; i.e. a.b.c = true, not a = true
         doc[it] = true;
     });
 

+ 2 - 9
test/lib/pipeline/DepsTracker.js

@@ -71,15 +71,8 @@ module.exports = {
             },
             "should be able to parse dependencies with subfields": function() {
                 var deps = new DepsTracker(),
-                    expected = {"_fields":{"a":true}};
-                deps.fields = {'a':1,'a.b':1};
-
-                assert.deepEqual(expected, deps.toParsedDeps());
-            },
-            "should be able to parse dependencies with arrays": function() {
-                var deps = new DepsTracker(),
-                    expected = {"_fields":{"a":true,"b":true}};
-                deps.fields = {'a':1,'b':[{'c':1},{'d':'d.e'}]};
+                    expected = {"_fields":{"a.b":true}};
+                deps.fields = {'a.b':1};
 
                 assert.deepEqual(expected, deps.toParsedDeps());
             }