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

diff --git a/test/parallel/test-http-request-methods.js b/test/parallel/test-http-request-methods.js
index b29a6663ff..cbfbbbc2ed 100644
--- a/test/parallel/test-http-request-methods.js
+++ b/test/parallel/test-http-request-methods.js
@@ -1,5 +1,5 @@
 'use strict';
-var common = require('../common');
+require('../common');
 var assert = require('assert');
 var net = require('net');
 var http = require('http');
@@ -7,8 +7,6 @@ var http = require('http');
 // Test that the DELETE, PATCH and PURGE verbs get passed through correctly
 
 ['DELETE', 'PATCH', 'PURGE'].forEach(function(method, index) {
-  var port = common.PORT + index;
-
   var server_response = '';
   var received_method = null;
 
@@ -19,10 +17,10 @@ var http = require('http');
     res.write('world\n');
     res.end();
   });
-  server.listen(port);
+  server.listen(0);
 
   server.on('listening', function() {
-    var c = net.createConnection(port);
+    var c = net.createConnection(this.address().port);
 
     c.setEncoding('utf8');