only enable colors when output to a terminal (not to a file)

sebastianbergmann · Oct 6, 2014 · 591efd8 · 591efd8
1 parent 1695564
commit 591efd8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/TextUI/Command.php b/src/TextUI/Command.php
@@ -278,8 +278,11 @@ protected function handleArguments(array $argv)
 
         foreach ($this->options[0] as $option) {
             switch ($option[0]) {
-                case '--colors': {
-                    $this->arguments['colors'] = true;
+                case '--colors':
+                    if (function_exists("posix_isatty") && !posix_isatty(STDOUT)) {
+                        $this->arguments['colors'] = false;
+                    } else {
+                        $this->arguments['colors'] = true;
                     }
                 break;
 

diff --git a/src/Util/Configuration.php b/src/Util/Configuration.php
@@ -582,6 +582,9 @@ public function getPHPUnitConfiguration()
             $result['colors'] = $this->getBoolean(
                 (string) $root->getAttribute('colors'), false
             );
+            if ($result['colors'] && function_exists("posix_isatty") && !posix_isatty(STDOUT)) {
+                $result['colors'] = false;
+            }
         }
 
         /**