test: add failing vm tests to known_issues

Currently, `Reflect.ownKeys(this)`, `Object.getOwnPropertyNames(this)`
and `Object.getOwnPropertySymbols(this)` when run in a sandbox, fails
to retrieve Symbol and non-enumerable values.
PR-URL: #16410
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>

Author: targos

test/known_issues/test-vm-ownkeys.js

@@ -0,0 +1,28 @@
+'use strict';
+
+require('../common');
+constvm=require('vm');
+constassert=require('assert');
+
+constsym1=Symbol('1');
+constsym2=Symbol('2');
+constsandbox={
+a: true,
+[sym1]: true
+};
+Object.defineProperty(sandbox,'b',{value: true});
+Object.defineProperty(sandbox,sym2,{value: true});
+
+constctx=vm.createContext(sandbox);
+
+// Sanity check
+// Please uncomment these when the test is no longer broken
+// assert.deepStrictEqual(Reflect.ownKeys(sandbox), ['a', 'b', sym1, sym2]);
+// assert.deepStrictEqual(Object.getOwnPropertyNames(sandbox), ['a', 'b']);
+// assert.deepStrictEqual(Object.getOwnPropertySymbols(sandbox), [sym1, sym2]);
+
+constnativeKeys=vm.runInNewContext('Reflect.ownKeys(this);');
+constownKeys=vm.runInContext('Reflect.ownKeys(this);',ctx);
+constrestKeys=ownKeys.filter((key)=>!nativeKeys.includes(key));
+// this should not fail
+assert.deepStrictEqual(Array.from(restKeys),['a','b',sym1,sym2]);

test/known_issues/test-vm-ownpropertynames.js

@@ -0,0 +1,28 @@
+'use strict';
+
+require('../common');
+constvm=require('vm');
+constassert=require('assert');
+
+constsym1=Symbol('1');
+constsym2=Symbol('2');
+constsandbox={
+a: true,
+[sym1]: true
+};
+Object.defineProperty(sandbox,'b',{value: true});
+Object.defineProperty(sandbox,sym2,{value: true});
+
+constctx=vm.createContext(sandbox);
+
+// Sanity check
+// Please uncomment these when the test is no longer broken
+// assert.deepStrictEqual(Reflect.ownKeys(sandbox), ['a', 'b', sym1, sym2]);
+// assert.deepStrictEqual(Object.getOwnPropertyNames(sandbox), ['a', 'b']);
+// assert.deepStrictEqual(Object.getOwnPropertySymbols(sandbox), [sym1, sym2]);
+
+constnativeNames=vm.runInNewContext('Object.getOwnPropertyNames(this);');
+constownNames=vm.runInContext('Object.getOwnPropertyNames(this);',ctx);
+constrestNames=ownNames.filter((name)=>!nativeNames.includes(name));
+// this should not fail
+assert.deepStrictEqual(Array.from(restNames),['a','b']);

test/known_issues/test-vm-ownpropertysymbols.js

@@ -0,0 +1,28 @@
+'use strict';
+
+require('../common');
+constvm=require('vm');
+constassert=require('assert');
+
+constsym1=Symbol('1');
+constsym2=Symbol('2');
+constsandbox={
+a: true,
+[sym1]: true
+};
+Object.defineProperty(sandbox,'b',{value: true});
+Object.defineProperty(sandbox,sym2,{value: true});
+
+constctx=vm.createContext(sandbox);
+
+// Sanity check
+// Please uncomment these when the test is no longer broken
+// assert.deepStrictEqual(Reflect.ownKeys(sandbox), ['a', 'b', sym1, sym2]);
+// assert.deepStrictEqual(Object.getOwnPropertyNames(sandbox), ['a', 'b']);
+// assert.deepStrictEqual(Object.getOwnPropertySymbols(sandbox), [sym1, sym2]);
+
+constnativeSym=vm.runInNewContext('Object.getOwnPropertySymbols(this);');
+constownSym=vm.runInContext('Object.getOwnPropertySymbols(this);',ctx);
+constrestSym=ownSym.filter((sym)=>!nativeSym.includes(sym));
+// this should not fail
+assert.deepStrictEqual(Array.from(restSym),[sym1,sym2]);