Skip to content

Commit

Permalink
test: unflake test-vm-timeout-escape-nexttick
Browse files Browse the repository at this point in the history
This wasn't failing on arm boxes, increase the `runInNewContext()`
timeout a bit to make sure the code it's allowed to fail.

PR-URL: #48078
Fixes: #43931
Fixes: #42496
Fixes: #47715
Fixes: #47259
Fixes: #47241
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Debadree Chatterjee <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Rafael Gonzaga <[email protected]>
Reviewed-By: Richard Lau <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
  • Loading branch information
santigimeno authored and nodejs-github-bot committed May 24, 2023
1 parent 9e68f94 commit 1620626
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/known_issues/test-vm-timeout-escape-nexttick.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ const NS_PER_MS = 1000000n;
const hrtime = process.hrtime.bigint;
const nextTick = process.nextTick;

const waitDuration = common.platformTimeout(100n);
const waitDuration = common.platformTimeout(200n);

function loop() {
const start = hrtime();
Expand All @@ -38,7 +38,7 @@ for (let i = 0; i < 4; i++) {
nextTick,
loop,
},
{ timeout: common.platformTimeout(10) },
{ timeout: common.platformTimeout(100) },
);
}, {
code: 'ERR_SCRIPT_EXECUTION_TIMEOUT',
Expand Down

0 comments on commit 1620626

Please sign in to comment.