diff options
author | Brian White <mscdex@mscdex.net> | 2016-05-29 03:06:56 -0400 |
---|---|---|
committer | Brian White <mscdex@mscdex.net> | 2016-06-10 22:30:55 -0400 |
commit | 2bc7841d0fcdd066fe477873229125b6f003b693 (patch) | |
tree | 2816555ef6ad2fc828a75dc3c564f8faa2dee6c7 /test/parallel/test-net-settimeout.js | |
parent | 624734e640717a826ab1a18845c083a638dc5ce6 (diff) | |
download | android-node-v8-2bc7841d0fcdd066fe477873229125b6f003b693.tar.gz android-node-v8-2bc7841d0fcdd066fe477873229125b6f003b693.tar.bz2 android-node-v8-2bc7841d0fcdd066fe477873229125b6f003b693.zip |
test: use random ports where possible
This helps to prevent issues where a failed test can keep a bound
socket open long enough to cause other tests to fail with EADDRINUSE
because the same port number is used.
PR-URL: https://github.com/nodejs/node/pull/7045
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Diffstat (limited to 'test/parallel/test-net-settimeout.js')
-rw-r--r-- | test/parallel/test-net-settimeout.js | 32 |
1 files changed, 16 insertions, 16 deletions
diff --git a/test/parallel/test-net-settimeout.js b/test/parallel/test-net-settimeout.js index db0329deea..688937fe46 100644 --- a/test/parallel/test-net-settimeout.js +++ b/test/parallel/test-net-settimeout.js @@ -12,20 +12,20 @@ const server = net.createServer(common.mustCall((c) => { c.write('hello'); })); -server.listen(common.PORT); - -const socket = net.createConnection(common.PORT, 'localhost'); - -const s = socket.setTimeout(T, () => { - common.fail('Socket timeout event is not expected to fire'); +server.listen(0, function() { + const socket = net.createConnection(this.address().port, 'localhost'); + + const s = socket.setTimeout(T, () => { + common.fail('Socket timeout event is not expected to fire'); + }); + assert.ok(s instanceof net.Socket); + + socket.on('data', common.mustCall(() => { + setTimeout(function() { + socket.destroy(); + server.close(); + }, T * 2); + })); + + socket.setTimeout(0); }); -assert.ok(s instanceof net.Socket); - -socket.on('data', common.mustCall(() => { - setTimeout(function() { - socket.destroy(); - server.close(); - }, T * 2); -})); - -socket.setTimeout(0); |