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, 4 insertions, 4 deletions

diff --git a/test/parallel/test-http-client-race-2.js b/test/parallel/test-http-client-race-2.js
index 0a98e06c7b..20b71f909a 100644
--- a/test/parallel/test-http-client-race-2.js
+++ b/test/parallel/test-http-client-race-2.js
@@ -1,5 +1,5 @@
 'use strict';
-var common = require('../common');
+require('../common');
 var assert = require('assert');
 var http = require('http');
 var url = require('url');
@@ -27,14 +27,14 @@ var server = http.createServer(function(req, res) {
                 {'Content-Type': 'text/plain', 'Content-Length': body.length});
   res.end(body);
 });
-server.listen(common.PORT);
+server.listen(0);
 
 var body1 = '';
 var body2 = '';
 var body3 = '';
 
 server.on('listening', function() {
-  var client = http.createClient(common.PORT);
+  var client = http.createClient(this.address().port);
 
   //
   // Client #1 is assigned Parser #1
@@ -59,7 +59,7 @@ server.on('listening', function() {
         // parser that previously belonged to Client #1. But we're not finished
         // with Client #1 yet!
         //
-        var client2 = http.createClient(common.PORT);
+        var client2 = http.createClient(server.address().port);
 
         //
         // At this point, the bug would manifest itself and crash because the