test: add vm crash regression test

Refs: #34606 PR-URL: #34673 Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Aug 17, 2020 · e4d3dbd · e4d3dbd
1 parent 5b2a1f1
commit e4d3dbd
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/test/parallel/test-vm-set-property-proxy.js b/test/parallel/test-vm-set-property-proxy.js
@@ -0,0 +1,16 @@
+'use strict';
+const common = require('../common');
+const assert = require('assert');
+const vm = require('vm');
+
+// Regression test for https://github.com/nodejs/node/issues/34606
+
+const handler = {
+  getOwnPropertyDescriptor: common.mustCallAtLeast(() => {
+    return {};
+  })
+};
+
+const proxy = new Proxy({}, handler);
+assert.throws(() => vm.runInNewContext('p = 6', proxy),
+              /getOwnPropertyDescriptor/);