Skip to content

Commit

Permalink
test: add known_issues test for #6287
Browse files Browse the repository at this point in the history
Deleting property in the vm context has no effect
as reported in #6287
The test is moved to the known_issues and will be fixed
with the 5.5 V8 API changes.

PR-URL: #10272
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
AnnaMag authored and cjihrig committed Dec 20, 2016
1 parent 30b44f3 commit 1726a17
Showing 1 changed file with 15 additions and 0 deletions.
15 changes: 15 additions & 0 deletions test/known_issues/test-vm-deleting-property.js
Original file line number Diff line number Diff line change
@@ -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);

0 comments on commit 1726a17

Please sign in to comment.