Browse Source

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: https://github.com/nodejs/node/issues/11650
PR-URL: https://github.com/nodejs/node/pull/11655
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Fedor Indutny <fedor@indutny.com>
v6.x
Shigeki Ohtsu 8 years ago
committed by Myles Borins
parent
commit
7cc4645b70
No known key found for this signature in database GPG Key ID: 933B01F40B5CA946
  1. 8
      test/parallel/test-tls-alpn-server-client.js
  2. 4
      test/parallel/test-tls-npn-server-client.js

8
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');

4
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;
}

Loading…
Cancel
Save