Fix TypeError in .containEql with checking that property exists.

tj · Feb 25, 2014 · a5d708d · a5d708d
1 parent cc7e885
commit a5d708d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib/ext/property.js b/lib/ext/property.js
@@ -166,7 +166,7 @@ module.exports = function(should, Assertion) {
     } else if(util.isObject(obj)) {
       // object contains object case
       util.forOwn(other, function(value, key) {
-        obj[key].should.eql(value);
+        obj.should.have.property(key, value);
       });
     } else {
       //other uncovered cases

diff --git a/test/ext/property.test.js b/test/ext/property.test.js
@@ -167,6 +167,8 @@ module.exports['property'] = {
     [{a: 'a'}, {b: 'b', c: 'c'}].should.containEql({a: 'a'});
     [{a: 'a'}, {b: 'b', c: 'c'}].should.not.containEql({b: 'b'});
 
+    ({}).should.not.containEql({ a: 10 });
+
     ({ b: 10 }).should.containEql({ b: 10 });
     [1, 2, 3].should.containEql(1);
     ([1, 2, { a: 10 }]).should.containEql({ a: 10 });