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>

1 files changed, 11 insertions, 11 deletions

diff --git a/test/parallel/test-http-many-ended-pipelines.js b/test/parallel/test-http-many-ended-pipelines.js
index c415af64f9..7966f47a9e 100644
--- a/test/parallel/test-http-many-ended-pipelines.js
+++ b/test/parallel/test-http-many-ended-pipelines.js
@@ -1,5 +1,5 @@
 'use strict';
-var common = require('../common');
+require('../common');
 
 // no warnings should happen!
 var trace = console.trace;
@@ -27,13 +27,13 @@ var server = http.createServer(function(req, res) {
   req.socket.destroy();
 });
 
-server.listen(common.PORT);
-
-var client = net.connect({ port: common.PORT, allowHalfOpen: true });
-for (var i = 0; i < numRequests; i++) {
-  client.write('GET / HTTP/1.1\r\n' +
-               'Host: some.host.name\r\n' +
-               '\r\n\r\n');
-}
-client.end();
-client.pipe(process.stdout);
+server.listen(0, function() {
+  var client = net.connect({ port: this.address().port, allowHalfOpen: true });
+  for (var i = 0; i < numRequests; i++) {
+    client.write('GET / HTTP/1.1\r\n' +
+                 'Host: some.host.name\r\n' +
+                 '\r\n\r\n');
+  }
+  client.end();
+  client.pipe(process.stdout);
+});