can-stache-key - npm Package Compare versions

Comparing version 1.2.0 to 1.3.0

can-stache-key-test.js

@@ -269,72 +269,38 @@ var observeReader = require("can-stache-key");
 
-QUnit.test("read sets parentHasKey even for undefined values", function() {
-	var child = {
-		prop: "foo"
+QUnit.test("parentHasKey and foundLastParent (#31)", function() {
+	var hasKeys = function(obj, keys) {
+		canReflect.assignSymbols(obj, {
+			"can.hasKey": function(key) {
+				return keys.indexOf(key) > -1;
+			}
+		});
 	};
-	canReflect.assignSymbols(child, {
-		"can.hasKey": function(key) {
-			return key === "prop" || key === "undefinedProp";
-		}
-	});
 
-	var parent = {
-		child: child
-	};
-	canReflect.assignSymbols(parent, {
-		"can.hasKey": function(key) {
-			return key === "child" || key === "undefinedChild";
-		}
-	});
+	var def = { ghi: undefined };
+	hasKeys(def, [ "ghi" ]);
 
-	var map = {
-		parent: parent
-	};
+	var abc = { def: def };
+	hasKeys(abc, [ "def" ]);
 
-	// map.parent
-	var reads = observeReader.reads("parent");
-	var value = observeReader.read(map, reads);
+	var parent = { abc: abc };
+	hasKeys(parent, [ "abc" ]);
 
-	QUnit.equal(value.value, parent, "parent.value === map");
-	QUnit.equal(value.parent, map, "parent.parent === map");
-	QUnit.equal(value.parentHasKey, true, "parent.parentHasKey === true");
+	var testCases = {
+		"abc.def.ghi": { parent: def, value: undefined, parentHasKey: true, foundLastParent: true },
+		"abc.def.jkl": { parent: def, value: undefined, parentHasKey: false, foundLastParent: true },
+		"abc.ghi.ghi": { parent: abc, value: undefined, parentHasKey: false, foundLastParent: false },
+		"def.ghi.jkl": { parent: parent, value: undefined, parentHasKey: false, foundLastParent: false }
+	};
 
-	// map.parent.child
-	reads = observeReader.reads("parent.child");
-	value = observeReader.read(map, reads);
+	var reads, actual, expected;
+	for (var key in testCases) {
+		reads = observeReader.reads(key);
+		actual = observeReader.read(parent, reads);
+		expected = testCases[key];
 
-	QUnit.equal(value.value, child, "parent.child.value === child");
-	QUnit.equal(value.parent, parent, "parent.child.parent === parent");
-	QUnit.equal(value.parentHasKey, true, "parent.child.parentHasKey === true");
-
-	// map.parent.undefinedChild
-	reads = observeReader.reads("parent.undefinedChild");
-	value = observeReader.read(map, reads);
-
-	QUnit.equal(value.value, undefined, "parent.undefinedChild.value === undefined");
-	QUnit.equal(value.parent, parent, "parent.undefinedChild.parent === parent");
-	QUnit.equal(value.parentHasKey, true, "parent.undefinedChild.parentHasKey === true");
-
-	// map.parent.undefinedChild.prop
-	reads = observeReader.reads("parent.undefinedChild.prop");
-	value = observeReader.read(map, reads);
-
-	QUnit.equal(value.value, undefined, "parent.undefinedChild.prop.value === undefined");
-	QUnit.equal(value.parent, parent, "parent.undefinedChild.prop.parent === parent");
-	QUnit.equal(value.parentHasKey, true, "parent.undefinedChild.prop.parentHasKey === true");
-
-	// map.parent.child.prop
-	reads = observeReader.reads("parent.child.prop");
-	value = observeReader.read(map, reads);
-
-	QUnit.equal(value.value, "foo", "parent.child.prop.value === 'foo'");
-	QUnit.equal(value.parent, child, "parent.child.prop.parent === child");
-	QUnit.equal(value.parentHasKey, true, "parent.child.prop.parentHasKey === true");
-
-	// map.parent.child.undefinedProp
-	reads = observeReader.reads("parent.child.undefinedProp");
-	value = observeReader.read(map, reads);
-
-	QUnit.equal(value.value, undefined, "parent.child.undefinedProp.value === undefined");
-	QUnit.equal(value.parent, child, "parent.child.undefinedProp.parent === child");
-	QUnit.equal(value.parentHasKey, true, "parent.child.undefinedProp.parentHasKey === true");
+		QUnit.equal(actual.value, expected.value, key + ".value");
+		QUnit.equal(actual.parent, expected.parent, key + ".parent");
+		QUnit.equal(actual.parentHasKey, expected.parentHasKey, key + ".parentHasKey");
+		QUnit.equal(actual.foundLastParent, expected.foundLastParent, key + ".foundLastParent");
+	}
 });

can-stache-key.js

@@ -133,3 +133,4 @@ var ObservationRecorder = require('can-observation-recorder');
 					parent: prev,
-					parentHasKey: parentHasKey
+					parentHasKey: parentHasKey,
+					foundLastParent: false
 				};
@@ -150,3 +151,4 @@ }
 			parent: prev,
-			parentHasKey: parentHasKey
+			parentHasKey: parentHasKey,
+			foundLastParent: true
 		};
@@ -153,0 +155,0 @@ },

package.json

 {
   "name": "can-stache-key",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "description": "Read and write keys on a value",
@@ -5,0 +5,0 @@ "homepage": "https://canjs.com",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Worsened metrics

Total package byte prevSize

27442

decreased by-4.38%

Lines of code

595

decreased by-3.88%

No dependency changes