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 ab8c631 commit 47cbb82
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion test/common/debugger.js
Original file line number Diff line number Diff line change
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 47cbb82

Please sign in to comment.