Skip to content

test: allow disabling crypto tests

Rodrigo Muino Tomonari requested to merge github/fork/codebytere/allow-disabling-crypto-tests into master

This PR adds a new environment variable to the test suit: NODE_SKIP_CRYPTO. Right now whether or not crypto tests can be run is solely determined by whether or not openssl is defined on the process object, which is true for Electron.js since many consumers use it to determine the presence of crypto even though we use BoringSSL and thus experience incompatibilities.

This new option would allow us to more effectively run our own smoke tests of Node.js' test suite without needing to manually disable every crypto test individually.

cc @tniessen

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

Merge request reports