Skip to content

Commit 95b2a39

Browse files
Trottaddaleax
authored andcommitted
test: remove common.localhostIPv6
common.localhostIPv6 is almost entirely unused and is unnecessary. Remove it. PR-URL: #34373 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 76ba129 commit 95b2a39

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

‎test/common/index.js‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -787,8 +787,6 @@ const common ={
787787
returnlocalhostIPv4;
788788
},
789789

790-
getlocalhostIPv6(){return'::1';},
791-
792790
// opensslCli defined lazily to reduce overhead of spawnSync
793791
getopensslCli(){
794792
if(opensslCli!==null)returnopensslCli;

‎test/sequential/test-net-connect-local-error.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@ const optionsIPv4 ={
1414
};
1515

1616
constoptionsIPv6={
17-
host: common.localhostIPv6,
17+
host: '::1',
1818
port: common.PORT+2,
1919
localPort: common.PORT+3,
20-
localAddress: common.localhostIPv6
20+
localAddress: '::1',
2121
};
2222

2323
functiononError(err,options){

0 commit comments

Comments
(0)