From 6eb6816e22f3863bb12563f2b507a40529793db5 Mon Sep 17 00:00:00 2001 From: Santiago Gimeno Date: Fri, 4 Nov 2016 09:55:36 +0100 Subject: [PATCH] test: fix flaky test-net-GH-5504 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The test is failing on `SmartOS` quite often. Removing the timeout seems to fix it. Fixes: https://github.com/nodejs/node/issues/8930 PR-URL: https://github.com/nodejs/node/pull/9461 Reviewed-By: Ben Noordhuis Reviewed-By: Colin Ihrig Reviewed-By: Michaël Zasso --- test/sequential/test-net-GH-5504.js | 8 -------- 1 file changed, 8 deletions(-) diff --git a/test/sequential/test-net-GH-5504.js b/test/sequential/test-net-GH-5504.js index c6e4e8e3d6a2b7..3ebf9db548d4db 100644 --- a/test/sequential/test-net-GH-5504.js +++ b/test/sequential/test-net-GH-5504.js @@ -48,14 +48,6 @@ function parent() { var spawn = require('child_process').spawn; var node = process.execPath; - setTimeout(function() { - if (s) s.kill(); - if (c) c.kill(); - setTimeout(function() { - throw new Error('hang'); - }); - }, common.platformTimeout(2000)).unref(); - var s = spawn(node, [__filename, 'server'], { env: Object.assign(process.env, { NODE_DEBUG: 'net'