Print out warning in verbose and mini reporter when --fail-fast is enabled

Author: Thomas

lib/colors.js

@@ -9,5 +9,6 @@ module.exports = {
 	pass: chalk.green,
 	duration: chalk.gray.dim,
 	errorStack: chalk.gray,
-	stack: chalk.red
+	stack: chalk.red,
+	failFast: chalk.magenta
 };

lib/reporters/mini.js

@@ -220,6 +220,10 @@ MiniReporter.prototype.finish = function (runStatus) {
 		});
 	}
 
+	if (runStatus.failFastEnabled === true) {
+		status += '\n\n  ' + colors.failFast('`--fail-fast` is on. Any number of tests may have been skipped');
+	}
+
 	return status + '\n\n';
 };
 

lib/reporters/verbose.js

@@ -114,6 +114,10 @@ VerboseReporter.prototype.finish = function (runStatus) {
 		});
 	}
 
+	if (runStatus.failFastEnabled === true) {
+		output += '\n\n\n  ' + colors.failFast('`--fail-fast` is on. Any number of tests may have been skipped');
+	}
+
 	return output + '\n';
 };
 

lib/run-status.js

@@ -28,6 +28,12 @@ function normalizeError(err) {
 	return err;
 }
 
+function isFailFastEnabled(stat) {
+	if (stat.failFastEnabled === true) {
+		return stat;
+	}
+}
+
 class RunStatus extends EventEmitter {
 	constructor(opts) {
 		super();
@@ -50,6 +56,7 @@ class RunStatus extends EventEmitter {
 		this.errors = [];
 		this.stats = [];
 		this.tests = [];
+		this.failFastEnabled = false;
 
 		autoBind(this);
 	}
@@ -138,6 +145,10 @@ class RunStatus extends EventEmitter {
 		this.skipCount = sum(this.stats, 'skipCount');
 		this.todoCount = sum(this.stats, 'todoCount');
 		this.failCount = sum(this.stats, 'failCount');
+		var failFastIsEnabled = this.stats.find(isFailFastEnabled);
+		if (failFastIsEnabled !== undefined && failFastIsEnabled.failFastEnabled === true) {
+			this.failFastEnabled = true;
+		}
 	}
 }
 

lib/runner.js

@@ -107,9 +107,14 @@ class Runner extends EventEmitter {
 		const stats = {
 			testCount: 0,
 			skipCount: 0,
-			todoCount: 0
+			todoCount: 0,
+			failFastEnabled: false
 		};
 
+		if (this._bail === true) {
+			stats.failFastEnabled = true;
+		}
+
 		this.results
 			.map(result => {
 				return result.result;

test/reporters/mini.js

@@ -383,6 +383,21 @@ test('results with errors', function (t) {
 	t.end();
 });
 
+test('results when fail-fast is enabled', function (t) {
+	var reporter = miniReporter();
+	var runStatus = {
+		failFastEnabled: true
+	};
+
+	var output = reporter.finish(runStatus);
+	compareLineOutput(t, output, [
+		'',
+		'',
+		'  ' + colors.failFast('`--fail-fast` is on. Any number of tests may have been skipped')
+	]);
+	t.end();
+});
+
 test('results with 1 previous failure', function (t) {
 	var reporter = miniReporter();
 	reporter.todoCount = 1;

test/reporters/verbose.js

@@ -381,6 +381,29 @@ test('results with errors', function (t) {
 	t.end();
 });
 
+test('results when fail-fast is enabled', function (t) {
+	var reporter = verboseReporter();
+	var runStatus = createRunStatus();
+	runStatus.failCount = 1;
+	runStatus.failFastEnabled = true;
+	runStatus.tests = [{
+		title: 'failed test'
+	}];
+
+	var output = reporter.finish(runStatus);
+	var expectedOutput = [
+		'',
+		'  ' + chalk.red('1 test failed') + time,
+		'',
+		'',
+		'  ' + colors.failFast('`--fail-fast` is on. Any number of tests may have been skipped'),
+		''
+	].join('\n');
+
+	t.is(output, expectedOutput);
+	t.end();
+});
+
 test('results with 1 previous failure', function (t) {
 	var reporter = createReporter();
 

test/runner.js

@@ -659,3 +659,19 @@ test('match applies to arrays of macros', t => {
 		t.end();
 	});
 });
+
+test('stats.failFastEnabled is false when bail is not set', t => {
+	const runner = new Runner();
+	var stats = runner._buildStats();
+
+	t.is(stats.failFastEnabled, false);
+	t.end();
+});
+
+test('stats.failFastEnabled is true when bail is set', t => {
+	const runner = new Runner({bail: true});
+	var stats = runner._buildStats();
+
+	t.is(stats.failFastEnabled, true);
+	t.end();
+});