src: fix objectwrap test case

Refs: #485 The test case was relyingon the ordering of "for in" which is not guarranteed as per https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/for...in Update the testcase to check in an way that does not depend on ordering. PR-URL: #495 Refs: #485 Reviewed-By: Gabriel Schulhof <[email protected]> Reviewed-By: NickNaso <[email protected]>
nodejs · Jun 13, 2019 · ab7d8fc · ab7d8fc
1 parent 3b6b9eb
commit ab7d8fc
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/test/objectwrap.js b/test/objectwrap.js
@@ -73,12 +73,11 @@ const test = (binding) => {
         keys.push(key);
       }
 
-      assert.deepEqual(keys, [
-        'testGetSet',
-        'testGetter',
-        'testValue',
-        'testMethod'
-      ]);
+      assert(keys.length = 4);
+      assert(obj.testGetSet);
+      assert(obj.testGetter);
+      assert(obj.testValue);
+      assert(obj.testMethod);
     }
   };