Skip to content

Commit 76ebc4b

Browse files
Trottruyadorno
authored andcommitted
test: increase wiggle room for memory in test-worker-resource-limits
PR-URL: #37901 Fixes: #37844 Reviewed-By: Daniel Bevenius <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
1 parent 649e04c commit 76ebc4b

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

test/parallel/test-worker-resource-limits.js

+3-2
Original file line numberDiff line numberDiff line change
@@ -35,9 +35,10 @@ if (!process.env.HAS_STARTED_WORKER) {
3535
assert.deepStrictEqual(resourceLimits, testResourceLimits);
3636
const array = [];
3737
while (true) {
38-
// Leave 10 % wiggle room here.
38+
// Leave 10% wiggle room here, and 20% on debug builds.
39+
const wiggleRoom = common.buildType === 'Release' ? 1.1 : 1.2;
3940
const usedMB = v8.getHeapStatistics().used_heap_size / 1024 / 1024;
40-
assert(usedMB < resourceLimits.maxOldGenerationSizeMb * 1.1);
41+
assert(usedMB < resourceLimits.maxOldGenerationSizeMb * wiggleRoom);
4142

4243
let seenSpaces = 0;
4344
for (const { space_name, space_size } of v8.getHeapSpaceStatistics()) {

0 commit comments

Comments
 (0)