Skip to content

Commit

Permalink
test: fix flaky test-net-GH-5504
Browse files Browse the repository at this point in the history
The test is failing on `SmartOS` quite often. Removing the timeout seems
to fix it.

Fixes: #8930
PR-URL: #9461
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
  • Loading branch information
santigimeno authored and evanlucas committed Nov 7, 2016
1 parent f640baf commit 6eb6816
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions test/sequential/test-net-GH-5504.js
Original file line number Diff line number Diff line change
Expand Up @@ -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'
Expand Down

0 comments on commit 6eb6816

Please sign in to comment.