test: refactor test-tls-two-cas-one-string

* order require() statements per test writing guide
* add keydir variable to make readFileSync() calls more readable
* make `next` argument to test() optional
* use common.mustCall() to guarantee second test runs

PR-URL: #13896
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Daniel Bevenius <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>

Author: Trott

test/parallel/test-tls-two-cas-one-string.js

@@ -1,22 +1,20 @@
 'use strict';
-
 const common = require('../common');
+
 if (!common.hasCrypto) {
   common.skip('missing crypto');
   return;
 }
 
-const tls = require('tls');
 const fs = require('fs');
+const tls = require('tls');
+
+const keydir = `${common.fixturesDir}/keys`;
 
-const ca1 =
-    fs.readFileSync(`${common.fixturesDir}/keys/ca1-cert.pem`, 'utf8');
-const ca2 =
-    fs.readFileSync(`${common.fixturesDir}/keys/ca2-cert.pem`, 'utf8');
-const cert =
-    fs.readFileSync(`${common.fixturesDir}/keys/agent3-cert.pem`, 'utf8');
-const key =
-    fs.readFileSync(`${common.fixturesDir}/keys/agent3-key.pem`, 'utf8');
+const ca1 = fs.readFileSync(`${keydir}/ca1-cert.pem`, 'utf8');
+const ca2 = fs.readFileSync(`${keydir}/ca2-cert.pem`, 'utf8');
+const cert = fs.readFileSync(`${keydir}/agent3-cert.pem`, 'utf8');
+const key = fs.readFileSync(`${keydir}/agent3-key.pem`, 'utf8');
 
 function test(ca, next) {
   const server = tls.createServer({ ca, cert, key }, function(conn) {
@@ -31,9 +29,11 @@ function test(ca, next) {
     tls.connect({ servername: 'agent3', host, port: this.address().port, ca });
   });
 
-  server.once('close', next);
+  if (next) {
+    server.once('close', next);
+  }
 }
 
 const array = [ca1, ca2];
 const string = `${ca1}\n${ca2}`;
-test(array, () => test(string, common.noop));
+test(array, common.mustCall(() => test(string)));