test: do not run release-npm test without crypto

npm requires crypto support and cannot be loaded without it. PR-URL: #30265 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: David Carlier <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Jiawen Geng <[email protected]>
nodejs · Nov 14, 2019 · 56e986a · 56e986a
1 parent b9fd18f
commit 56e986a
Showing 1 changed file with 3 additions and 2 deletions.
diff --git 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]);