test: use localhost test instead of connecting to remote

AdamMajer · targos · commit df17c6281876 · 2021-07-11T09:43:52.000+02:00
Fixes: #39008 PR-URL: #39011 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
diff --git 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);
+ });
+ });
+}

-Original file line number
+Diff line change
 consthttps=require('https');
 -constrequest=https.get('https://example.com');
 +if(process.argv[2]==='localhost'){
 +constrequest=https.get('https://localhost:'+process.argv[3]);
 -request.on('socket',(socket)=>{
 -socket.unref();
 -});
 +request.on('socket',(socket)=>{
 +socket.unref();
 +});
 +}else{
 +constassert=require('assert');
 +constnet=require('net');
 +constserver=net.createServer();
 +server.listen(0);
 +server.on('listening',()=>{
 +constport=server.address().port;
 +const{ fork }=require('child_process');
 +constchild=fork(__filename,['localhost',port],{});
 +child.on('close',(exit_code)=>{
 +server.close();
 +assert.strictEqual(exit_code,0);
 +});
 +});
 +}