Closes #1131

Author: sebastianbergmann

ChangeLog-11.0.md

@@ -7,6 +7,7 @@ All notable changes are documented in this file using the [Keep a CHANGELOG](htt
 ### Fixed
 
 * [#1092](https://github.com/sebastianbergmann/php-code-coverage/issues/1092): Error in `DOMDocument::saveXML()` is not handled
+* [#1131](https://github.com/sebastianbergmann/php-code-coverage/issues/1131): Invalid XML generated when both PCOV and Xdebug are loaded
 
 ## [11.0.11] - 2025-08-27
 

src/CodeCoverage.php

@@ -25,6 +25,8 @@
 use SebastianBergmann\CodeCoverage\Data\ProcessedCodeCoverageData;
 use SebastianBergmann\CodeCoverage\Data\RawCodeCoverageData;
 use SebastianBergmann\CodeCoverage\Driver\Driver;
+use SebastianBergmann\CodeCoverage\Driver\PcovDriver;
+use SebastianBergmann\CodeCoverage\Driver\XdebugDriver;
 use SebastianBergmann\CodeCoverage\Node\Builder;
 use SebastianBergmann\CodeCoverage\Node\Directory;
 use SebastianBergmann\CodeCoverage\StaticAnalysis\CachingFileAnalyser;
@@ -377,6 +379,22 @@ public function detectsDeadCode(): bool
         return $this->driver->detectsDeadCode();
     }
 
+    /**
+     * @internal
+     */
+    public function driverIsPcov(): bool
+    {
+        return $this->driver instanceof PcovDriver;
+    }
+
+    /**
+     * @internal
+     */
+    public function driverIsXdebug(): bool
+    {
+        return $this->driver instanceof XdebugDriver;
+    }
+
     /**
      * @throws ReflectionException
      * @throws UnintentionallyCoveredCodeException

src/Report/Xml/BuildInformation.php

@@ -13,6 +13,7 @@
 use function phpversion;
 use DateTimeImmutable;
 use DOMElement;
+use SebastianBergmann\CodeCoverage\CodeCoverage;
 use SebastianBergmann\Environment\Runtime;
 
 /**
@@ -27,7 +28,7 @@ public function __construct(DOMElement $contextNode)
         $this->contextNode = $contextNode;
     }
 
-    public function setRuntimeInformation(Runtime $runtime): void
+    public function setRuntimeInformation(Runtime $runtime, CodeCoverage $coverage): void
     {
         $runtimeNode = $this->nodeByName('runtime');
 
@@ -37,14 +38,12 @@ public function setRuntimeInformation(Runtime $runtime): void
 
         $driverNode = $this->nodeByName('driver');
 
-        if ($runtime->hasXdebug()) {
-            $driverNode->setAttribute('name', 'xdebug');
-            $driverNode->setAttribute('version', phpversion('xdebug'));
-        }
-
-        if ($runtime->hasPCOV()) {
+        if ($coverage->driverIsPcov()) {
             $driverNode->setAttribute('name', 'pcov');
             $driverNode->setAttribute('version', phpversion('pcov'));
+        } elseif ($coverage->driverIsXdebug()) {
+            $driverNode->setAttribute('name', 'xdebug');
+            $driverNode->setAttribute('version', phpversion('xdebug'));
         }
     }
 

src/Report/Xml/Facade.php

@@ -64,17 +64,17 @@ public function process(CodeCoverage $coverage, string $target): void
             $coverage->getReport()->name(),
         );
 
-        $this->setBuildInformation();
+        $this->setBuildInformation($coverage);
         $this->processTests($coverage->getTests());
         $this->processDirectory($report, $this->project);
 
         $this->saveDocument($this->project->asDom(), 'index');
     }
 
-    private function setBuildInformation(): void
+    private function setBuildInformation(CodeCoverage $coverage): void
     {
         $buildNode = $this->project->buildInformation();
-        $buildNode->setRuntimeInformation(new Runtime);
+        $buildNode->setRuntimeInformation(new Runtime, $coverage);
         $buildNode->setBuildTime(new DateTimeImmutable);
         $buildNode->setGeneratorVersions($this->phpUnitVersion, Version::id());
     }