Skip to content

Commit 90917ad

Browse files
committed
line coverage
1 parent 15a38ac commit 90917ad

File tree

2 files changed

+2
-24
lines changed

2 files changed

+2
-24
lines changed

src/Report/Xml/Facade.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@ private function processFile(FileNode $file, Directory $context): void
166166
$this->processFunction($function, $fileReport);
167167
}
168168

169+
$this->xmlWriter->startElement('coverage');
169170
foreach ($file->lineCoverageData() as $line => $tests) {
170171
if (!is_array($tests) || count($tests) === 0) {
171172
continue;
@@ -174,6 +175,7 @@ private function processFile(FileNode $file, Directory $context): void
174175
$coverage = $fileReport->lineCoverage((string) $line);
175176
$coverage->finalize($tests);
176177
}
178+
$this->xmlWriter->endElement();
177179

178180
$fileReport->source()->setSourceCode(
179181
file_get_contents($file->pathAsString()),

src/Report/Xml/File.php

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -34,30 +34,6 @@ public function totals(): Totals
3434

3535
public function lineCoverage(string $line): Coverage
3636
{
37-
if ($this->lineCoverage === null) {
38-
$this->lineCoverage = $this->contextNode->appendChild(
39-
$this->dom->createElementNS(
40-
Facade::XML_NAMESPACE,
41-
'coverage',
42-
),
43-
);
44-
}
45-
assert($this->lineCoverage instanceof DOMElement);
46-
47-
$lineNode = $this->lineCoverage->appendChild(
48-
$this->dom->createElementNS(
49-
Facade::XML_NAMESPACE,
50-
'line',
51-
),
52-
);
53-
54-
assert($lineNode instanceof DOMElement);
55-
5637
return new Coverage($this->xmlWriter, $line);
5738
}
58-
59-
protected function contextNode(): DOMElement
60-
{
61-
return $this->contextNode;
62-
}
6339
}

0 commit comments

Comments
 (0)