Correcting classname used in CodeCoverageManager

Moving CodeCoverageManager::report() to be inside the html document.
cakephp · Jan 10, 2010 · 03da53b · 03da53b
1 parent f52900f
commit 03da53b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 9 deletions.
diff --git a/cake/tests/lib/cake_test_suite_dispatcher.php b/cake/tests/lib/cake_test_suite_dispatcher.php
@@ -230,9 +230,6 @@ function _runGroupTest() {
 				CodeCoverageManager::start($this->params['group'], $Reporter);
 			}
 			$this->Manager->runGroupTest(ucfirst($this->params['group']), $Reporter);
-			if ($this->params['codeCoverage']) {
-				CodeCoverageManager::report();
-			}
 		}
 	}
 
@@ -246,12 +243,7 @@ function _runTestCase() {
 		if ($this->params['codeCoverage']) {
 			CodeCoverageManager::start($this->params['case'], $Reporter);
 		}
-
 		$this->Manager->runTestCase($this->params['case'], $Reporter);
-
-		if ($this->params['codeCoverage']) {
-			CodeCoverageManager::report();
-		}
 	}
 }
 ?>
diff --git a/cake/tests/lib/code_coverage_manager.php b/cake/tests/lib/code_coverage_manager.php
@@ -155,7 +155,7 @@ function report($output = true) {
 				case 'CakeHtmlReporter':
 					$result = $manager->reportCaseHtmlDiff(@file($testObjectFile), $coverageData, $execCodeLines, $manager->numDiffContextLines);
 					break;
-				case 'CLIReporter':
+				case 'CakeCliReporter':
 					$result = $manager->reportCaseCli(@file($testObjectFile), $coverageData, $execCodeLines, $manager->numDiffContextLines);
 					break;
 				default:

diff --git a/cake/tests/lib/reporter/cake_html_reporter.php b/cake/tests/lib/reporter/cake_html_reporter.php
@@ -175,6 +175,13 @@ function paintFooter($test_name) {
 		}
 		echo $this->_paintLinks();
 		echo '</div>';
+		if (
+			isset($this->params['codeCoverage']) && 
+			$this->params['codeCoverage'] && 
+			class_exists('CodeCoverageManager')
+		) {
+			CodeCoverageManager::report();
+		}
 		$this->paintDocumentEnd();
 	}