Closes #1737

sebastianbergmann · Jun 3, 2015 · 00b9b5d · 00b9b5d
1 parent 06b645c
commit 00b9b5d
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/ChangeLog-4.6.md b/ChangeLog-4.6.md
@@ -7,6 +7,7 @@
 * Merged [#1725](https://github.com/sebastianbergmann/phpunit/pull/1725): Update phpDox XSD URI
 * Merged [#1735](https://github.com/sebastianbergmann/phpunit/pull/1735): Mute `chdir()` failures in XInclude handling of XML configuration file
 * Merged [#1736](https://github.com/sebastianbergmann/phpunit/pull/1736): Verify that phar file can be overwritten before attempting self update
+* Fixed [#1737](https://github.com/sebastianbergmann/phpunit/issues/1737): Confusing output from `--testdox` for empty test class
 
 ## PHPUnit 4.6.9
 

diff --git a/src/Util/TestDox/ResultPrinter/Text.php b/src/Util/TestDox/ResultPrinter/Text.php
@@ -21,13 +21,23 @@
  */
 class PHPUnit_Util_TestDox_ResultPrinter_Text extends PHPUnit_Util_TestDox_ResultPrinter
 {
+    /**
+     * @var bool
+     */
+    private $isWarning = false;
+
     /**
      * Handler for 'start class' event.
      *
      * @param string $name
      */
     protected function startClass($name)
     {
+        if ($name == 'PHPUnit_Framework_Warning') {
+            $this->isWarning = true;
+            return;
+        }
+
         $this->write($this->currentTestClassPrettified . "\n");
     }
 
@@ -39,6 +49,10 @@ protected function startClass($name)
      */
     protected function onTest($name, $success = true)
     {
+        if ($this->isWarning) {
+            return;
+        }
+
         if ($success) {
             $this->write(' [x] ');
         } else {
@@ -55,6 +69,11 @@ protected function onTest($name, $success = true)
      */
     protected function endClass($name)
     {
+        if ($this->isWarning) {
+            $this->isWarning = false;
+            return;
+        }
+
         $this->write("\n");
     }
 }