Skip to content

Commit

Permalink
test: fix test broken under --node-builtin-modules-path
Browse files Browse the repository at this point in the history
PR-URL: #45894
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
  • Loading branch information
GeoffreyBooth authored and RafaelGSS committed Jan 5, 2023
1 parent 353dab5 commit 95ce16d
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/parallel/test-v8-serialize-leak.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,8 @@ const after = process.memoryUsage.rss();

if (process.config.variables.asan) {
assert(after < before * 10, `asan: before=${before} after=${after}`);
} else if (process.config.variables.node_builtin_modules_path) {
assert(after < before * 4, `node_builtin_modules_path: before=${before} after=${after}`);
} else {
assert(after < before * 2, `before=${before} after=${after}`);
}

0 comments on commit 95ce16d

Please sign in to comment.