test: fix tests when npn feature is disabled.

ALPN test needs NPN feature to run. It also change the messages when ALPN and NPN tests are skipped. Fixes: nodejs/node#11650 PR-URL: nodejs/node#11655 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Fedor Indutny <[email protected]>
michielbaird · Jul 19, 2017 · c752f20 · c752f20
1 parent 3e8231c
commit c752f20
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/test/parallel/test-tls-alpn-server-client.js b/test/parallel/test-tls-alpn-server-client.js
@@ -6,10 +6,10 @@ if (!common.hasCrypto) {
   return;
 }
 
-if (!process.features.tls_alpn) {
-  console.error('Skipping because node compiled without OpenSSL or ' +
-                'with old OpenSSL version.');
-  process.exit(0);
+if (!process.features.tls_alpn || !process.features.tls_npn) {
+  common.skip('Skipping because node compiled without NPN or ALPN' +
+              ' feature of OpenSSL.');
+  return;
 }
 
 const assert = require('assert');

diff --git a/test/parallel/test-tls-npn-server-client.js b/test/parallel/test-tls-npn-server-client.js
@@ -1,7 +1,7 @@
 'use strict';
 if (!process.features.tls_npn) {
-  common.skip('node compiled without OpenSSL or ' +
-              'with old OpenSSL version.');
+  common.skip('Skipping because node compiled without NPN feature of' +
+              ' OpenSSL.');
   return;
 }