Adding doc blocks and correcting constructor of CakeCliReporter

Fixing TestSuite console code coverage output.
cakephp · Jan 10, 2010 · f4bda62 · f4bda62
1 parent bcd7d34
commit f4bda62
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 4 deletions.
diff --git a/cake/console/libs/testsuite.php b/cake/console/libs/testsuite.php
@@ -278,7 +278,11 @@ function __canRun() {
  * @access private
  */
 	function __run() {
-		$Reporter = new CakeCliReporter();
+		$Reporter = new CakeCliReporter('utf-8', array(
+			'app' => $this->Manager->appTest,
+			'plugin' => $this->Manager->pluginTest,
+			'group' => ($this->type === 'group')
+		));
 
 		if ($this->type == 'all') {
 			return $this->Manager->runAllTests($Reporter);
@@ -332,6 +336,7 @@ function __run() {
 		$result = $this->Manager->runTestCase($case, $Reporter);
 		if ($this->doCoverage) {
 			CodeCoverageManager::report();
+			$this->out();
 		}
 
 		return $result;

diff --git a/cake/tests/lib/reporter/cake_cli_reporter.php b/cake/tests/lib/reporter/cake_cli_reporter.php
@@ -34,11 +34,29 @@
  * @subpackage cake.tests.libs.reporter
  */
 class CakeCliReporter extends CakeBaseReporter {
-
+/**
+ * separator string for fail, error, exception, and skip messages.
+ *
+ * @var string
+ */
 	var $separator = '->';
 
-	function CakeCLIReporter($separator = NULL) {
-		$this->SimpleReporter();
+/**
+ * array of 'request' parameters
+ *
+ * @var array
+ */
+	var $params = array();
+
+/**
+ * Constructor
+ *
+ * @param string $separator 
+ * @param array $params 
+ * @return void
+ */
+	function CakeCLIReporter($separator = NULL, $params = array()) {
+		$this->CakeBaseReporter('utf-8', $params);
 		if (!is_null($separator)) {
 			$this->setFailDetailSeparator($separator);
 		}