Permalink
Browse files

test: fix http-client-timeout-option-listeners

test-http-client-timeout-option-listeners is flaky due to depending on
completing operations before a 100ms socket timeout. The socket timeout
is an integral part of the test but can be very large. Set to the
maximum allowable value.

PR-URL: #10224
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
  • Loading branch information...
Trott committed Dec 11, 2016
1 parent e27c2a7 commit 4a5c719be50c7998684768b54d06860cb2c1d6fb
Showing with 4 additions and 1 deletion.
  1. +4 −1 test/parallel/test-http-client-timeout-option-listeners.js
@@ -9,13 +9,16 @@ const server = http.createServer((req, res) => {
res.end('');
});
// Maximum allowed value for timeouts
const timeout = 2 ** 31 - 1;
const options = {
agent,
method: 'GET',
port: undefined,
host: common.localhostIPv4,
path: '/',
timeout: common.platformTimeout(100)
timeout: timeout
};
server.listen(0, options.host, common.mustCall(() => {

0 comments on commit 4a5c719

Please sign in to comment.