test: rename and document tls test

Add a short description and one question to a TLS test.
Also, rename it since the previous name might unintentionally
evoke connotations to an internet meme that would generally
not be considered appropriate in our context.

PR-URL: #20820
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ujjwal Sharma <[email protected]>

Author: addaleax

test/parallel/test-tls-two-cas-one-string.js renamed to test/parallel/test-tls-multiple-cas-as-string.js

@@ -4,6 +4,9 @@ const common = require('../common');
 if (!common.hasCrypto)
   common.skip('missing crypto');
 
+// Verify that multiple CA certificates can be provided, and that for
+// convenience that can also be in newline-separated strings.
+
 const tls = require('tls');
 const fixtures = require('../common/fixtures');
 
@@ -30,6 +33,8 @@ function test(ca, next) {
   }
 }
 
+// `ca1` is not actually necessary for the certificate validation -- maybe
+// the fixtures should be written in a way that requires it?
 const array = [ca1, ca2];
 const string = `${ca1}\n${ca2}`;
 test(array, common.mustCall(() => test(string)));