| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
1 parent 72710d0 commit 8e5c8a3
2 files changed
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -0,0 +1,20 @@ | |||
| 1 | + 'use strict'; | ||
| 2 | + | ||
| 3 | + // Sandbox throws in CopyProperties() despite no code being run | ||
| 4 | + // Issue: https://github.com/nodejs/node/issues/11902 | ||
| 5 | + | ||
| 6 | + | ||
| 7 | + require('../common'); | ||
| 8 | + const assert = require('assert'); | ||
| 9 | + const vm = require('vm'); | ||
| 10 | + | ||
| 11 | + const handler = { | ||
| 12 | + getOwnPropertyDescriptor: (target, prop) => { | ||
| 13 | + throw new Error('whoops'); | ||
| 14 | + } | ||
| 15 | + }; | ||
| 16 | + const sandbox = new Proxy({foo: 'bar'}, handler); | ||
| 17 | + const context = vm.createContext(sandbox); | ||
| 18 | + | ||
| 19 | + | ||
| 20 | + assert.doesNotThrow(() => vm.runInContext('', context)); | ||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -16,16 +16,3 @@ sandbox = { Proxy: Proxy }; | |||
| 16 | 16 | vm.runInNewContext('this.Proxy = Proxy', sandbox); | |
| 17 | 17 | assert.strictEqual(typeof sandbox.Proxy, 'function'); | |
| 18 | 18 | assert.strictEqual(sandbox.Proxy, Proxy); | |
| 19 | - | ||
| 20 | - // Handle a sandbox that throws while copying properties | ||
| 21 | - assert.throws(() => { | ||
| 22 | - const handler = { | ||
| 23 | - getOwnPropertyDescriptor: (target, prop) => { | ||
| 24 | - throw new Error('whoops'); | ||
| 25 | - } | ||
| 26 | - }; | ||
| 27 | - const sandbox = new Proxy({foo: 'bar'}, handler); | ||
| 28 | - const context = vm.createContext(sandbox); | ||
| 29 | - | ||
| 30 | - vm.runInContext('', context); | ||
| 31 | - }, /whoops/); | ||
| Back | FazBrowse Home | New Git URL |
0 commit comments