test: fix nits in test/fixtures/tls-connect.js

PR-URL: #28880
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>

Author: lpinca

test/fixtures/tls-connect.js

@@ -18,7 +18,7 @@ exports.debug = util.debuglog('test');
 exports.tls = tls;
 
 // Pre-load keys from common fixtures for ease of use by tests.
-const keys = exports.keys = {
+exports.keys = {
   agent1: load('agent1', 'ca1'),
   agent2: load('agent2', 'agent2'),
   agent3: load('agent3', 'ca2'),
@@ -31,9 +31,9 @@ const keys = exports.keys = {
   ec: load('ec', 'ec'),
 };
 
-// root is the self-signed root of the trust chain, not an intermediate ca.
+// `root` is the self-signed root of the trust chain, not an intermediate ca.
 function load(cert, root) {
-  root = root || cert; // Assume self-signed if no issuer
+  root = root || cert; // Assume self-signed if no issuer.
   const id = {
     key: fixtures.readKey(cert + '-key.pem', 'binary'),
     cert: fixtures.readKey(cert + '-cert.pem', 'binary'),
@@ -53,7 +53,7 @@ exports.connect = function connect(options, callback) {
     tls.createServer(options.server, function(conn) {
       server.conn = conn;
       conn.pipe(conn);
-      maybeCallback()
+      maybeCallback();
     }).listen(0, function() {
       server.server = this;
 
@@ -92,7 +92,7 @@ exports.connect = function connect(options, callback) {
   function maybeCallback() {
     if (!callback)
       return;
-    if (server.conn && (client.conn || client.err)) {
+    if (server.conn && client.conn) {
       const err = pair.client.err || pair.server.err;
       callback(err, pair, cleanup);
       callback = null;
@@ -105,4 +105,4 @@ exports.connect = function connect(options, callback) {
     if (client.conn)
       client.conn.end();
   }
-}
+};