Browse Source

test: added test for indexed properties

Currently, indexed properties are correctly copied
onto the sandbox by CopyProperties().
This will break when CopyProperties() is removed
after adjusting NamedPropertyHandlerConfiguration
config() to use property callbacks from the new
V8 API. To fix it, we will set a config for indexed
properties.

This test is a preparation step for the patch
that removes CopyProperties().

PR-URL: https://github.com/nodejs/node/pull/11769
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
v7.x
AnnaMag 8 years ago
committed by Italo A. Casas
parent
commit
2649dab274
No known key found for this signature in database GPG Key ID: 23EFEFE93C4CFFFE
  1. 17
      test/parallel/test-vm-indexed-properties.js

17
test/parallel/test-vm-indexed-properties.js

@ -0,0 +1,17 @@
'use strict';
require('../common');
const assert = require('assert');
const vm = require('vm');
const code = `Object.defineProperty(this, 99, {
value: 20,
enumerable: true
});`;
const sandbox = {};
const ctx = vm.createContext(sandbox);
vm.runInContext(code, ctx);
assert.strictEqual(sandbox[99], 20);
Loading…
Cancel
Save