From 27c06535175f4ef908e0a77c1aa0598b8b031fdd Mon Sep 17 00:00:00 2001
From: Anna Henningsen <anna@addaleax.net>
Date: Fri, 7 Aug 2020 23:41:00 +0200
Subject: [PATCH] test: add vm crash regression test

Refs: https://github.com/nodejs/node/issues/34606
PR-URL: https://github.com/nodejs/node/pull/34673
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 test/parallel/test-vm-set-property-proxy.js | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)
 create mode 100644 test/parallel/test-vm-set-property-proxy.js

diff --git a/test/parallel/test-vm-set-property-proxy.js b/test/parallel/test-vm-set-property-proxy.js
new file mode 100644
index 00000000000000..2e3293bf62947c
--- /dev/null
+++ 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/);