Skip to content

Commit a3ad9d1

Browse files
Slamdunksebastianbergmann
authored andcommitted
CodeCoverage: process PHP report as first in list to avoid serializing cache data
1 parent 63d1346 commit a3ad9d1

File tree

1 file changed

+15
-15
lines changed

1 file changed

+15
-15
lines changed

src/TextUI/TestRunner.php

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -660,6 +660,21 @@ public function run(TestSuite $suite, array $arguments = [], array $warnings = [
660660
$this->printer->printResult($result);
661661

662662
if (isset($codeCoverage)) {
663+
if (isset($arguments['coveragePHP'])) {
664+
$this->codeCoverageGenerationStart('PHP');
665+
666+
try {
667+
$writer = new PhpReport;
668+
$writer->process($codeCoverage, $arguments['coveragePHP']);
669+
670+
$this->codeCoverageGenerationSucceeded();
671+
672+
unset($writer);
673+
} catch (CodeCoverageException $e) {
674+
$this->codeCoverageGenerationFailed($e);
675+
}
676+
}
677+
663678
if (isset($arguments['coverageClover'])) {
664679
$this->codeCoverageGenerationStart('Clover XML');
665680

@@ -728,21 +743,6 @@ public function run(TestSuite $suite, array $arguments = [], array $warnings = [
728743
}
729744
}
730745

731-
if (isset($arguments['coveragePHP'])) {
732-
$this->codeCoverageGenerationStart('PHP');
733-
734-
try {
735-
$writer = new PhpReport;
736-
$writer->process($codeCoverage, $arguments['coveragePHP']);
737-
738-
$this->codeCoverageGenerationSucceeded();
739-
740-
unset($writer);
741-
} catch (CodeCoverageException $e) {
742-
$this->codeCoverageGenerationFailed($e);
743-
}
744-
}
745-
746746
if (isset($arguments['coverageText'])) {
747747
if ($arguments['coverageText'] === 'php://stdout') {
748748
$outputStream = $this->printer;

0 commit comments

Comments
 (0)