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 bc65262 commit 3a6dcdf178ba9746d804369560128149aa453563
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 3a6dcdf

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