test: remove unnecessary use of common.PORT in addons test

Using port 0 to request an open port from the operating system is
sufficient in openssl-client-cert-engine/test.js.

PR-URL: https://github.com/nodejs/node/pull/17563
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>

1 files changed, 2 insertions, 4 deletions

diff --git a/test/addons/openssl-client-cert-engine/test.js b/test/addons/openssl-client-cert-engine/test.js
index d07b9c6a1c..1c0e4564a5 100644
--- a/test/addons/openssl-client-cert-engine/test.js
+++ b/test/addons/openssl-client-cert-engine/test.js
@@ -21,8 +21,6 @@ const agentKey = fs.readFileSync(fixture.path('/keys/agent1-key.pem'));
 const agentCert = fs.readFileSync(fixture.path('/keys/agent1-cert.pem'));
 const agentCa = fs.readFileSync(fixture.path('/keys/ca1-cert.pem'));
 
-const port = common.PORT;
-
 const serverOptions = {
   key: agentKey,
   cert: agentCert,
@@ -34,11 +32,11 @@ const serverOptions = {
 const server = https.createServer(serverOptions, (req, res) => {
   res.writeHead(200);
   res.end('hello world');
-}).listen(port, common.localhostIPv4, () => {
+}).listen(0, common.localhostIPv4, () => {
   const clientOptions = {
     method: 'GET',
     host: common.localhostIPv4,
-    port: port,
+    port: server.address().port,
     path: '/test',
     clientCertEngine: engine,  // engine will provide key+cert
     rejectUnauthorized: false, // prevent failing on self-signed certificates