From f70fd00fb3cc1436c73a3330366840577c18a36e Mon Sep 17 00:00:00 2001 From: Adam Majer Date: Fri, 11 Jun 2021 20:21:22 +0200 Subject: [PATCH] test: use localhost test instead of connecting to remote Fixes: https://github.com/nodejs/node/issues/39008 PR-URL: https://github.com/nodejs/node/pull/39011 Reviewed-By: Luigi Pinca Reviewed-By: Darshan Sen Reviewed-By: Zijian Liu --- .../parallel/test-https-agent-unref-socket.js | 24 +++++++++++++++---- 1 file changed, 20 insertions(+), 4 deletions(-) diff --git a/test/parallel/test-https-agent-unref-socket.js b/test/parallel/test-https-agent-unref-socket.js index b2863a74d817d4..49169b523e1042 100644 --- a/test/parallel/test-https-agent-unref-socket.js +++ b/test/parallel/test-https-agent-unref-socket.js @@ -6,8 +6,24 @@ if (!common.hasCrypto) const https = require('https'); -const request = https.get('https://example.com'); +if (process.argv[2] === 'localhost') { + const request = https.get('https://localhost:' + process.argv[3]); -request.on('socket', (socket) => { - socket.unref(); -}); + request.on('socket', (socket) => { + socket.unref(); + }); +} else { + const assert = require('assert'); + const net = require('net'); + const server = net.createServer(); + server.listen(0); + server.on('listening', () => { + const port = server.address().port; + const { fork } = require('child_process'); + const child = fork(__filename, ['localhost', port], {}); + child.on('close', (exit_code) => { + server.close(); + assert.strictEqual(exit_code, 0); + }); + }); +}