Skip to content

Commit

Permalink
test: fix flaky test-vm-timeout-escape-nexttick
Browse files Browse the repository at this point in the history
Increase the VM timeout. If it is too small, the VM does not exit before
the code has a chance to create the problematic condition that causes
the timeout to be ignored.

Fixes: #24120

PR-URL: #24251
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
  • Loading branch information
Trott authored and BridgeAR committed Nov 13, 2018
1 parent 7bcc4cc commit 989c2aa
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/known_issues/test-vm-timeout-escape-nexttick.js
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ assert.throws(() => {
nextTick,
loop
},
{ timeout: common.platformTimeout(5) }
{ timeout: common.platformTimeout(10) }
);
}, {
code: 'ERR_SCRIPT_EXECUTION_TIMEOUT'
Expand Down

0 comments on commit 989c2aa

Please sign in to comment.