.throw() should check that given object is a function. Fix shouldjs#27

HaraldSchlager · Dec 8, 2014 · 481ab87 · 481ab87
1 parent 1f07d24
commit 481ab87
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/lib/ext/error.js b/lib/ext/error.js
@@ -14,6 +14,8 @@ module.exports = function(should, Assertion) {
       , errorInfo = ''
       , thrown = false;
 
+    this.is.a.Function;
+
     var errorMatched = true;
 
     try {

diff --git a/test/ext/error.test.js b/test/ext/error.test.js
@@ -3,6 +3,10 @@ var err = require('../util').err,
 
 describe('error', function() {
   it('test throw()', function() {
+    err(function() {
+      'a'.should.throw();
+    }, "expected 'a' to be a function");
+
     (function(){}).should.not.throw();
     (function(){ throw new Error('fail') }).should.throw();