Skip to content

Commit bc6562f

Browse files
committed
test: increase debugger waitFor timeout on macOS and Windows
The test-debugger-restart-message test is flaky on macOS CI because the Debugger.getScriptSource CDP round-trip in the initial break handler can be slow under CI load, causing the 15s timeout to be exceeded. Increase the macOS/Windows timeout from 15s to 30s to provide adequate headroom.
1 parent 95245a7 commit bc6562f

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

test/common/debugger.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,12 @@ const BREAK_MESSAGE = new RegExp('(?:' + [
88
].join('|') + ') in', 'i');
99

1010
let TIMEOUT = common.platformTimeout(10000);
11-
// Some macOS and Windows machines require more time to receive the outputs from the client.
11+
// Some macOS and Windows machines require more time to receive the outputs from the
12+
// client, especially under CI load where the async Debugger.getScriptSource CDP
13+
// round-trip in the initial break handler can be slow.
1214
// https://github.com/nodejs/build/issues/3014
1315
if (common.isWindows || common.isMacOS) {
14-
TIMEOUT = common.platformTimeout(15000);
16+
TIMEOUT = common.platformTimeout(30000);
1517
}
1618

1719
function isPreBreak(output) {

0 commit comments

Comments
 (0)