Skip to content

Commit

Permalink
test: increate test-net-nodejsGH-5504 timeout
Browse files Browse the repository at this point in the history
test-net-nodejsGH-5504 failed on smartos14-64 in CI. The internal test timeout
expired. It is currently 2 seconds. That appears to be somewhat
arbitrary. It may be possible to rewrite the test without the timer.  As
an immediate workaround, increase the timeout to 5 seconds.
  • Loading branch information
Trott committed Nov 4, 2016
1 parent 7537718 commit 24af7ca
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions test/sequential/test-net-GH-5504.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
'use strict';
var common = require('../common');

// this test only fails with CentOS 6.3 using kernel version 2.6.32
// https://github.com/nodejs/node-v0.x-archive/issues/5504
//
// This test only fails with CentOS 6.3 using kernel version 2.6.32
// On other linuxes and darwin, the `read` call gets an ECONNRESET in
// that case. On sunos, the `write` call fails with EPIPE.
//
Expand Down Expand Up @@ -54,7 +56,7 @@ function parent() {
setTimeout(function() {
throw new Error('hang');
});
}, common.platformTimeout(2000)).unref();
}, common.platformTimeout(5000)).unref();

var s = spawn(node, [__filename, 'server'], {
env: Object.assign(process.env, {
Expand Down

0 comments on commit 24af7ca

Please sign in to comment.