Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 1c8716e

Browse filesBrowse files
legendecasaduh95
authored andcommitted
test: increase debugger waitFor timeout on macOS
PR-URL: #60367 Refs: https://github.com/nodejs/node/actions/runs/18694915575/job/53309761263?pr=60343 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
1 parent 17b4f38 commit 1c8716e
Copy full SHA for 1c8716e

File tree

Expand file treeCollapse file tree

1 file changed

+3
-5
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+3
-5
lines changed
Open diff view settings
Collapse file

‎test/common/debugger.js‎

Copy file name to clipboardExpand all lines: test/common/debugger.js
+3-5Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,10 @@ const BREAK_MESSAGE = new RegExp('(?:' + [
77
'exception', 'other', 'promiseRejection', 'step',
88
].join('|') + ') in', 'i');
99

10-
// Some macOS machines require more time to receive the outputs from the client.
1110
let TIMEOUT = common.platformTimeout(10000);
12-
if (common.isWindows) {
13-
// Some of the windows machines in the CI need more time to receive
14-
// the outputs from the client.
15-
// https://github.com/nodejs/build/issues/3014
11+
// Some macOS and Windows machines require more time to receive the outputs from the client.
12+
// https://github.com/nodejs/build/issues/3014
13+
if (common.isWindows || common.isMacOS) {
1614
TIMEOUT = common.platformTimeout(15000);
1715
}
1816

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.