fix

Author: staabm

src/Data/ProcessedBranchCoverageData.php

@@ -17,7 +17,7 @@
  * @phpstan-import-type TestIdType from ProcessedCodeCoverageData
  * @phpstan-import-type XdebugBranchCoverageType from XdebugDriver
  */
-final readonly class ProcessedBranchCoverageData
+final class ProcessedBranchCoverageData
 {
     /**
      * @param XdebugBranchCoverageType $xdebugCoverageData
@@ -36,19 +36,20 @@ public static function fromXdebugCoverage(array $xdebugCoverageData): self
     }
 
     public function __construct(
-        public int $op_start,
-        public int $op_end,
-        public int $line_start,
-        public int $line_end,
+        readonly public int $op_start,
+        readonly public int $op_end,
+        readonly public int $line_start,
+        readonly public int $line_end,
         /** @var list<TestIdType> */
         public array $hit,
         /** @var array<int, int> */
-        public array $out,
+        readonly public array $out,
         /** @var array<int, int> */
-        public array $out_hit,
+        readonly public array $out_hit,
     ) {
     }
 
+    #[\NoDiscard]
     public function merge(self $data): self
     {
         return new self(
@@ -65,19 +66,8 @@ public function merge(self $data): self
     /**
      * @param TestIdType $testCaseId
      */
-    public function recordHit(string $testCaseId): self
+    public function recordHit(string $testCaseId): void
     {
-        $hit   = $this->hit;
-        $hit[] = $testCaseId;
-
-        return new self(
-            $this->op_start,
-            $this->op_end,
-            $this->line_start,
-            $this->line_end,
-            $hit,
-            $this->out,
-            $this->out_hit,
-        );
+        $this->hit[] = $testCaseId;
     }
 }

src/Data/ProcessedFunctionCoverageData.php

@@ -15,7 +15,7 @@
  * @phpstan-import-type TestIdType from ProcessedCodeCoverageData
  * @phpstan-import-type XdebugFunctionCoverageType from XdebugDriver
  */
-final class ProcessedFunctionCoverageData
+final readonly class ProcessedFunctionCoverageData
 {
     /**
      * @param XdebugFunctionCoverageType $xdebugCoverageData

src/Data/ProcessedPathCoverageData.php

@@ -9,6 +9,7 @@
  */
 namespace SebastianBergmann\CodeCoverage\Data;
 
+use NoDiscard;
 use function array_merge;
 use function array_unique;
 use SebastianBergmann\CodeCoverage\Driver\XdebugDriver;
@@ -17,7 +18,7 @@
  * @phpstan-import-type TestIdType from ProcessedCodeCoverageData
  * @phpstan-import-type XdebugPathCoverageType from XdebugDriver
  */
-final readonly class ProcessedPathCoverageData
+final class ProcessedPathCoverageData
 {
     /**
      * @param XdebugPathCoverageType $xdebugCoverageData
@@ -32,12 +33,13 @@ public static function fromXdebugCoverage(array $xdebugCoverageData): self
 
     public function __construct(
         /** @var array<int, int> */
-        public array $path,
+        public readonly array $path,
         /** @var list<TestIdType> */
         public array $hit,
     ) {
     }
 
+    #[\NoDiscard]
     public function merge(self $data): self
     {
         return new self(
@@ -49,14 +51,8 @@ public function merge(self $data): self
     /**
      * @param TestIdType $testCaseId
      */
-    public function recordHit(string $testCaseId): self
+    public function recordHit(string $testCaseId): void
     {
-        $hit   = $this->hit;
-        $hit[] = $testCaseId;
-
-        return new self(
-            $this->path,
-            $hit,
-        );
+        $this->hit[] = $testCaseId;
     }
 }