From cc36a632055ca0b3c06575eaef3a1dcc7870b03a Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Fri, 4 Nov 2016 23:04:36 -0700 Subject: [PATCH] test: remove watchdog in test-debug-signal-cluster MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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: https://github.com/nodejs/node/pull/9476 Reviewed-By: Michaƫl Zasso Reviewed-By: Colin Ihrig Reviewed-By: Santiago Gimeno Reviewed-By: James M Snell --- test/parallel/test-debug-signal-cluster.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/test/parallel/test-debug-signal-cluster.js b/test/parallel/test-debug-signal-cluster.js index 7c0c332801727b..d5319a616939d1 100644 --- a/test/parallel/test-debug-signal-cluster.js +++ b/test/parallel/test-debug-signal-cluster.js @@ -51,10 +51,6 @@ function onNoMoreLines() { 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.