Fix .match(RegExp) for everything not supported

shouldjs · Mar 9, 2017 · 52b984f · 52b984f
1 parent 64ad890
commit 52b984f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/ext/match.js b/lib/ext/match.js
@@ -97,6 +97,9 @@ export default function(should, Assertion) {
 
           this.assert(notMatchedProps.length === 0);
         } // should we try to convert to String and exec?
+        else {
+          this.assert(false);
+        }
       } else if (typeof other == 'function') {
         var res;
 

diff --git a/test/ext/match.test.js b/test/ext/match.test.js
@@ -1,5 +1,5 @@
 var err = require('../util').err;
-require('../../');
+var should = require('../../');
 
 describe('match', function() {
 
@@ -29,6 +29,8 @@ describe('match', function() {
 
     ({ a: 'a' }).should.not.match(/^http/);
 
+    should(null).not.match(/bar/);
+
     // positive false
     err(function() {
       ({ a: 'foo', c: 'barfoo' }).should.not.match(/foo$/);