Skip to content

Commit ee2e641

Browse files
Trottrvagg
authored andcommitted
test: add Symbol test for assert.deepEqual()
The existing test never ran because typeof Symbol === 'function' and not 'symbol'. We have Symbols now so remove the check and just run the test. PR-URL: #3327 Reviewed-By: James M Snell <[email protected]>
1 parent a89eeca commit ee2e641

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

test/parallel/test-assert.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -140,9 +140,7 @@ assert.throws(makeBlock(a.deepEqual, 'a', ['a']), a.AssertionError);
140140
assert.throws(makeBlock(a.deepEqual, 'a', {0: 'a'}), a.AssertionError);
141141
assert.throws(makeBlock(a.deepEqual, 1, {}), a.AssertionError);
142142
assert.throws(makeBlock(a.deepEqual, true, {}), a.AssertionError);
143-
if (typeof Symbol === 'symbol') {
144-
assert.throws(makeBlock(assert.deepEqual, Symbol(), {}), a.AssertionError);
145-
}
143+
assert.throws(makeBlock(a.deepEqual, Symbol(), {}), a.AssertionError);
146144

147145
// primitive wrappers and object
148146
assert.doesNotThrow(makeBlock(a.deepEqual, new String('a'), ['a']),

0 commit comments

Comments
 (0)