From fd073809d047244559b8c132c1d2043351fdd5f1 Mon Sep 17 00:00:00 2001
From: Mark Story <mark@mark-story.com>
Date: Sat, 8 May 2010 17:14:47 -0400
Subject: [PATCH] Removing references to non phpunit code coverage parsing.
 Adding stub of new code coverage report generator. Updating related files.

---
 cake/tests/lib/cake_test_suite_dispatcher.php |  7 ----
 .../lib/coverage/html_coverage_report.php     | 35 +++++++++++++++++++
 .../tests/lib/reporter/cake_html_reporter.php | 21 +++++++----
 .../tests/lib/reporter/cake_text_reporter.php |  9 ++---
 cake/tests/lib/test_manager.php               |  3 --
 5 files changed, 53 insertions(+), 22 deletions(-)
 create mode 100644 cake/tests/lib/coverage/html_coverage_report.php

diff --git a/cake/tests/lib/cake_test_suite_dispatcher.php b/cake/tests/lib/cake_test_suite_dispatcher.php
index cc91a822639..1a4f4850300 100644
--- a/cake/tests/lib/cake_test_suite_dispatcher.php
+++ b/cake/tests/lib/cake_test_suite_dispatcher.php
@@ -236,7 +236,6 @@ function _parseParams() {
 		if (isset($_GET['code_coverage'])) {
 			$this->params['codeCoverage'] = true;
 			$this->_checkXdebug();
-			require_once CAKE_TESTS_LIB . 'code_coverage_manager.php';
 		}
 		$this->params['baseUrl'] = $this->_baseUrl;
 		$this->params['baseDir'] = $this->_baseDir;
@@ -250,9 +249,6 @@ function _parseParams() {
  */
 	function _runGroupTest() {
 		$Reporter = CakeTestSuiteDispatcher::getReporter();
-		if ($this->params['codeCoverage']) {
-			CodeCoverageManager::init($this->params['group'], $Reporter);
-		}
 		if ('all' == $this->params['group']) {
 			$this->Manager->runAllTests($Reporter);
 		} else {
@@ -267,9 +263,6 @@ function _runGroupTest() {
  */
 	function _runTestCase() {
 		$Reporter = CakeTestSuiteDispatcher::getReporter();
-		if ($this->params['codeCoverage']) {
-			CodeCoverageManager::init($this->params['case'], $Reporter);
-		}
 		$this->Manager->runTestCase($this->params['case'], $Reporter, $this->params['codeCoverage']);
 	}
 }
diff --git a/cake/tests/lib/coverage/html_coverage_report.php b/cake/tests/lib/coverage/html_coverage_report.php
new file mode 100644
index 00000000000..4c1840a7e68
--- /dev/null
+++ b/cake/tests/lib/coverage/html_coverage_report.php
@@ -0,0 +1,35 @@
+<?php
+
+/**
+ * Generates an HTML coverage report from data provided by PHPUnit.
+ *
+ * @package default
+ * @author Mark Story
+ */
+class HtmlCoverageReport {
+/**
+ * coverage data
+ *
+ * @var string
+ */
+	protected $_coverage;
+
+/**
+ * Constructor
+ *
+ * @param array $coverage Array of coverage data from PHPUnit_Test_Result
+ * @return void
+ */
+	public function __construct($coverage) {
+		$this->_coverage = $coverage;
+	}
+
+/**
+ * Generates report html to display.
+ *
+ * @return string compiled html report.
+ */
+	public function report() {
+		
+	}
+}
\ No newline at end of file
diff --git a/cake/tests/lib/reporter/cake_html_reporter.php b/cake/tests/lib/reporter/cake_html_reporter.php
index 68d6dbd092b..eec5de13661 100755
--- a/cake/tests/lib/reporter/cake_html_reporter.php
+++ b/cake/tests/lib/reporter/cake_html_reporter.php
@@ -175,16 +175,25 @@ public function paintFooter($result) {
 		}
 		echo $this->_paintLinks();
 		echo '</div>';
-		if (
-			isset($this->params['codeCoverage']) &&
-			$this->params['codeCoverage'] &&
-			class_exists('CodeCoverageManager')
-		) {
-			//CodeCoverageManager::report();
+		if (isset($this->params['codeCoverage']) && $this->params['codeCoverage']) {
+			$coverage = $result->getCodeCoverageInformation();
+			echo $this->paintCoverage($coverage);
 		}
 		$this->paintDocumentEnd();
 	}
 
+/**
+ * Paints a code coverage report.
+ *
+ * @return void
+ */
+	public function paintCoverage($coverage) {
+		$file = dirname(dirname(__FILE__)) . '/coverage/html_coverage_report.php';
+		include $file;
+		$reporter = new HtmlCoverageReport($coverage);
+		echo $reporter->report();
+	}
+
 /**
  * Renders the links that for accessing things in the test suite.
  *
diff --git a/cake/tests/lib/reporter/cake_text_reporter.php b/cake/tests/lib/reporter/cake_text_reporter.php
index 4bc51d509e5..31cec5c3e3f 100644
--- a/cake/tests/lib/reporter/cake_text_reporter.php
+++ b/cake/tests/lib/reporter/cake_text_reporter.php
@@ -61,12 +61,9 @@ public function paintFooter($test_name) {
 		if (function_exists('memory_get_peak_usage')) {
 			echo 'Peak memory use: (in bytes): ' . number_format(memory_get_peak_usage()) . "\n";
 		}
-		if (
-			isset($this->params['codeCoverage']) && 
-			$this->params['codeCoverage'] && 
-			class_exists('CodeCoverageManager')
-		) {
-			CodeCoverageManager::report();
+		if (isset($this->params['codeCoverage']) && $this->params['codeCoverage']) {
+			$coverage = $result->getCodeCoverageInformation();
+			echo $this->paintCoverage($coverage);
 		}
 	}
 
diff --git a/cake/tests/lib/test_manager.php b/cake/tests/lib/test_manager.php
index 07bb09993b0..67e35bc61e9 100644
--- a/cake/tests/lib/test_manager.php
+++ b/cake/tests/lib/test_manager.php
@@ -167,9 +167,6 @@ protected function run($reporter, $codeCoverage = false) {
 		$reporter->paintHeader();
 		$this->getTestSuite()->run($result);
 		$reporter->paintResult($result);
-		// echo '<pre>';
-		// var_dump($result->getCodeCoverageInformation());
-		// echo '</pre>';
 		return $result;
 	}