Skip to content

Commit

Permalink
test: increase timeout in the debugger tests on Windows
Browse files Browse the repository at this point in the history
On certain Windows machines it may take longer for the testing
processes to receive outputs from the debugger process. Increasing
the timeout to 15 seconds to avoid failing the tests in those
cases.
  • Loading branch information
joyeecheung committed Aug 24, 2022
1 parent e123a52 commit f7b0721
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion test/common/debugger.js
Expand Up @@ -7,7 +7,13 @@ const BREAK_MESSAGE = new RegExp('(?:' + [
'exception', 'other', 'promiseRejection',
].join('|') + ') in', 'i');

const TIMEOUT = common.platformTimeout(5000);
let TIMEOUT = common.platformTimeout(5000);
if (common.isWindows) {
// Some of the windows machines in the CI need more time to receive
// the outputs from the client.
// https://github.com/nodejs/build/issues/3014
TIMEOUT = common.platformTimeout(15000);
}

function isPreBreak(output) {
return /Break on start/.test(output) && /1 \(function \(exports/.test(output);
Expand Down

0 comments on commit f7b0721

Please sign in to comment.