fixup! build: add option for running all component tests

Rename yarn test --all to yarn test all

Author: devversion

scripts/run-component-tests.js

@@ -4,6 +4,7 @@
  * Script that simplifies the workflow of running unit tests for a component
  * using Bazel. Here are a few examples:
  *
+ *   node ./scripts/run-component-tests all               | Runs tests for all components
  *   node ./scripts/run-component-tests button            | Runs Material button tests
  *   node ./scripts/run-component-tests overlay           | Runs CDK overlay tests
  *   node ./scripts/run-component-tests src/cdk/a11y      | Runs CDK a11y tests
@@ -14,7 +15,6 @@
  *   --local    | If specified, no browser will be launched.
  *   --firefox  | Instead of Chrome being used for tests, Firefox will be used.
  *   --no-watch | Watch mode is enabled by default. This flag opts-out to standard Bazel.
- *   --all      | Runs tests for all components in the project.
  */
 
 const minimist = require('minimist');
@@ -39,11 +39,14 @@ shelljs.set('-e');
 shelljs.cd(projectDir);
 
 // Extracts the supported command line options.
-const {_: components, local, firefox, watch, all} = minimist(args, {
-  boolean: ['local', 'firefox', 'watch', 'all'],
+const {_: components, local, firefox, watch} = minimist(args, {
+  boolean: ['local', 'firefox', 'watch'],
   default: {watch: true},
 });
 
+// Whether tests for all components should be run.
+const all = components.length === 1 && components[0] === 'all';
+
 // We can only run a single target with "--local". Running multiple targets within the
 // same Karma server is not possible since each test target runs isolated from the others.
 if (local && (components.length > 1 || all)) {
@@ -57,14 +60,9 @@ const bazelBinary = `yarn -s ${watch ? 'ibazel' : 'bazel'}`;
 const testTargetName =
     `unit_tests_${local ? 'local' : firefox ? 'firefox-local' : 'chromium-local'}`;
 
-// If the `--all` flag has been specified, we run tests for all components in the
-// repository. The `--firefox` flag can be still specified.
+// If `all` has been specified as component, we run tests for all components
+// in the repository. The `--firefox` flag can be still specified.
 if (all) {
-  if (components.length) {
-    console.error(
-        chalk.red('Script cannot be run with `--all` if individual components are specified.'));
-    process.exit(1);
-  }
   shelljs.exec(
       `${bazelBinary} test //src/... --test_tag_filters=-e2e,-browser:${testTargetName} ` +
       `--build_tag_filters=-browser:${testTargetName} --build_tests_only`);
@@ -74,10 +72,10 @@ if (all) {
 // Exit if no component has been specified.
 if (!components.length) {
   console.error(chalk.red(
-      'No component specified. Please either specify individual components, or pass the "--all" ' +
-      'flag in order to run tests for all components.'));
+      'No component specified. Please either specify individual components, or pass "all" ' +
+      'in order to run tests for all components.'));
   console.info(chalk.yellow('Below are a few examples of how the script can be run:'));
-  console.info(chalk.yellow(` - yarn test --all`));
+  console.info(chalk.yellow(` - yarn test all`));
   console.info(chalk.yellow(` - yarn test cdk/overlay material/stepper`));
   console.info(chalk.yellow(` - yarn test button toolbar`));
   process.exit(1);