From 06149cc1597f88f26caa2c8e8bc9a0facedf01a5 Mon Sep 17 00:00:00 2001 From: Niek van der Maas 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 2ccfc9943a9175..f2b1b252291998 100644 --- a/test/parallel/test-v8-serdes.js +++ b/test/parallel/test-v8-serdes.js @@ -47,6 +47,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) => { @@ -60,6 +63,7 @@ const hostObject = new (internalBinding('js_stream').JSStream)(); ser.writeValue(new Proxy({}, {})); }, /foobar/); } +*/ { const ser = new v8.DefaultSerializer();