test_runner: remove plan option from run()

This commit removes the plan option to run(). I believe it was
added by mistake. It is not documented, untested, and a test
plan does not make sense in the context of run().

This commit also fixes a minor formatting issue in a related
fixture.

Refs: #52860
PR-URL: #53834
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Marco Ippolito <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>

Author: cjihrig

lib/internal/test_runner/runner.js

@@ -483,7 +483,6 @@ function run(options = kEmptyObject) {
     watch,
     setup,
     only,
-    plan,
   } = options;
 
   if (files != null) {
@@ -552,7 +551,7 @@ function run(options = kEmptyObject) {
     });
   }
 
-  const root = createTestTree({ __proto__: null, concurrency, timeout, signal, plan });
+  const root = createTestTree({ __proto__: null, concurrency, timeout, signal });
   root.harness.shouldColorizeTestFiles ||= shouldColorizeTestFiles(root);
 
   if (process.env.NODE_TEST_CONTEXT !== undefined) {

test/fixtures/test-runner/output/test-runner-plan.js

@@ -74,6 +74,6 @@ test('planning with streams', (t, done) => {
   });
 
   stream.on('end', () => {
-   done();
+    done();
   });
 })