From 47cbb8237fc59bd10552936a3dadc47303df8045 Mon Sep 17 00:00:00 2001 From: Joyee Cheung Date: Wed, 24 Aug 2022 16:36:34 +0800 Subject: [PATCH] test: increase timeout in the debugger tests on Windows 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. --- test/common/debugger.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/test/common/debugger.js b/test/common/debugger.js index d2ac4f3b6c5fbf..4aff5b9a0f74d9 100644 --- a/test/common/debugger.js +++ b/test/common/debugger.js @@ -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);