Change PHP report to use serialize, rather than var_export for better compatibility with data structure changes

Author: dvdoug

src/Report/PHP.php

@@ -22,21 +22,10 @@ final class PHP
      */
     public function process(CodeCoverage $coverage, ?string $target = null): string
     {
-        $filter = $coverage->filter();
-
         $buffer = \sprintf(
             '<?php
-$coverage = new SebastianBergmann\CodeCoverage\CodeCoverage;
-$coverage->setData(%s);
-$coverage->setTests(%s);
-
-$filter = $coverage->filter();
-$filter->setWhitelistedFiles(%s);
-
-return $coverage;',
-            \var_export($coverage->getData(true), true),
-            \var_export($coverage->getTests(), true),
-            \var_export($filter->getWhitelistedFiles(), true)
+return \unserialize(\'%s\');',
+            \serialize($coverage)
         );
 
         if ($target !== null) {

tests/tests/PHPTest.php

@@ -0,0 +1,33 @@
+<?php declare(strict_types=1);
+/*
+ * This file is part of phpunit/php-code-coverage.
+ *
+ * (c) Sebastian Bergmann <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+namespace SebastianBergmann\CodeCoverage\Report;
+
+use SebastianBergmann\CodeCoverage\TestCase;
+
+class PHPTest extends TestCase
+{
+    protected function tearDown(): void
+    {
+        parent::tearDown();
+
+        $this->removeTemporaryFiles();
+    }
+
+    public function testPHPSerialisationProducesValidCode(): void
+    {
+        $coverage = $this->getCoverageForBankAccount();
+
+        (new PHP())->process($coverage, self::$TEST_TMP_PATH . '/serialised.php');
+
+        $unserialised = require self::$TEST_TMP_PATH . '/serialised.php';
+
+        $this->assertEquals($coverage, $unserialised);
+    }
+}