From 0f99320aa0d13c83e41d14f3cbbdbd12884ce5eb Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Mon, 12 Oct 2015 09:50:56 -0700 Subject: [PATCH] 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: https://github.com/nodejs/node/pull/3327 Reviewed-By: James M Snell --- test/parallel/test-assert.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/test/parallel/test-assert.js b/test/parallel/test-assert.js index ce19462a62387c..db36abc184b024 100644 --- a/test/parallel/test-assert.js +++ b/test/parallel/test-assert.js @@ -140,9 +140,7 @@ assert.throws(makeBlock(a.deepEqual, 'a', ['a']), a.AssertionError); assert.throws(makeBlock(a.deepEqual, 'a', {0: 'a'}), a.AssertionError); assert.throws(makeBlock(a.deepEqual, 1, {}), a.AssertionError); assert.throws(makeBlock(a.deepEqual, true, {}), a.AssertionError); -if (typeof Symbol === 'symbol') { - assert.throws(makeBlock(assert.deepEqual, Symbol(), {}), a.AssertionError); -} +assert.throws(makeBlock(a.deepEqual, Symbol(), {}), a.AssertionError); // primitive wrappers and object assert.doesNotThrow(makeBlock(a.deepEqual, new String('a'), ['a']),