making canReflect.hasOwnKey work

canjs · Sep 5, 2018 · ab9b443 · ab9b443
1 parent 8b9ea3c
commit ab9b443
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/can-simple-map.js b/can-simple-map.js
@@ -60,7 +60,7 @@ var SimpleMap = Construct.extend("SimpleMap",
 						}
 					}
 					//!steal-remove-end
-					
+
 					var dispatched = {
 						keyChanged: !had ? prop : undefined,
 						type: prop
@@ -203,7 +203,10 @@ var simpleMapProto = {
 	},
 	"can.getKeyDependencies": function(key) {
 		return undefined;
-	}	
+	},
+	"can.hasOwnKey": function(key){
+		return this._data.hasOwnProperty(key);
+	}
 };
 
 //!steal-remove-start

diff --git a/can-simple-map_test.js b/can-simple-map_test.js
@@ -242,6 +242,15 @@ QUnit.test("don't dispatch events for sets that don't change", 2, function(){
 	map.attr("foo","BAR");
 });
 
+QUnit.test("Reflect.hasOwnKey", function(){
+	var map = new SimpleMap({a: undefined, b: null, c: ""});
+
+	QUnit.ok( canReflect.hasOwnKey(map,"a"), "undefined is a key");
+	QUnit.ok( canReflect.hasOwnKey(map,"b"), "null is a key");
+	QUnit.ok( canReflect.hasOwnKey(map,"c"), "empty string is a key");
+	QUnit.ok( !canReflect.hasOwnKey(map,"d"), "no prop is not a key");
+});
+
 require("can-reflect-tests/observables/map-like/instance/on-get-set-delete-key")("", function(){
 	return new SimpleMap();
 });