test: increase limit again for network space overhead test · nodejs/node@2952c5d · GitHub

nodejs / node Public

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

• Code
• Issues 632
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit 2952c5d

Browse filesBrowse the repository at this point in the historyBrowse files

targos

authored and

MylesBorins

committed

test: increase limit again for network space overhead test

On certain platforms, updating V8 to 7.9 made the numbers a bit higher than the current limit. Backport-PR-URL: #30513 PR-URL: #30020 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

1 parent f0ff2ed commit 2952c5d
Copy full SHA for 2952c5d

1 file changed

+1-1Lines changed: 1 addition & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/sequential
  • test-net-bytes-per-incoming-chunk-overhead.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/sequential/test-net-bytes-per-incoming-chunk-overhead.js‎

Copy file name to clipboardExpand all lines: test/sequential/test-net-bytes-per-incoming-chunk-overhead.js

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -37,5 +37,5 @@ process.on('exit', () => {
37	37	const bytesPerChunk =
38	38	(process.memoryUsage().rss - baseRSS) / receivedChunks.length;
39	39	// We should always have less than one page (usually ~ 4 kB) per chunk.
40		- assert(bytesPerChunk < 600, `measured ${bytesPerChunk} bytes per chunk`);
	40	+ assert(bytesPerChunk < 650, `measured ${bytesPerChunk} bytes per chunk`);
41	41	});

0 commit comments

Comments

0 (0)