simplify

Author: staabm

src/Report/Xml/Coverage.php

@@ -10,7 +10,6 @@
 namespace SebastianBergmann\CodeCoverage\Report\Xml;
 
 use DOMElement;
-use SebastianBergmann\CodeCoverage\ReportAlreadyFinalizedException;
 use XMLWriter;
 
 /**
@@ -19,6 +18,7 @@
 final class Coverage
 {
     private readonly DOMElement $contextNode;
+    private bool $finalized = false;
     private readonly string $line;
 
     public function __construct(DOMElement $context, string $line)
@@ -29,14 +29,20 @@ public function __construct(DOMElement $context, string $line)
 
     public function finalize(array $tests): void
     {
-        $xml = '<line nr="'. $this->line .'">';
+        $writer = new XMLWriter;
+        $writer->openMemory();
+        $writer->startElementNs(null, $this->contextNode->nodeName, 'https://schema.phpunit.de/coverage/1.0');
+        $writer->writeAttribute('nr', $this->line);
+
         foreach ($tests as $test) {
-            $xml .= '<covered by="'. htmlspecialchars($test, ENT_QUOTES | ENT_XML1) .'" />';
+            $writer->startElement('covered');
+            $writer->writeAttribute('by', $test);
+            $writer->endElement();
         }
-        $xml .= '</line>';
+        $writer->endElement();
 
         $fragment = $this->contextNode->ownerDocument->createDocumentFragment();
-        $fragment->appendXML($xml);
+        $fragment->appendXML($writer->outputMemory());
 
         $this->contextNode->parentNode->replaceChild(
             $fragment,