1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: skip tests with common.skip

Browse Source 
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>
This commit is contained in:
Sakthipriyan Vairamani (thefourtheye) 2017-02-28 00:24:03 +05:30 committed by James M Snell
parent 42766a8183
commit 5d4273d122
2 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -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
View File

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