Close #625 PR: Test chalk colors in reporter tests. Fixes #605

lib/reporters/mini.js

@@ -7,6 +7,11 @@ var spinners = require('cli-spinners');
 var chalk = require('chalk');
 var colors = require('../colors');
 
+chalk.enabled = true;
+Object.keys(colors).forEach(function (key) {
+	colors[key].enabled = true;
+});
+
 function MiniReporter() {
 	if (!(this instanceof MiniReporter)) {
 		return new MiniReporter();

lib/reporters/verbose.js

@@ -4,6 +4,10 @@ var figures = require('figures');
 var colors = require('../colors');
 var plur = require('plur');
 
+Object.keys(colors).forEach(function (key) {
+	colors[key].enabled = true;
+});
+
 function VerboseReporter() {
 	if (!(this instanceof VerboseReporter)) {
 		return new VerboseReporter();
@@ -40,7 +44,7 @@ VerboseReporter.prototype.test = function (test) {
 
 VerboseReporter.prototype.unhandledError = function (err) {
 	if (err.type === 'exception' && err.name === 'AvaError') {
-		return '  ' + colors.error(figures.cross) + ' ' + err.message;
+		return colors.error('  ' + figures.cross + ' ' + err.message);
 	}
 
 	var types = {

test/reporters/mini.js

@@ -5,6 +5,10 @@ var AvaError = require('../../lib/ava-error');
 var _miniReporter = require('../../lib/reporters/mini');
 var beautifyStack = require('../../lib/beautify-stack');
 
+chalk.enabled = true;
+
+var graySpinner = chalk.gray.dim('⠋');
+
 function miniReporter() {
 	var reporter = _miniReporter();
 	reporter.start = function () {
@@ -18,7 +22,7 @@ process.stderr.setMaxListeners(50);
 test('start', function (t) {
 	var reporter = _miniReporter();
 
-	t.is(reporter.start(), ' \n ⠋ ');
+	t.is(reporter.start(), ' \n ' + graySpinner + ' ');
 	reporter.clearInterval();
 	t.end();
 });
@@ -32,7 +36,7 @@ test('passing test', function (t) {
 
 	var expectedOutput = [
 		' ',
-		' ⠋ ' + chalk.green('passed'),
+		' ' + graySpinner + ' passed',
 		'',
 		'   ' + chalk.green('1 passed')
 	].join('\n');
@@ -53,7 +57,7 @@ test('failing test', function (t) {
 
 	var expectedOutput = [
 		' ',
-		' ⠋ ' + chalk.red('failed'),
+		' ' + graySpinner + ' ' + chalk.red('failed'),
 		'',
 		'   ' + chalk.red('1 failed')
 	].join('\n');
@@ -76,7 +80,7 @@ test('passing test after failing', function (t) {
 
 	var expectedOutput = [
 		' ',
-		' ⠋ ' + chalk.green('passed'),
+		' ' + graySpinner + ' passed',
 		'',
 		'   ' + chalk.green('1 passed'),
 		'   ' + chalk.red('1 failed')
@@ -100,7 +104,7 @@ test('failing test after passing', function (t) {
 
 	var expectedOutput = [
 		' ',
-		' ⠋ ' + chalk.red('failed'),
+		' ' + graySpinner + ' ' + chalk.red('failed'),
 		'',
 		'   ' + chalk.green('1 passed'),
 		'   ' + chalk.red('1 failed')
@@ -120,7 +124,7 @@ test('skipped test', function (t) {
 
 	var expectedOutput = [
 		' ',
-		' ⠋ ' + chalk.yellow('skipped'),
+		' ' + graySpinner + ' skipped',
 		'',
 		'   ' + chalk.yellow('1 skipped')
 	].join('\n');
@@ -140,7 +144,7 @@ test('todo test', function (t) {
 
 	var expectedOutput = [
 		' ',
-		' ⠋ ' + chalk.blue('todo'),
+		' ' + graySpinner + ' todo',
 		'',
 		'   ' + chalk.blue('1 todo')
 	].join('\n');

test/reporters/verbose.js

@@ -5,6 +5,8 @@ var test = require('tap').test;
 var beautifyStack = require('../../lib/beautify-stack');
 var verboseReporter = require('../../lib/reporters/verbose');
 
+chalk.enabled = true;
+
 function createReporter() {
 	var reporter = verboseReporter();
 	reporter.api = {
@@ -98,7 +100,7 @@ test('skipped test', function (t) {
 		skip: true
 	});
 
-	var expectedOutput = '  ' + chalk.cyan('- skipped');
+	var expectedOutput = '  ' + chalk.yellow('- skipped');
 
 	t.is(actualOutput, expectedOutput);
 	t.end();