Fixing CliReporter class name as well as making cli/text output the d…

…efault if the reporter is unknown.
cakephp · Jan 10, 2010 · 02dbbcd · 02dbbcd
1 parent e7e1a9b
commit 02dbbcd
Showing 1 changed file with 5 additions and 8 deletions.
diff --git a/cake/tests/lib/code_coverage_manager.php b/cake/tests/lib/code_coverage_manager.php
@@ -159,7 +159,8 @@ function setParams(&$reporter) {
 	}
 
 /**
- * Stops the current code coverage analyzation and dumps a nice report depending on the reporter that was passed to start()
+ * Stops the current code coverage analyzation and dumps a nice report 
+ * depending on the reporter that was passed to start()
  *
  * @return void
  * @static
@@ -185,10 +186,8 @@ function report($output = true) {
 					$result = $manager->reportCaseHtmlDiff(@file($testObjectFile), $coverageData, $execCodeLines, $manager->numDiffContextLines);
 					break;
 				case 'CakeCliReporter':
-					$result = $manager->reportCaseCli(@file($testObjectFile), $coverageData, $execCodeLines, $manager->numDiffContextLines);
-					break;
 				default:
-					trigger_error('Currently only HTML and CLI reporting is supported for code coverage analysis.');
+					$result = $manager->reportCaseCli(@file($testObjectFile), $coverageData, $execCodeLines, $manager->numDiffContextLines);
 					break;
 			}
 		} else {
@@ -199,11 +198,9 @@ function report($output = true) {
 				case 'CakeHtmlReporter':
 					$result = $manager->reportGroupHtml($testObjectFile, $coverageData, $execCodeLines, $manager->numDiffContextLines);
 					break;
-				case 'CLIReporter':
-					$result = $manager->reportGroupCli($testObjectFile, $coverageData, $execCodeLines, $manager->numDiffContextLines);
-					break;
+				case 'CakeCliReporter':
 				default:
-					trigger_error('Currently only HTML and CLI reporting is supported for code coverage analysis.');
+					$result = $manager->reportGroupCli($testObjectFile, $coverageData, $execCodeLines, $manager->numDiffContextLines);
 					break;
 			}
 		}