test: remove common.PORT from http tests

Trott · Fishrock123 · commit b383fdde791d · 2016-07-05T22:36:54.000+02:00
PR-URL: #7467 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Brian White <mscdex@mscdex.net> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/test/parallel/test-http-client-keep-alive-release-before-finish.js b/test/parallel/test-http-client-keep-alive-release-before-finish.js
@@ -4,16 +4,18 @@ const http = require('http');
 
 const server = http.createServer((req, res) => {
   res.end();
-}).listen(common.PORT, common.mustCall(() => {
+}).listen(0, common.mustCall(() => {
   const agent = new http.Agent({
     maxSockets: 1,
     keepAlive: true
   });
 
+  const port = server.address().port;
+
   const post = http.request({
-    agent: agent,
+    agent,
     method: 'POST',
-    port: common.PORT,
+    port,
   }, common.mustCall((res) => {
     res.resume();
   }));
@@ -28,9 +30,9 @@ const server = http.createServer((req, res) => {
   }, 100);
 
   http.request({
-    agent: agent,
+    agent,
     method: 'GET',
-    port: common.PORT,
+    port,
   }, common.mustCall((res) => {
     server.close();
     res.connection.end();
diff --git a/test/parallel/test-http-no-read-no-dump.js b/test/parallel/test-http-no-read-no-dump.js
@@ -15,16 +15,18 @@ const server = http.createServer((req, res) => {
     res.end();
     onPause();
   });
-}).listen(common.PORT, common.mustCall(() => {
+}).listen(0, common.mustCall(() => {
   const agent = new http.Agent({
     maxSockets: 1,
     keepAlive: true
   });
 
+  const port = server.address().port;
+
   const post = http.request({
-    agent: agent,
+    agent,
     method: 'POST',
-    port: common.PORT,
+    port,
   }, common.mustCall((res) => {
     res.resume();
 
@@ -41,9 +43,9 @@ const server = http.createServer((req, res) => {
   post.write('initial');
 
   http.request({
-    agent: agent,
+    agent,
     method: 'GET',
-    port: common.PORT,
+    port,
   }, common.mustCall((res) => {
     server.close();
     res.connection.end();
