Revert '--coverage' in favor of #54499

Herebyfile.mjs

@@ -577,7 +577,6 @@ export const runTests = task({
 // task("runtests").flags = {
 //     "-t --tests=<regex>": "Pattern for tests to run.",
 //     "   --failed": "Runs tests listed in '.failed-tests'.",
-//     "   --coverage": "Generate test coverage using c8",
 //     "-r --reporter=<reporter>": "The mocha reporter to use.",
 //     "-i --break": "Runs tests in inspector mode (NodeJS 8 and later)",
 //     "   --keepFailed": "Keep tests in .failed-tests even if they pass",

scripts/build/options.mjs

@@ -4,7 +4,7 @@ import os from "os";
 const ci = ["1", "true"].includes(process.env.CI ?? "");
 
 const parsed = minimist(process.argv.slice(2), {
-    boolean: ["dirty", "light", "colors", "lkg", "soft", "fix", "failed", "keepFailed", "force", "built", "ci", "bundle", "typecheck", "lint", "coverage"],
+    boolean: ["dirty", "light", "colors", "lkg", "soft", "fix", "failed", "keepFailed", "force", "built", "ci", "bundle", "typecheck", "lint"],
     string: ["browser", "tests", "break", "host", "reporter", "stackTraceLimit", "timeout", "shards", "shardId"],
     alias: {
         /* eslint-disable quote-props */
@@ -42,7 +42,6 @@ const parsed = minimist(process.argv.slice(2), {
         bundle: true,
         typecheck: true,
         lint: true,
-        coverage: false,
     }
 });
 
@@ -89,6 +88,5 @@ export default options;
  * @property {boolean} bundle
  * @property {boolean} typecheck
  * @property {boolean} lint
- * @property {boolean} coverage
  */
 void 0;

scripts/build/tests.mjs

@@ -35,7 +35,6 @@ export async function runConsoleTests(runJs, defaultReporter, runInParallel, opt
     const keepFailed = cmdLineOptions.keepFailed;
     const shards = +cmdLineOptions.shards || undefined;
     const shardId = +cmdLineOptions.shardId || undefined;
-    const coverage = cmdLineOptions.coverage;
     if (!cmdLineOptions.dirty) {
         if (options.watching) {
             console.log(chalk.yellowBright(`[watch] cleaning test directories...`));
@@ -83,13 +82,6 @@ export async function runConsoleTests(runJs, defaultReporter, runInParallel, opt
     /** @type {string[]} */
     const args = [];
 
-    let execPath = process.execPath;
-    if (coverage) {
-        args.push("exec", "c8");
-        args.push(execPath);
-        execPath = "npm";
-    }
-
     // timeout normally isn't necessary but Travis-CI has been timing out on compiler baselines occasionally
     // default timeout is 2sec which really should be enough, but maybe we just need a small amount longer
     if (!runInParallel) {
@@ -139,7 +131,7 @@ export async function runConsoleTests(runJs, defaultReporter, runInParallel, opt
     try {
         setNodeEnvToDevelopment();
 
-        const { exitCode } = await exec(execPath, args, { token: options.token });
+        const { exitCode } = await exec(process.execPath, args, { token: options.token });
         if (exitCode !== 0) {
             errorStatus = exitCode;
             error = new Error(`Process exited with status code ${errorStatus}.`);