test: add hasCrypto check to common flags check

Currently, if node is configured --without-ssl there will be a number of test errors related to crypto flags: Error: Test has to be started with the flag: '--tls-v1.1' This commit adds a hasCrypto check to the flags checking similar to what is done for --without-intl. PR-URL: #25147 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

Author: danbev

test/common/index.js

@@ -37,6 +37,8 @@ const {
 
 const noop = () => {};
 
+const hasCrypto = Boolean(process.versions.openssl);
+
 const isMainThread = (() => {
  try {
  return require('worker_threads').isMainThread;
@@ -74,6 +76,9 @@ if (process.argv.length === 2 &&
  const args = process.execArgv.map((arg) => arg.replace(/_/g, '-'));
  for (const flag of flags) {
  if (!args.includes(flag) &&
+ // If the binary was built without-ssl then the crypto flags are
+ // invalid (bad option). The test itself should handle this case.
+ hasCrypto &&
  // If the binary is build without `intl` the inspect option is
  // invalid. The test itself should handle this case.
  (process.config.variables.v8_enable_inspector !== 0 ||
@@ -106,7 +111,6 @@ const rootDir = isWindows ? 'c:\\' : '/';
 
 const buildType = process.config.target_defaults.default_configuration;
 
-const hasCrypto = Boolean(process.versions.openssl);
 
 // If env var is set then enable async_hook hooks for all tests.
 if (process.env.NODE_TEST_WITH_ASYNC_HOOKS) {