Commit 89e23b7
test: deflake test-debugger-probe-timeout
On slow machines in the CI, the 200ms timeout may fire before
the probed process finishes evaluation. Increase the timeout
to common.platformTimeout(1000) to accomondate.
Signed-off-by: Joyee Cheung <joyeec9h3@gmail.com>
PR-URL: #63547
Refs: https://github.com/nodejs/reliability/blob/main/reports/2026-05-24.md
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 68edc2b commit 89e23b7
1 file changed
Lines changed: 3 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -9,10 +9,11 @@ const { spawnSyncAndExit } = require('../common/child_process'); | |||
| 9 | 9 | const { assertProbeJson } = require('../common/debugger-probe'); | |
| 10 | 10 | const cwd = fixtures.path('debugger'); | |
| 11 | 11 | ||
| 12 | + const timeout = common.platformTimeout(1000); | ||
| 12 | 13 | spawnSyncAndExit(process.execPath, [ | |
| 13 | 14 | 'inspect', | |
| 14 | 15 | '--json', | |
| 15 | - '--timeout=200', | ||
| 16 | + `--timeout=${timeout}`, | ||
| 16 | 17 | '--probe', 'probe-timeout.js:99', | |
| 17 | 18 | '--expr', '1', | |
| 18 | 19 | 'probe-timeout.js', | |
@@ -31,7 +32,7 @@ spawnSyncAndExit(process.execPath, [ | |||
| 31 | 32 | pending: [0], | |
| 32 | 33 | error: { | |
| 33 | 34 | code: 'probe_timeout', | |
| 34 | - message: 'Timed out after 200ms waiting for probes: probe-timeout.js:99', | ||
| 35 | + message: `Timed out after ${timeout}ms waiting for probes: probe-timeout.js:99`, | ||
| 35 | 36 | }, | |
| 36 | 37 | }], | |
| 37 | 38 | }); | |
0 commit comments