can-reflect - npm Package Compare versions

Comparing version 1.10.1 to 1.10.2

package.json

 {
   "name": "can-reflect",
-  "version": "1.10.1",
+  "version": "1.10.2",
   "description": "reflection on unknown data types",
@@ -5,0 +5,0 @@ "homepage": "http://canjs.com",

reflections/observe/observe-test.js

@@ -73,9 +73,8 @@ var QUnit = require('steal-qunit');
 
-QUnit.test("getChangesDependencyRecord", function() {
-	try {
-		observeReflections.getChangesDependencyRecord({});
-		QUnit.ok(false, "should throw error");
-	} catch(e) {
-		QUnit.ok(true, "threw error");
-	}
+QUnit.test("getChangesDependencyRecord", function(assert) {
+	assert.equal(
+		typeof observeReflections.getChangesDependencyRecord({}),
+		"undefined",
+		"should return undefined when symbol is not implemented"
+	);
 });
@@ -82,0 +81,0 @@

reflections/observe/observe.js

@@ -247,7 +247,10 @@ var canSymbol = require("can-symbol");
 	 */
-	getChangesDependencyRecord: makeErrorIfMissing(
-		"can.getChangesDependencyRecord",
-		"can-reflect: can not determine dependencies"
-	),
+	getChangesDependencyRecord: function getChangesDependencyRecord(handler) {
+		var fn = handler[canSymbol.for("can.getChangesDependencyRecord")];
 
+		if (typeof fn === "function") {
+			return fn();
+		}
+	},
+
 	/**
@@ -254,0 +257,0 @@ * @function {Object, String} can-reflect/observe.keyHasDependencies keyHasDependencies

No alert changes

Improved metrics

Total package byte prevSize

148080

increased by0.05%

Lines of code

3984

increased by0.03%

No dependency changes