From 989c2aaf830c974dc87a0f080834ff08bfd831ab Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Thu, 8 Nov 2018 10:03:30 -0800 Subject: [PATCH] test: fix flaky test-vm-timeout-escape-nexttick MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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: https://github.com/nodejs/node/issues/24120 PR-URL: https://github.com/nodejs/node/pull/24251 Reviewed-By: Refael Ackermann Reviewed-By: Richard Lau Reviewed-By: Michaƫl Zasso --- test/known_issues/test-vm-timeout-escape-nexttick.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/known_issues/test-vm-timeout-escape-nexttick.js b/test/known_issues/test-vm-timeout-escape-nexttick.js index 814da178fb2f78..40937c96d91a69 100644 --- a/test/known_issues/test-vm-timeout-escape-nexttick.js +++ b/test/known_issues/test-vm-timeout-escape-nexttick.js @@ -35,7 +35,7 @@ assert.throws(() => { nextTick, loop }, - { timeout: common.platformTimeout(5) } + { timeout: common.platformTimeout(10) } ); }, { code: 'ERR_SCRIPT_EXECUTION_TIMEOUT'