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

For better performance of the test, the parent kills child processes so as not to wait them to be ended. (cherry picked from commit 833b236) Reviewed-By: James M Snell <[email protected]> PR-URL: #25368
nodejs · Jun 4, 2015 · cb63d93 · cb63d93
1 parent 83b7c07
commit cb63d93
Showing 1 changed file with 8 additions and 2 deletions.
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() {