Closes #6446

Author: sebastianbergmann

ChangeLog-12.5.md

@@ -2,6 +2,12 @@
 
 All notable changes of the PHPUnit 12.5 release series are documented in this file using the [Keep a CHANGELOG](https://keepachangelog.com/) principles.
 
+## [12.5.4] - 2025-MM-DD
+
+### Fixed
+
+* [#6446](https://github.com/sebastianbergmann/phpunit/issues/6446): Test runner crashes with `Timer::start() has to be called before Timer::stop()`
+
 ## [12.5.3] - 2025-12-11
 
 ### Changed
@@ -32,6 +38,7 @@ All notable changes of the PHPUnit 12.5 release series are documented in this fi
 * [#6380](https://github.com/sebastianbergmann/phpunit/pull/6380): Allow `Throwable` in `expectExceptionObject()`
 * A PHPUnit notice is now emitted for test methods that create a mock object but do not configure an expectation for it
 
+[12.5.4]: https://github.com/sebastianbergmann/phpunit/compare/12.5.3...12.5
 [12.5.3]: https://github.com/sebastianbergmann/phpunit/compare/12.5.2...12.5.3
 [12.5.2]: https://github.com/sebastianbergmann/phpunit/compare/12.5.1...12.5.2
 [12.5.1]: https://github.com/sebastianbergmann/phpunit/compare/12.5.0...12.5.1

src/Runner/CodeCoverage.php

@@ -213,9 +213,9 @@ public function stop(bool $append, null|false|TargetCollection $covers = null, ?
             return;
         }
 
-        $time = $this->timer()->stop()->asSeconds();
-
-        $status = TestStatus::unknown();
+        $time             = $this->timer()->stop()->asSeconds();
+        $status           = TestStatus::unknown();
+        $this->collecting = false;
 
         if ($this->test !== null) {
             if ($this->test->status()->isSuccess()) {
@@ -257,8 +257,7 @@ public function stop(bool $append, null|false|TargetCollection $covers = null, ?
 
         $this->codeCoverage->stop($append, $status, $covers, $uses, $time);
 
-        $this->test       = null;
-        $this->collecting = false;
+        $this->test = null;
     }
 
     public function deactivate(): void