Merge pull request #618 from sindresorhus/mini-reporter-color

remove color from test title in mini reporter
avajs · Mar 10, 2016 · f1966d9 · f1966d9
2 parents 73c8b6b + 9c867e9
commit f1966d9
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/lib/reporters/mini.js b/lib/reporters/mini.js
@@ -70,19 +70,16 @@ MiniReporter.prototype.prefix = function (str) {
 };
 
 MiniReporter.prototype._test = function (test) {
-	var title;
+	var title = test.title;
 
 	if (test.todo) {
-		title = colors.todo('- ' + test.title);
 		this.todoCount++;
 	} else if (test.skip) {
-		title = colors.skip('- ' + test.title);
 		this.skipCount++;
 	} else if (test.error) {
 		title = colors.error(test.title);
 		this.failCount++;
 	} else {
-		title = colors.pass(test.title);
 		this.passCount++;
 	}
 

diff --git a/test/reporters/mini.js b/test/reporters/mini.js
@@ -120,7 +120,7 @@ test('skipped test', function (t) {
 
 	var expectedOutput = [
 		' ',
-		' ⠋ ' + chalk.yellow('- skipped'),
+		' ⠋ ' + chalk.yellow('skipped'),
 		'',
 		'   ' + chalk.yellow('1 skipped')
 	].join('\n');
@@ -140,7 +140,7 @@ test('todo test', function (t) {
 
 	var expectedOutput = [
 		' ',
-		' ⠋ ' + chalk.blue('- todo'),
+		' ⠋ ' + chalk.blue('todo'),
 		'',
 		'   ' + chalk.blue('1 todo')
 	].join('\n');