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: nodejs#48078
Fixes: nodejs#43931
Fixes: nodejs#42496
Fixes: nodejs#47715
Fixes: nodejs#47259
Fixes: nodejs#47241
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Debadree Chatterjee <debadree333@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
  • Loading branch information
santigimeno authored and Ceres6 committed Aug 14, 2023
1 parent 82d2233 commit 9925783
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 9925783

Please sign in to comment.