test: replace commented out expectations with tests

Signed-off-by: Darshan Sen <raisinten@gmail.com>
nodejs · Jan 23, 2022 · e8042e0 · e8042e0
1 parent b2edcfe
commit e8042e0
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/test/js-native-api/test_symbol/test2.js b/test/js-native-api/test_symbol/test2.js
@@ -6,10 +6,12 @@ const assert = require('assert');
 const test_symbol = require(`./build/${common.buildType}/test_symbol`);
 
 const fooSym = test_symbol.New('foo');
+assert.strictEqual(fooSym.toString(), 'Symbol(foo)');
+
 const myObj = {};
 myObj.foo = 'bar';
 myObj[fooSym] = 'baz';
-Object.keys(myObj); // -> [ 'foo' ]
-Object.getOwnPropertyNames(myObj); // -> [ 'foo' ]
-Object.getOwnPropertySymbols(myObj); // -> [ Symbol(foo) ]
-assert.strictEqual(Object.getOwnPropertySymbols(myObj)[0], fooSym);
+
+assert.deepStrictEqual(Object.keys(myObj), ['foo']);
+assert.deepStrictEqual(Object.getOwnPropertyNames(myObj), ['foo']);
+assert.deepStrictEqual(Object.getOwnPropertySymbols(myObj), [fooSym]);