Commit ee2e641
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 <jasnell@gmail.com>1 parent a89eeca commit ee2e641
1 file changed
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -140,9 +140,7 @@ assert.throws(makeBlock(a.deepEqual, 'a', ['a']), a.AssertionError); | |||
| 140 | 140 | assert.throws(makeBlock(a.deepEqual, 'a', {0: 'a'}), a.AssertionError); | |
| 141 | 141 | assert.throws(makeBlock(a.deepEqual, 1, {}), a.AssertionError); | |
| 142 | 142 | 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); | ||
| 146 | 144 | ||
| 147 | 145 | // primitive wrappers and object | |
| 148 | 146 | assert.doesNotThrow(makeBlock(a.deepEqual, new String('a'), ['a']), | |
0 commit comments