test: fix test-tls-securepair-client

test-tls-securepair-client has been failing for over a year but no one noticed because pummel tests are almost never run. In preparation for running pummel tests once a day in CI, fix test-tls-securepair-client. PR-URL: #25222 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · May 16, 2019 · 26b69fd · 26b69fd
1 parent 21dc7cc
commit 26b69fd
Showing 1 changed file with 14 additions and 11 deletions.
diff --git a/test/pummel/test-tls-securepair-client.js b/test/pummel/test-tls-securepair-client.js
@@ -29,6 +29,9 @@ if (!common.opensslCli)
 if (!common.hasCrypto)
   common.skip('missing crypto');
 
+if (common.isWindows)
+  common.skip('test does not work on Windows'); // ...but it should!
+
 const net = require('net');
 const assert = require('assert');
 const fixtures = require('../common/fixtures');
@@ -54,14 +57,14 @@ function test2() {
   test('keys/agent4-key.pem', 'keys/agent4-cert.pem', check);
 }
 
-function test(keyfn, certfn, check, next) {
-  const key = fixtures.readSync(keyfn).toString();
-  const cert = fixtures.readSync(certfn).toString();
+function test(keyPath, certPath, check, next) {
+  const key = fixtures.readSync(keyPath).toString();
+  const cert = fixtures.readSync(certPath).toString();
 
   const server = spawn(common.opensslCli, ['s_server',
                                            '-accept', common.PORT,
-                                           '-cert', certfn,
-                                           '-key', keyfn]);
+                                           '-cert', fixtures.path(certPath),
+                                           '-key', fixtures.path(keyPath)]);
   server.stdout.pipe(process.stdout);
   server.stderr.pipe(process.stdout);
 
@@ -72,7 +75,7 @@ function test(keyfn, certfn, check, next) {
   server.stdout.setEncoding('utf8');
   server.stdout.on('data', function(s) {
     serverStdoutBuffer += s;
-    console.error(state);
+    console.log(state);
     switch (state) {
       case 'WAIT-ACCEPT':
         if (/ACCEPT/.test(serverStdoutBuffer)) {
@@ -132,6 +135,11 @@ function test(keyfn, certfn, check, next) {
 
     s.on('connect', function() {
       console.log('client connected');
+      setTimeout(function() {
+        pair.cleartext.write('hello\r\n', function() {
+          gotWriteCallback = true;
+        });
+      }, 500);
     });
 
     pair.on('secure', function() {
@@ -141,11 +149,6 @@ function test(keyfn, certfn, check, next) {
       console.log('client pair.cleartext.getCipher(): %j',
                   pair.cleartext.getCipher());
       if (check) check(pair);
-      setTimeout(function() {
-        pair.cleartext.write('hello\r\n', function() {
-          gotWriteCallback = true;
-        });
-      }, 500);
     });
 
     pair.cleartext.on('data', function(d) {