From 07b43be1259212d7d4146d4f77d55cf0adf4b42c Mon Sep 17 00:00:00 2001 From: Mark Story Date: Mon, 11 Jan 2010 18:00:45 -0500 Subject: [PATCH] Making CLI case code coverage reports include text. --- cake/tests/lib/code_coverage_manager.php | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/cake/tests/lib/code_coverage_manager.php b/cake/tests/lib/code_coverage_manager.php index 503a7dcdf5a..94684290113 100644 --- a/cake/tests/lib/code_coverage_manager.php +++ b/cake/tests/lib/code_coverage_manager.php @@ -681,7 +681,6 @@ function __paintGroupResultLineCli($file, $lineCount, $coveredCount) { $manager =& CodeCoverageManager::getInstance(); $codeCoverage = $manager->__calcCoverage($lineCount, $coveredCount); $class = 'bad'; - if ($codeCoverage > 50) { $class = 'ok'; } @@ -702,7 +701,14 @@ function __paintGroupResultLineCli($file, $lineCount, $coveredCount) { function __paintHeaderCli($lineCount, $coveredCount, $report) { $manager =& CodeCoverageManager::getInstance(); $codeCoverage = $manager->__calcCoverage($lineCount, $coveredCount); - return $report = 'Code Coverage: ' . $codeCoverage . '%'; + $class = 'bad'; + if ($codeCoverage > 50) { + $class = 'ok'; + } + if ($codeCoverage > 80) { + $class = 'good'; + } + return $report = "Code Coverage: $codeCoverage% ($class)\n"; } /**