review comments fixes

test/assert.js

@@ -1456,6 +1456,10 @@ describe('assert', function () {
       assert.property(undefined, 'a', 'blah');
     }, "blah: Target cannot be null or undefined.");
 
+    err(function () {
+      assert.property({a:1}, {'a':'1'}, 'blah');
+    }, 'blah: the argument to property must be a string, number, or symbol');
+
     err(function () {
       assert.propertyVal(dummyObj, 'a', '2', 'blah');
     }, "blah: expected { a: '1' } to have property 'a' of '2', but got '1'");

test/expect.js

@@ -1463,9 +1463,9 @@ describe('expect', function () {
       expect(undefined, 'blah').to.have.property("a");
     }, "blah: Target cannot be null or undefined.");
 
-    expect(function () {
-      expect({a:1}).to.have.property(null);
-    }).to.throw('the argument to property must be a string, number, or symbol');
+    err(function () {
+      expect({a:1}, 'blah').to.have.property(null)
+    }, "blah: the argument to property must be a string, number, or symbol");
   });
 
   it('property(name, val)', function(){
@@ -1760,9 +1760,9 @@ describe('expect', function () {
         .to.have.nested.property('foo.bar');
     }, "expected { 'foo.bar': 'baz' } to have nested property 'foo.bar'");
 
-    expect(function () {
-      expect({a:1}).to.have.nested.property({'a':'1'});
-    }).to.throw('the argument to property must be a string when using nested syntax');
+    err(function () {
+      expect({a:1}, 'blah').to.have.nested.property({'a':'1'});
+    }, "blah: the argument to property must be a string when using nested syntax");
   });
 
   it('nested.property(name, val)', function(){

test/should.js

@@ -1182,6 +1182,10 @@ describe('should', function() {
     err(function() {
       ({a: {b: 1}}).should.have.own.nested.property("a.b");
     }, "The \"nested\" and \"own\" flags cannot be combined.");
+
+    err(function () {
+      ({a:1}).should.have.property(undefined);
+    }, "the argument to property must be a string, number, or symbol");
   });
 
   it('property(name, val)', function(){
@@ -1409,13 +1413,9 @@ describe('should', function() {
 
     ({a:1}).should.have.nested.property('a');
 
-    (function () {
-      ({a:1}).should.have.nested.property('{a:1}');
-    }).should.not.throw('the argument to property must be a string when using nested syntax');
-
-    (function () {
+    err(function(){
       ({a:1}).should.have.nested.property({'a':'1'});
-    }).should.throw('the argument to property must be a string when using nested syntax');
+    }, "the argument to property must be a string when using nested syntax");
 
     err(function(){
       ({ 'foo.bar': 'baz' }).should.have.nested.property('foo.bar');