diff options
author | Michaël Zasso <targos@protonmail.com> | 2019-11-05 14:54:08 +0300 |
---|---|---|
committer | Michaël Zasso <targos@protonmail.com> | 2019-11-14 15:17:19 +0300 |
commit | 56e986aa2386b796acd77f843062d46c53cebbae (patch) | |
tree | 69ba4499de00a11346b64a35d26fac59608d0985 | |
parent | b9fd18f9fbed13cd2538f46e0072c923cbfd95cd (diff) |
test: do not run release-npm test without crypto
npm requires crypto support and cannot be loaded without it.
PR-URL: https://github.com/nodejs/node/pull/30265
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
-rw-r--r-- | test/parallel/test-release-npm.js | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/test/parallel/test-release-npm.js b/test/parallel/test-release-npm.js index 23d9cb39cea..412cb0cc64a 100644 --- a/test/parallel/test-release-npm.js +++ b/test/parallel/test-release-npm.js @@ -7,12 +7,13 @@ const path = require('path'); const releaseReg = /^v\d+\.\d+\.\d+$/; -if (!releaseReg.test(process.version)) { +// Npm requires crypto support. +if (!releaseReg.test(process.version) || !common.hasCrypto) { common.skip('This test is only for release builds'); } { - // Verify that npm does not print out a warning when executed + // Verify that npm does not print out a warning when executed. const npmCli = path.join(__dirname, '../../deps/npm/bin/npm-cli.js'); const npmExec = child_process.spawnSync(process.execPath, [npmCli]); |