test: kill child in tls-server-verify for speed up

Shigeki Ohtsu · Shigeki Ohtsu · commit 833b23636045 · 2015-06-02T00:58:42.000+09:00
For better performance of the test, the parent kills child processes
so as not to wait them to be ended.
diff --git a/test/simple/test-tls-server-verify.js b/test/simple/test-tls-server-verify.js
@@ -211,15 +211,15 @@ function runClient(prefix, port, options, cb) {
     if (!goodbye && /_unauthed/g.test(out)) {
       console.error(prefix + '  * unauthed');
       goodbye = true;
-      client.stdin.end('goodbye\n');
+      client.kill();
       authed = false;
       rejected = false;
     }
 
     if (!goodbye && /_authed/g.test(out)) {
       console.error(prefix + '  * authed');
       goodbye = true;
-      client.stdin.end('goodbye\n');
+      client.kill();
       authed = true;
       rejected = false;
     }
@@ -281,6 +281,12 @@ function runTest(port, testIndex) {
 
   var renegotiated = false;
   var server = tls.Server(serverOptions, function handleConnection(c) {
+    c.on('error', function(e) {
+      // child.kill() leads ECONNRESET errro in the TLS connection of
+      // openssl s_client via spawn(). A Test result is already
+      // checked by the data of client.stdout before child.kill() so
+      // these tls errors can be ignored.
+    });
     if (tcase.renegotiate && !renegotiated) {
       renegotiated = true;
       setTimeout(function() {
