test: fix test broken under --node-builtin-modules-path · nodejs/node@95ce16d · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 633
• Pull requests 592
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 95ce16d

Browse filesBrowse the repository at this point in the historyBrowse files

GeoffreyBooth

authored and

RafaelGSS

committed

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

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>

1 parent 353dab5 commit 95ce16d
Copy full SHA for 95ce16d

1 file changed

+2Lines changed: 2 additions & 0 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-v8-serialize-leak.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-v8-serialize-leak.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-v8-serialize-leak.js

+2Lines changed: 2 additions & 0 deletions

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	}

0 commit comments

Comments

0 (0)