[tests] Suppress extra JSON progress records

Author: coisa

docs/commands/tests.rst

@@ -167,6 +167,8 @@ Behavior
   command stores that payload inside ``output`` while keeping the standard
   DevTools JSON envelope. ``--json`` and ``--pretty-json`` therefore expose
   the same structured result, with formatting as the only difference.
+- in structured mode, the command suppresses intermediary ``Running...`` log
+  records so the output stream contains a single final JSON document.
 - when structured capture is active but PHPUnit does not emit parseable JSON,
   the command preserves the raw subprocess text inside ``output.raw_output``
   instead of dropping it.

src/Console/Command/TestsCommand.php

@@ -181,10 +181,12 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         $processOutput = $structuredOutput ? new BufferedOutput() : $output;
         $cacheEnabled = $this->isCacheEnabled($input);
 
-        $this->getLogger()
-            ->info('Running PHPUnit tests...', [
-                'input' => $input,
-            ]);
+        if (! $structuredOutput) {
+            $this->getLogger()
+                ->info('Running PHPUnit tests...', [
+                    'input' => $input,
+                ]);
+        }
 
         try {
             $minimumCoverage = $this->resolveMinimumCoverage($input);

tests/Console/Command/TestsCommandTest.php

@@ -254,9 +254,7 @@ public function executeWillDisablePhpUnitProgressWhenJsonIsRequested(): void
 
                 return TestsCommand::SUCCESS;
             })->shouldBeCalled();
-        $this->logger->info('Running PHPUnit tests...', Argument::that(
-            static fn(array $context): bool => $context['input'] instanceof InputInterface
-        ))->shouldBeCalled();
+        $this->logger->info(Argument::cetera())->shouldNotBeCalled();
         $this->logger->log(
             'info',
             'PHPUnit tests completed successfully.',
@@ -299,9 +297,7 @@ public function executeWillCaptureStructuredPhpUnitSummaryWhenAgentEnvironmentIs
 
                 return TestsCommand::SUCCESS;
             })->shouldBeCalled();
-        $this->logger->info('Running PHPUnit tests...', Argument::that(
-            static fn(array $context): bool => $context['input'] instanceof InputInterface
-        ))->shouldBeCalled();
+        $this->logger->info(Argument::cetera())->shouldNotBeCalled();
         $this->logger->log(
             'info',
             'PHPUnit tests completed successfully.',
@@ -343,9 +339,7 @@ public function executeWillKeepPrettyJsonInsideTheStandardCommandLogOutput(): vo
 
                 return TestsCommand::SUCCESS;
             })->shouldBeCalled();
-        $this->logger->info('Running PHPUnit tests...', Argument::that(
-            static fn(array $context): bool => $context['input'] instanceof InputInterface
-        ))->shouldBeCalled();
+        $this->logger->info(Argument::cetera())->shouldNotBeCalled();
         $this->logger->log(
             'info',
             'PHPUnit tests completed successfully.',
@@ -384,9 +378,7 @@ public function executeWillCaptureStructuredPhpUnitSummaryAfterCoveragePreludeWh
 
                 return TestsCommand::SUCCESS;
             })->shouldBeCalled();
-        $this->logger->info('Running PHPUnit tests...', Argument::that(
-            static fn(array $context): bool => $context['input'] instanceof InputInterface
-        ))->shouldBeCalled();
+        $this->logger->info(Argument::cetera())->shouldNotBeCalled();
         $this->logger->log(
             'info',
             'PHPUnit tests completed successfully.',
@@ -424,9 +416,7 @@ public function executeWillKeepRawPhpUnitOutputWhenStructuredSummaryCannotBeDeco
 
                 return TestsCommand::SUCCESS;
             })->shouldBeCalled();
-        $this->logger->info('Running PHPUnit tests...', Argument::that(
-            static fn(array $context): bool => $context['input'] instanceof InputInterface
-        ))->shouldBeCalled();
+        $this->logger->info(Argument::cetera())->shouldNotBeCalled();
         $this->logger->log(
             'info',
             'PHPUnit tests completed successfully.',
@@ -604,9 +594,7 @@ public function executeWillEmitStructuredCoverageFailurePayloadWhenMinimumCovera
 
                 return TestsCommand::SUCCESS;
             })->shouldBeCalled();
-        $this->logger->info('Running PHPUnit tests...', Argument::that(
-            static fn(array $context): bool => $context['input'] instanceof InputInterface
-        ))->shouldBeCalled();
+        $this->logger->info(Argument::cetera())->shouldNotBeCalled();
         $this->logger->log(Argument::cetera())->shouldNotBeCalled();
         $this->logger->error(
             'Minimum line coverage of 80.00% was not met. Current coverage: 75.00% (75/100 lines).',