Skip to content
Permalink
Browse files

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.

PR-URL: nodejs#12692
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>/
  • Loading branch information...
danbev authored and sam-github committed Apr 27, 2017
1 parent d2871fc commit 083b8c3c28d36c6365f3c992325cc056f48e048e
Showing with 5 additions and 3 deletions.
  1. +5 −3 test/parallel/test-cli-node-options.js
@@ -51,9 +51,11 @@ expect('--track-heap-objects', 'B\n');
expect('--throw-deprecation', 'B\n');
expect('--zero-fill-buffers', 'B\n');
expect('--v8-pool-size=10', 'B\n');
expect('--use-openssl-ca', 'B\n');
expect('--use-bundled-ca', 'B\n');
expect('--openssl-config=_ossl_cfg', 'B\n');
if (common.hasCrypto) {
expect('--use-openssl-ca', 'B\n');
expect('--use-bundled-ca', 'B\n');
expect('--openssl-config=_ossl_cfg', 'B\n');
}
expect('--icu-data-dir=_d', 'B\n');

// V8 options

0 comments on commit 083b8c3

Please sign in to comment.
You can’t perform that action at this time.