Merge branch '10.5' into 11.0

Author: sebastianbergmann

.psalm/baseline.xml

@@ -68,6 +68,7 @@
   <file src="src/Event/Value/Telemetry/HRTime.php">
     <ImpureMethodCall>
       <code><![CDATA[fromSecondsAndNanoseconds]]></code>
+      <code><![CDATA[fromSecondsAndNanoseconds]]></code>
     </ImpureMethodCall>
   </file>
   <file src="src/Event/Value/Telemetry/MemoryUsage.php">

ChangeLog-11.0.md

@@ -6,6 +6,7 @@ All notable changes of the PHPUnit 11.0 release series are documented in this fi
 
 ### Fixed
 
+* [#5652](https://github.com/sebastianbergmann/phpunit/issues/5652): `HRTime::duration()` throws `InvalidArgumentException`
 * `--exclude-filter` CLI option does not work
 
 ## [11.0.4] - 2024-02-29

src/Event/Value/Telemetry/HRTime.php

@@ -56,9 +56,6 @@ public function nanoseconds(): int
         return $this->nanoseconds;
     }
 
-    /**
-     * @throws InvalidArgumentException
-     */
     public function duration(self $start): Duration
     {
         $seconds     = $this->seconds - $start->seconds();
@@ -71,7 +68,7 @@ public function duration(self $start): Duration
         }
 
         if ($seconds < 0) {
-            throw new InvalidArgumentException('Start needs to be smaller.');
+            return Duration::fromSecondsAndNanoseconds(0, 0);
         }
 
         return Duration::fromSecondsAndNanoseconds(

tests/unit/Event/Value/Telemetry/HRTimeTest.php

@@ -124,7 +124,7 @@ public function testFromSecondsAndNanosecondsReturnsHRTime(): void
     }
 
     #[DataProvider('provideStartGreaterThanEnd')]
-    public function testDurationRejectsStartGreaterThanEnd(int $startSeconds, int $startNanoseconds, int $endSeconds, int $endNanoseconds): void
+    public function testDurationIgnoresStartGreaterThanEnd(int $startSeconds, int $startNanoseconds, int $endSeconds, int $endNanoseconds): void
     {
         $start = HRTime::fromSecondsAndNanoseconds(
             $startSeconds,
@@ -136,10 +136,10 @@ public function testDurationRejectsStartGreaterThanEnd(int $startSeconds, int $s
             $endNanoseconds,
         );
 
-        $this->expectException(InvalidArgumentException::class);
-        $this->expectExceptionMessage('Start needs to be smaller.');
+        $duration = $end->duration($start);
 
-        $end->duration($start);
+        $this->assertSame(0, $duration->seconds());
+        $this->assertSame(0, $duration->nanoseconds());
     }
 
     #[DataProvider('provideStartEndAndDuration')]