Skip to content

Commit 2601c06

Browse files
thefourtheyeaddaleax
authored andcommitted
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: #11585 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Yuta Hiroto <[email protected]>
1 parent b116830 commit 2601c06

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

test/parallel/test-crypto-authenticated.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -318,7 +318,7 @@ for (const i in TEST_CASES) {
318318
}
319319

320320
if (common.hasFipsCrypto && test.iv.length < 24) {
321-
console.log('1..0 # Skipped: IV len < 12 bytes unsupported in FIPS mode');
321+
common.skip('IV len < 12 bytes unsupported in FIPS mode');
322322
continue;
323323
}
324324

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

+2-4
Original file line numberDiff line numberDiff line change
@@ -3,16 +3,14 @@
33
const common = require('../common');
44

55
if (!process.features.tls_sni) {
6-
console.log('1..0 # Skipped: node compiled without OpenSSL or ' +
7-
'with old OpenSSL version.');
6+
common.skip('node compiled without OpenSSL or with old OpenSSL version.');
87
return;
98
}
109

1110
const assert = require('assert');
1211

1312
if (!common.hasCrypto) {
14-
console.log('1..0 # Skipped: missing crypto');
15-
return;
13+
return common.skip('missing crypto');
1614
}
1715

1816
const tls = require('tls');

0 commit comments

Comments
 (0)