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

GeoffreyBooth · targos · commit 8b29bb5cc98d · 2023-01-01T10:44:52.000Z
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>
diff --git a/test/parallel/test-v8-serialize-leak.js b/test/parallel/test-v8-serialize-leak.js
@@ -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}`);
 }

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,8 @@ const after = process.memoryUsage.rss();`
`22`	`22`
`23`	`23`	`if (process.config.variables.asan) {`
`24`	`24`	assert(after < before * 10, `asan: before=${before} after=${after}`);
	`25`	`+} else if (process.config.variables.node_builtin_modules_path) {`
	`26`	+ assert(after < before * 4, `node_builtin_modules_path: before=${before} after=${after}`);
`25`	`27`	`} else {`
`26`	`28`	assert(after < before * 2, `before=${before} after=${after}`);
`27`	`29`	`}`