cluster: move handle tracking out of utils

internal/cluster/utils.js exported a handles object, which was used in a test. That test, test-cluster-disconnect-handles.js, was removed in #12495. This commit moves the handles object to the only file in the codebase that still uses it. PR-URL: #23131 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: cjihrig

lib/internal/cluster/master.js

@@ -7,7 +7,7 @@ const EventEmitter = require('events');
 constRoundRobinHandle=require('internal/cluster/round_robin_handle');
 constSharedHandle=require('internal/cluster/shared_handle');
 constWorker=require('internal/cluster/worker');
-const{ internal, sendHelper, handles}=require('internal/cluster/utils');
+const{ internal, sendHelper }=require('internal/cluster/utils');
 const{ERR_SOCKET_BAD_PORT}=require('internal/errors').codes;
 constkeys=Object.keys;
 constcluster=newEventEmitter();
@@ -19,6 +19,7 @@ const [ minPort, maxPort ] = [ 1024, 65535 ];
 
 module.exports=cluster;
 
+consthandles=newMap();
 cluster.isWorker=false;
 cluster.isMaster=true;
 cluster.Worker=Worker;

lib/internal/cluster/utils.js

@@ -3,8 +3,7 @@ const util = require('util');
 
 module.exports={
  sendHelper,
- internal,
-handles: newMap()// Used in tests.
+ internal
 };
 
 constcallbacks=newMap();