Exceptions in getters/setters with vm.runInNewContext don't get raised

[cloudhead]

The error here doesn't get raised, it fails silently.

var ctx = {};
Object.defineProperty(ctx, 'poo', { get: function () { throw new(Error) } }); 
vm.runInNewContext('this.poo', ctx);

[shimondoodkin]

the default of defineProperty is Enumerable=false
the correct diagnosis is that runInNewContext internals do not copy not Enumerable values
as in issue: #1402

also with getters and setters there is another issue
#540

a solution could be to put getters and setters in a sub object
or to define them from inside the script.

thanks for reporting this

i think this issue can be closed

[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.)

[isaacs]

Closed by 200df86