Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Merge pull request #1005 from meeber/skip-failing-tests
test: skip failing `.include` tests in IE11
  • Loading branch information
keithamus committed Jul 10, 2017
2 parents 2eddd79 + 11a6f85 commit 3397dda
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 3 deletions.
6 changes: 5 additions & 1 deletion test/assert.js
Expand Up @@ -657,7 +657,11 @@ describe('assert', function () {

assert.include(set, val);
assert.include(set, 2);
assert.include(set, 0);
if (set.has(0)) {
// This test is skipped in IE11 because (contrary to spec) IE11 uses
// SameValue instead of SameValueZero equality for sets.
assert.include(set, 0);
}
assert.include(set, NaN);
}

Expand Down
6 changes: 5 additions & 1 deletion test/expect.js
Expand Up @@ -1910,7 +1910,11 @@ describe('expect', function () {
expect(set).to.not.include([{a: 1}]);
expect(set).to.include(2);
expect(set).to.not.include(3);
expect(set).to.include(0);
if (set.has(0)) {
// This test is skipped in IE11 because (contrary to spec) IE11 uses
// SameValue instead of SameValueZero equality for sets.
expect(set).to.include(0);
}
expect(set).to.include(NaN);
}

Expand Down
6 changes: 5 additions & 1 deletion test/should.js
Expand Up @@ -1568,7 +1568,11 @@ describe('should', function() {
set.should.not.include([{a: 1}]);
set.should.include(2);
set.should.not.include(3);
set.should.include(0);
if (set.has(0)) {
// This test is skipped in IE11 because (contrary to spec) IE11 uses
// SameValue instead of SameValueZero equality for sets.
set.should.include(0);
}
set.should.include(NaN);
}

Expand Down

0 comments on commit 3397dda

Please sign in to comment.