Moving buffer start in CakeHtmlReporter to fix errors when displaying…

… menus. Adding paintDocumentStart() to CakeTextReporter::paintHeader() to fix issues with test results not being text/plain.
cakephp · Jan 10, 2010 · 3dad64c · 3dad64c
1 parent a41678e
commit 3dad64c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/cake/tests/lib/reporter/cake_html_reporter.php b/cake/tests/lib/reporter/cake_html_reporter.php
@@ -38,7 +38,6 @@ class CakeHtmlReporter extends CakeBaseReporter {
  */
 	function paintHeader($testName) {
 		$this->sendNoCacheHeaders();
-		ob_start();
 		$this->paintDocumentStart();
 		$this->paintTestMenu();
 		echo "<h2>$testName</h2>\n";
@@ -51,6 +50,7 @@ function paintHeader($testName) {
  * @return void
  */
 	function paintDocumentStart() {
+		ob_start();
 		$baseDir = $this->params['baseDir'];
 		include CAKE_TESTS_LIB . 'templates' . DS . 'header.php';
 	}

diff --git a/cake/tests/lib/reporter/cake_text_reporter.php b/cake/tests/lib/reporter/cake_text_reporter.php
@@ -72,6 +72,7 @@ function paintFooter($test_name) {
  * @access public
  */
 	function paintHeader($test_name) {
+		$this->paintDocumentStart();
 		echo "$test_name\n";
 		flush();
 	}