Skip to content

test: fix test broken under --node-builtin-modules-path

Rodrigo Muino Tomonari requested to merge github/fork/GeoffreyBooth/fix-v8-leak-test into main

When developing using ./configure --node-builtin-modules-path $(pwd), the test test/parallel/test-v8-serialize-leak.js always fails (at least for me). This update adds a tweak to account for the different memory usage under --node-builtin-modules-path, to make the test pass. Alternatively we could have this test be skipped under --node-builtin-modules-path.

See also https://openjs-foundation.slack.com/archives/C019Y2T6STH/p1664596265175919

Merge request reports