Skip to content

Commit dd946c3

Browse files
danbevMylesBorins
authored andcommitted
test: add hasCrypto check to test-cli-node-options
Currently when configure --without-ssl the test will throw the following error: bad option: --use-openssl-ca This commit checks if crypto was enabled and skips the crypto related tests if that is the case. Backport-PR-URL: #12677 PR-URL: #12692 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Sam Roberts <[email protected]>/
1 parent ba830f0 commit dd946c3

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

test/parallel/test-cli-node-options.js

+5-3
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,11 @@ expect('--track-heap-objects', 'B\n');
5151
expect('--throw-deprecation', 'B\n');
5252
expect('--zero-fill-buffers', 'B\n');
5353
expect('--v8-pool-size=10', 'B\n');
54-
expect('--use-openssl-ca', 'B\n');
55-
expect('--use-bundled-ca', 'B\n');
56-
expect('--openssl-config=_ossl_cfg', 'B\n');
54+
if (common.hasCrypto) {
55+
expect('--use-openssl-ca', 'B\n');
56+
expect('--use-bundled-ca', 'B\n');
57+
expect('--openssl-config=_ossl_cfg', 'B\n');
58+
}
5759
expect('--icu-data-dir=_d', 'B\n');
5860

5961
// V8 options

0 commit comments

Comments
 (0)