From 8a974d965c9cf1317c36df8fc6193102d2e62e99 Mon Sep 17 00:00:00 2001 From: Julian Simioni Date: Fri, 29 Sep 2017 11:58:35 -0400 Subject: [PATCH] Always use red for regression count color We currently display the regression count in yellow if there are zero regressions, and red if there are regressions. However, in general regressions are colored red (in test output, for example), known failures yellow, and passes/improvements green. Keeping this consistency feels a little more clear, so change the regression color to always be red. --- output_generators/terminal.js | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/output_generators/terminal.js b/output_generators/terminal.js index abcdae9..5426565 100644 --- a/output_generators/terminal.js +++ b/output_generators/terminal.js @@ -93,11 +93,10 @@ function prettyPrintSuiteResults( suiteResults, config, testSuites ){ console.log( 'Placeholders: ' + suiteResults.stats.placeholder.toString().cyan ); var numRegressions = suiteResults.stats.regression; - var regressionsColor = ( numRegressions > 0 ) ? 'red' : 'yellow'; var total = suiteResults.stats.pass + suiteResults.stats.fail + suiteResults.stats.regression; var pass = total - numRegressions; - console.log( 'Regressions: ' + numRegressions.toString()[ regressionsColor ] ); + console.log( 'Regressions: ' + numRegressions.toString().red); console.log( 'Took %sms', suiteResults.stats.timeTaken ); console.log( 'Test success rate %s%%', percentageForDisplay(total,pass));