From ea96bea195809b43991fafe57829c023f236b25c Mon Sep 17 00:00:00 2001
From: Niek van der Maas <mail@niekvandermaas.nl>
Date: Tue, 16 Mar 2021 11:30:24 +0100
Subject: [PATCH] [#11570] Disable Proxy test in parallel/test-v8-serdes.js

See: https://bugs.chromium.org/p/v8/issues/detail?id=11570

This PR disables the Proxy test in test/parallel/test-v8-serdes.js while https://chromium-review.googlesource.com/c/v8/v8/+/2739980 is being merged.

Will do another PR to re-enabled the (fixed) test once the 2739980 has landed.
---
 test/parallel/test-v8-serdes.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/test/parallel/test-v8-serdes.js b/test/parallel/test-v8-serdes.js
index 1b6638ac1a90bd..e7b162bce71e02 100644
--- a/test/parallel/test-v8-serdes.js
+++ b/test/parallel/test-v8-serdes.js
@@ -55,6 +55,9 @@ const hostObject = new (internalBinding('js_stream').JSStream)();
   }
 }
 
+/*
+// TODO(v8:11570): We disable the following test, while https://chromium-review.googlesource.com/c/v8/v8/+/2739980 is being merged
+// After merging, a follow-up PR will be made to re-enable this test with changed behavior
 {
   const ser = new v8.DefaultSerializer();
   ser._getDataCloneError = common.mustCall((message) => {
@@ -68,6 +71,7 @@ const hostObject = new (internalBinding('js_stream').JSStream)();
     ser.writeValue(new Proxy({}, {}));
   }, /foobar/);
 }
+*/
 
 {
   const ser = new v8.DefaultSerializer();