diff options
author | AnnaMag <AnnaMag@users.noreply.github.com> | 2016-12-14 16:55:40 +0100 |
---|---|---|
committer | Italo A. Casas <me@italoacasas.com> | 2016-12-17 20:30:03 -0500 |
commit | 0d3ac89ff7e8e664bd0d09dc8de56f302cbc9ee0 (patch) | |
tree | e65c921a4f5466be12b9e007a0993c288b76c468 /test/known_issues | |
parent | 92ed1ab45059e3c822dc661a4917bb5232707315 (diff) | |
download | android-node-v8-0d3ac89ff7e8e664bd0d09dc8de56f302cbc9ee0.tar.gz android-node-v8-0d3ac89ff7e8e664bd0d09dc8de56f302cbc9ee0.tar.bz2 android-node-v8-0d3ac89ff7e8e664bd0d09dc8de56f302cbc9ee0.zip |
test: add known_issues test for #6287
Deleting property in the vm context has no effect
as reported in https://github.com/nodejs/node/issues/6287
The test is moved to the known_issues and will be fixed
with the 5.5 V8 API changes.
PR-URL: https://github.com/nodejs/node/pull/10272
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Diffstat (limited to 'test/known_issues')
-rw-r--r-- | test/known_issues/test-vm-deleting-property.js | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/test/known_issues/test-vm-deleting-property.js b/test/known_issues/test-vm-deleting-property.js new file mode 100644 index 0000000000..cda3371a33 --- /dev/null +++ b/test/known_issues/test-vm-deleting-property.js @@ -0,0 +1,15 @@ +'use strict'; +// Refs: https://github.com/nodejs/node/issues/6287 + +require('../common'); +const assert = require('assert'); +const vm = require('vm'); + +const context = vm.createContext(); +const res = vm.runInContext(` + this.x = 'prop'; + delete this.x; + Object.getOwnPropertyDescriptor(this, 'x'); +`, context); + +assert.strictEqual(res.value, undefined); |