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

diff --git a/test/parallel/test-http-server-stale-close.js b/test/parallel/test-http-server-stale-close.js
index 349e869c1a..7430566dfd 100644
--- a/test/parallel/test-http-server-stale-close.js
+++ b/test/parallel/test-http-server-stale-close.js
@@ -1,15 +1,15 @@
 'use strict';
-var common = require('../common');
+require('../common');
 var http = require('http');
 var util = require('util');
 var fork = require('child_process').fork;
 
-if (process.env.NODE_TEST_FORK) {
+if (process.env.NODE_TEST_FORK_PORT) {
   var req = http.request({
     headers: {'Content-Length': '42'},
     method: 'POST',
     host: '127.0.0.1',
-    port: common.PORT,
+    port: +process.env.NODE_TEST_FORK_PORT,
   }, process.exit);
   req.write('BAM');
   req.end();
@@ -23,9 +23,9 @@ else {
       res.end();
     });
   });
-  server.listen(common.PORT, function() {
+  server.listen(0, function() {
     fork(__filename, {
-      env: util._extend(process.env, {NODE_TEST_FORK: '1'})
+      env: util._extend(process.env, {NODE_TEST_FORK_PORT: this.address().port})
     });
   });
 }