From fcac58e04b8de5e9b9fb6c668afe1c3d0e0a67f7 Mon Sep 17 00:00:00 2001 From: Mike Erickson Date: Fri, 5 Oct 2018 12:31:48 -0700 Subject: [PATCH] final cleanup --- src/ResultPrinter71.php | 4 ++++ tests/Unit/PrinterTest.php | 8 +++----- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/ResultPrinter71.php b/src/ResultPrinter71.php index d0318bf..21cd8c6 100755 --- a/src/ResultPrinter71.php +++ b/src/ResultPrinter71.php @@ -164,6 +164,10 @@ protected function setMessageColor(string $type, $msg): string $marker = "fail"; } + // if user turned off colors, reset to white + if(!$this->colors) { + $color = "37"; + } $testMarker = $this->markers[$marker]; return "\033[01;{$color}m{$testMarker}{$msg}\033[0m"; } diff --git a/tests/Unit/PrinterTest.php b/tests/Unit/PrinterTest.php index 259fbc6..f85abdf 100755 --- a/tests/Unit/PrinterTest.php +++ b/tests/Unit/PrinterTest.php @@ -66,22 +66,21 @@ public function should_fail_again() } - - /** @test */ + /** @skip */ public function should_skip() { $this->markTestSkipped(); } - /** @test */ + /** @skip */ public function should_skip_another() { $this->markTestSkipped(); } - /** @test */ + /** @skip */ public function should_be_incomplete() { $this->markTestIncomplete(); @@ -95,5 +94,4 @@ public function should_be_risky() } - }