Skip to content

Commit 4501167

Browse files
committed
Coverage output. It shows the coverage when enabling the strict coverage and guard coverage.
1 parent 4509dfc commit 4501167

File tree

2 files changed

+28
-3
lines changed

2 files changed

+28
-3
lines changed

src/PhpGitHooks/Module/PhpUnit/Service/GuardCoverageTool.php

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ public function run($warningMessage)
6363
{
6464
$outputMessage = new PreCommitOutputWriter(self::CHECKING_MESSAGE);
6565
$this->output->write($outputMessage->getMessage());
66-
66+
6767
$this->currentCoverage = $this->strictCoverageProcessor->process();
6868
$this->previousCoverage = $this->guardReader->read();
6969

@@ -75,7 +75,9 @@ public function run($warningMessage)
7575
$this->previousCoverage,
7676
$this->currentCoverage
7777
)
78-
) : $this->output->writeln($outputMessage->getSuccessfulMessage());
78+
) : $this->output->writeln(
79+
$outputMessage->getSuccessfulMessage() . $this->printGuardCoverage()
80+
);
7981

8082
$this->guardWriter->write($this->currentCoverage);
8183
}
@@ -87,4 +89,16 @@ private function isLowerCurrentCoverage()
8789
{
8890
return $this->currentCoverage < $this->previousCoverage;
8991
}
92+
93+
/**
94+
* @return string
95+
*/
96+
private function printGuardCoverage()
97+
{
98+
return ' <comment>[' .
99+
round($this->currentCoverage, 0) .
100+
'% >= ' .
101+
round($this->previousCoverage, 0) .
102+
'%]</comment>';
103+
}
90104
}

src/PhpGitHooks/Module/PhpUnit/Tests/Behaviour/GuardCoverageToolCommandHandlerTest.php

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,11 +67,22 @@ public function itShouldWorksFine()
6767
$this->shouldProcessStrictCoverage($currentCoverage);
6868
$this->shouldWriteOutput($outputMessage->getMessage());
6969
$this->shouldReadGuardCoverage($previousCoverage);
70-
$this->shouldWriteLnOutput($outputMessage->getSuccessfulMessage());
70+
$this->shouldWriteLnOutput($this->buildStrictCoverageSuccessfulMessage($currentCoverage, $previousCoverage,
71+
$outputMessage->getSuccessfulMessage()));
7172
$this->shouldWriteGuardCoverage($currentCoverage);
7273

7374
$this->guardCoverageToolCommandHandler->handle(
7475
new GuardCoverageCommand(HookQuestions::PHPUNIT_GUARD_COVERAGE_MESSAGE_DEFAULT)
7576
);
7677
}
78+
79+
private function buildStrictCoverageSuccessfulMessage($currentCoverage, $previousCoverage, $getSuccessfulMessage)
80+
{
81+
return $getSuccessfulMessage .
82+
' <comment>[' .
83+
round($currentCoverage, 0) .
84+
'% >= ' .
85+
round($previousCoverage, 0) .
86+
'%]</comment>';
87+
}
7788
}

0 commit comments

Comments
 (0)