Skip to content

Commit

Permalink
test: remove watchdog in test-debug-signal-cluster
Browse files Browse the repository at this point in the history
test-debug-signal-cluster contains a watchdog timer that results in
false positives in CI. Remove the watchdog timer and let the test runner
determine that the test has timed out.

PR-URL: #9476
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
Trott authored and addaleax committed Nov 22, 2016
1 parent 3efb43c commit 9491352
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions test/parallel/test-debug-signal-cluster.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,10 +65,6 @@ function onNoMoreDebuggerAgentsOutput() {
process.exit();
}

setTimeout(function testTimedOut() {
common.fail('test timed out');
}, common.platformTimeout(4000)).unref();

process.on('exit', function onExit() {
// Kill processes in reverse order to avoid timing problems on Windows where
// the parent process is killed before the children.
Expand Down

0 comments on commit 9491352

Please sign in to comment.