Improve use of for...of loop to enable / disable colors. Removed chalk and colors when throwing bable-config error

Author: Thomas

lib/babel-config.js

@@ -1,9 +1,7 @@
 'use strict';
 const path = require('path');
-const chalk = require('chalk');
 const figures = require('figures');
 const convertSourceMap = require('convert-source-map');
-const colors = require('./colors');
 
 function validate(conf) {
 	if (conf === undefined || conf === null) {
@@ -14,9 +12,9 @@ function validate(conf) {
 	const isValidShortcut = conf === 'default' || conf === 'inherit';
 
 	if (!conf || (typeof conf === 'string' && !isValidShortcut)) {
-		let message = colors.error(figures.cross);
+		let message = figures.cross;
 		message += ' Unexpected Babel configuration for AVA. ';
-		message += 'See ' + chalk.underline('https://github.com/avajs/ava#es2015-support') + ' for allowed values.';
+		message += 'See https://github.com/avajs/ava#es2015-support for allowed values.';
 
 		throw new Error(message);
 	}

lib/reporters/mini.js

@@ -14,11 +14,6 @@ const extractStack = require('../extract-stack');
 const codeExcerpt = require('../code-excerpt');
 const colors = require('../colors');
 
-chalk.enabled = true;
-Object.keys(colors).forEach(key => {
-	colors[key].enabled = true;
-});
-
 // TODO(@jamestalamge): This should be fixed in log-update and ansi-escapes once we are confident it's a good solution.
 const CSI = '\u001b[';
 const ERASE_LINE = CSI + '2K';

test/cli.js

@@ -17,7 +17,6 @@ const cliPath = path.join(__dirname, '../cli.js');
 
 // For some reason chalk is disabled by default
 chalk.enabled = true;
-const colors = require('../lib/colors');
 
 function execCli(args, opts, cb) {
 	let dirname;
@@ -75,8 +74,8 @@ test('disallow invalid babel config shortcuts', t => {
 		t.ok(err);
 
 		let expectedOutput = '\n  ';
-		expectedOutput += colors.error(figures.cross) + ' Unexpected Babel configuration for AVA.';
-		expectedOutput += ' See ' + chalk.underline('https://github.com/avajs/ava#es2015-support') + ' for allowed values.';
+		expectedOutput += figures.cross + ' Unexpected Babel configuration for AVA.';
+		expectedOutput += ' See https://github.com/avajs/ava#es2015-support for allowed values.';
 		expectedOutput += '\n';
 
 		t.is(stderr, expectedOutput);