runInNewContext does not pass properties that Enumerable= false

[shimondoodkin]

example:

var vm= require('vm');
var ctx = {};
Object.defineProperty(ctx, 'myNonEnumerablePropery', { value: 1 }); 
vm.runInNewContext('var result=typeof myNonEnumerablePropery', ctx);
console.log(ctx.result);

var ctx = {};
Object.defineProperty(ctx, 'myNonEnumerablePropery', { value: 1 , enumerable:true }); 
vm.runInNewContext('var result=typeof myNonEnumerablePropery', ctx);
console.log(ctx.result);

test case:

function node_script_cc_runInNewContext_ignores_non_enumerable()
{
 var assert=require('assert');
 var vm= require('vm');
 var ctx = {};
 Object.defineProperty(ctx, 'myNonEnumeratiblePropery', { value: 1 }); 
 vm.runInNewContext('var result=typeof myNonEnumeratiblePropery', ctx);
 assert.equal(ctx.result,'number', "runInNewContext ignores non enumerable ");
}
node_script_cc_runInNewContext_ignores_non_enumerable()

[ELLIOTTCABLE]

I’m having issues with this as well.

+1

[ELLIOTTCABLE]

As documented in #1801, I believe I’ve closed this with elliottcable@cf21650. I’d appreciate if you’d clone and compile against that, and see if you can still reproduce this issue. (Many of these issues seem related.)

[shimondoodkin]

works on v0.5.10-pre