From 5269e952a1541598feb17749b516b6ea50385150 Mon Sep 17 00:00:00 2001
From: Santiago Gimeno <santiago.gimeno@gmail.com>
Date: Thu, 2 Mar 2017 12:34:07 +0100
Subject: [PATCH] test: fix flaky test-https-agent-create-connection

Use a different server instance for every test to avoid that they reuse
the same port.

PR-URL: https://github.com/nodejs/node/pull/11649
Fixes: https://github.com/nodejs/node/issues/11644
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
---
 .../test-https-agent-create-connection.js     | 158 ++++++++++--------
 1 file changed, 90 insertions(+), 68 deletions(-)

diff --git a/test/parallel/test-https-agent-create-connection.js b/test/parallel/test-https-agent-create-connection.js
index f593700081a88f..6bd86fe166d3f2 100644
--- a/test/parallel/test-https-agent-create-connection.js
+++ b/test/parallel/test-https-agent-create-connection.js
@@ -18,10 +18,6 @@ const options = {
   cert: fs.readFileSync(common.fixturesDir + '/keys/agent1-cert.pem'),
 };
 
-const server = https.createServer(options, (req, res) => {
-  res.end('hello world\n');
-});
-
 const expectedHeader = /^HTTP\/1.1 200 OK/;
 const expectedBody = /hello world\n/;
 const expectCertError = /^Error: unable to verify the first certificate$/;
@@ -42,83 +38,109 @@ const checkRequest = (socket, server) => {
   }));
 };
 
+function createServer() {
+  return https.createServer(options, (req, res) => {
+    res.end('hello world\n');
+  });
+}
+
 // use option connect
-server.listen(0, common.mustCall(() => {
-  const port = server.address().port;
-  const host = 'localhost';
-  const options = {
-    port: port,
-    host: host,
-    rejectUnauthorized: false,
-    _agentKey: agent.getName({
+{
+  const server = createServer();
+  server.listen(0, common.mustCall(() => {
+    const port = server.address().port;
+    const host = 'localhost';
+    const options = {
       port: port,
       host: host,
-    }),
-  };
+      rejectUnauthorized: false,
+      _agentKey: agent.getName({
+        port: port,
+        host: host,
+      }),
+    };
 
-  const socket = agent.createConnection(options);
-  checkRequest(socket, server);
-}));
+    const socket = agent.createConnection(options);
+    checkRequest(socket, server);
+  }));
+}
 
 // use port and option connect
-server.listen(0, common.mustCall(() => {
-  const port = server.address().port;
-  const host = 'localhost';
-  const options = {
-    rejectUnauthorized: false,
-    _agentKey: agent.getName({
-      port: port,
-      host: host,
-    }),
-  };
-  const socket = agent.createConnection(port, options);
-  checkRequest(socket, server);
-}));
+{
+  const server = createServer();
+  server.listen(0, common.mustCall(() => {
+    const port = server.address().port;
+    const host = 'localhost';
+    const options = {
+      rejectUnauthorized: false,
+      _agentKey: agent.getName({
+        port: port,
+        host: host,
+      }),
+    };
+    const socket = agent.createConnection(port, options);
+    checkRequest(socket, server);
+  }));
+}
 
 // use port and host and option connect
-server.listen(0, common.mustCall(() => {
-  const port = server.address().port;
-  const host = 'localhost';
-  const options = {
-    rejectUnauthorized: false,
-    _agentKey: agent.getName({
-      port: port,
-      host: host,
-    }),
-  };
-  const socket = agent.createConnection(port, host, options);
-  checkRequest(socket, server);
-}));
+{
+  const server = createServer();
+  server.listen(0, common.mustCall(() => {
+    const port = server.address().port;
+    const host = 'localhost';
+    const options = {
+      rejectUnauthorized: false,
+      _agentKey: agent.getName({
+        port: port,
+        host: host,
+      }),
+    };
+    const socket = agent.createConnection(port, host, options);
+    checkRequest(socket, server);
+  }));
+}
 
 // use port and host and option does not have agentKey
-server.listen(0, common.mustCall(() => {
-  const port = server.address().port;
-  const host = 'localhost';
-  const options = {
-    rejectUnauthorized: false,
-  };
-  const socket = agent.createConnection(port, host, options);
-  checkRequest(socket, server);
-}));
+{
+  const server = createServer();
+  server.listen(0, common.mustCall(() => {
+    const port = server.address().port;
+    const host = 'localhost';
+    const options = {
+      rejectUnauthorized: false,
+    };
+    const socket = agent.createConnection(port, host, options);
+    checkRequest(socket, server);
+  }));
+}
 
 // options is null
-server.listen(0, common.mustCall(() => {
-  const port = server.address().port;
-  const host = 'localhost';
-  const options = null;
-  const socket = agent.createConnection(port, host, options);
-  socket.on('error', common.mustCall((e) => {
-    assert(expectCertError.test(e.toString()));
+{
+  const server = createServer();
+  server.listen(0, common.mustCall(() => {
+    const port = server.address().port;
+    const host = 'localhost';
+    const options = null;
+    const socket = agent.createConnection(port, host, options);
+    socket.on('error', common.mustCall((e) => {
+      assert(expectCertError.test(e.toString()));
+      server.close();
+    }));
   }));
-}));
+}
 
 // options is undefined
-server.listen(0, common.mustCall(() => {
-  const port = server.address().port;
-  const host = 'localhost';
-  const options = undefined;
-  const socket = agent.createConnection(port, host, options);
-  socket.on('error', common.mustCall((e) => {
-    assert(expectCertError.test(e.toString()));
+{
+  const server = createServer();
+  server.listen(0, common.mustCall(() => {
+    const port = server.address().port;
+    const host = 'localhost';
+    const options = undefined;
+    const socket = agent.createConnection(port, host, options);
+    socket.on('error', common.mustCall((e) => {
+      assert(expectCertError.test(e.toString()));
+      server.close();
+    }));
   }));
-}));
+}