expose keepAlive symbols from net for testing

Author: tadjik1

lib/internal/net.js

@@ -100,6 +100,9 @@ function isLoopback(host) {
 
 module.exports = {
   kReinitializeHandle: Symbol('kReinitializeHandle'),
+  kSetNoDelay: Symbol('kSetNoDelay'),
+  kSetKeepAlive: Symbol('kSetKeepAlive'),
+  kSetKeepAliveInitialDelay: Symbol('kSetKeepAliveInitialDelay'),
   isIP,
   isIPv4,
   isIPv6,

lib/net.js

@@ -48,6 +48,9 @@ let debug = require('internal/util/debuglog').debuglog('net', (fn) => {
 });
 const {
   kReinitializeHandle,
+  kSetNoDelay,
+  kSetKeepAlive,
+  kSetKeepAliveInitialDelay,
   isIP,
   isIPv4,
   isIPv6,
@@ -356,9 +359,6 @@ function closeSocketHandle(self, isException, isCleanupPending = false) {
 
 const kBytesRead = Symbol('kBytesRead');
 const kBytesWritten = Symbol('kBytesWritten');
-const kSetNoDelay = Symbol('kSetNoDelay');
-const kSetKeepAlive = Symbol('kSetKeepAlive');
-const kSetKeepAliveInitialDelay = Symbol('kSetKeepAliveInitialDelay');
 const kSetTOS = Symbol('kSetTOS');
 
 function Socket(options) {

test/parallel/test-tls-connect-keepalive-nodelay.js

@@ -1,3 +1,4 @@
+// Flags: --expose-internals
 'use strict';
 
 const common = require('../common');
@@ -9,21 +10,17 @@ if (!common.hasCrypto)
   common.skip('missing crypto');
 
 const assert = require('assert');
-const net = require('net');
 const tls = require('tls');
 const fixtures = require('../common/fixtures');
+const {
+  kSetNoDelay,
+  kSetKeepAlive,
+  kSetKeepAliveInitialDelay,
+} = require('internal/net');
 
 const key = fixtures.readKey('agent1-key.pem');
 const cert = fixtures.readKey('agent1-cert.pem');
 
-// Retrieve internal symbols used by net.Socket to store socket options.
-const symbols = Object.getOwnPropertySymbols(new net.Socket());
-const kSetNoDelay = symbols.find((s) => s.toString() === 'Symbol(kSetNoDelay)');
-const kSetKeepAlive =
-  symbols.find((s) => s.toString() === 'Symbol(kSetKeepAlive)');
-const kSetKeepAliveInitialDelay =
-  symbols.find((s) => s.toString() === 'Symbol(kSetKeepAliveInitialDelay)');
-
 // Test: keepAlive, keepAliveInitialDelay, and noDelay
 {
   const server = tls.createServer({ key, cert }, (socket) => {