Browse Source

test: skip tests with common.skip

The `common.skip` function adds proper message in TAP format to skipped
tests. It is better not to have the message rewritten in the tests.

PR-URL: https://github.com/nodejs/node/pull/11585
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
v6.x
Sakthipriyan Vairamani (thefourtheye) 8 years ago
committed by Myles Borins
parent
commit
0322d3b8d5
No known key found for this signature in database GPG Key ID: 933B01F40B5CA946
  1. 2
      test/parallel/test-crypto-authenticated.js
  2. 6
      test/parallel/test-tls-empty-sni-context.js

2
test/parallel/test-crypto-authenticated.js

@ -318,7 +318,7 @@ for (const i in TEST_CASES) {
}
if (common.hasFipsCrypto && test.iv.length < 24) {
console.log('1..0 # Skipped: IV len < 12 bytes unsupported in FIPS mode');
common.skip('IV len < 12 bytes unsupported in FIPS mode');
continue;
}

6
test/parallel/test-tls-empty-sni-context.js

@ -3,16 +3,14 @@
const common = require('../common');
if (!process.features.tls_sni) {
console.log('1..0 # Skipped: node compiled without OpenSSL or ' +
'with old OpenSSL version.');
common.skip('node compiled without OpenSSL or with old OpenSSL version.');
return;
}
const assert = require('assert');
if (!common.hasCrypto) {
console.log('1..0 # Skipped: missing crypto');
return;
return common.skip('missing crypto');
}
const tls = require('tls');

Loading…
Cancel
Save