Merge pull request #512 from dereke/master

fixed expect('').to.contain('') so it passes
chaijs · Sep 11, 2015 · cf9f5d9 · cf9f5d9
2 parents 5044dc0 + eaba0be
commit cf9f5d9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/chai/core/assertions.js b/lib/chai/core/assertions.js
@@ -214,7 +214,7 @@ module.exports = function (chai, _) {
       for (var k in val) subset[k] = obj[k];
       expected = _.eql(subset, val);
     } else {
-      expected = obj && ~obj.indexOf(val);
+      expected = (obj != undefined) && ~obj.indexOf(val);
     }
     this.assert(
         expected

diff --git a/test/assert.js b/test/assert.js
@@ -430,6 +430,7 @@ describe('assert', function () {
 
   it('include', function() {
     assert.include('foobar', 'bar');
+    assert.include('', '');
     assert.include([ 1, 2, 3], 3);
     assert.include({a:1, b:2}, {b:2});