Update mini reporter to include files-without-tests in final output

Author: novemberborn

lib/reporters/mini.js

@@ -103,6 +103,7 @@ class MiniReporter {
 		this.failFastEnabled = false;
 		this.failures = [];
 		this.filesWithMissingAvaImports = new Set();
+		this.filesWithoutDeclaredTests = new Set();
 		this.internalErrors = [];
 		this.knownFailures = [];
 		this.matching = false;
@@ -183,10 +184,15 @@ class MiniReporter {
 				this.unhandledRejections.push(evt);
 				break;
 			case 'worker-failed':
-				// Ignore
+				if (this.stats.byFile.get(evt.testFile).declaredTests === 0) {
+					this.filesWithoutDeclaredTests.add(evt.testFile);
+				}
 				break;
 			case 'worker-finished':
-				// Ignore
+				if (this.stats.byFile.get(evt.testFile).declaredTests === 0) {
+					this.filesWithoutDeclaredTests.add(evt.testFile);
+					this.writeWithCounts(colors.error(`${figures.cross} No tests found in ${path.relative('.', evt.testFile)}`));
+				}
 				break;
 			case 'worker-stderr':
 			case 'worker-stdout':
@@ -353,6 +359,26 @@ class MiniReporter {
 			' ' + chalk.gray.dim('[' + new Date().toLocaleTimeString('en-US', {hour12: false}) + ']') :
 			'';
 
+		if (this.filesWithMissingAvaImports.size > 0) {
+			for (const testFile of this.filesWithMissingAvaImports) {
+				this.lineWriter.writeLine(colors.error(`${figures.cross} No tests found in ${path.relative('.', testFile)}, make sure to import "ava" at the top of your test file`) + firstLinePostfix);
+				firstLinePostfix = '';
+			}
+		}
+
+		if (this.filesWithoutDeclaredTests.size > 0) {
+			for (const testFile of this.filesWithoutDeclaredTests) {
+				if (!this.filesWithMissingAvaImports.has(testFile)) {
+					this.lineWriter.writeLine(colors.error(`${figures.cross} No tests found in ${path.relative('.', testFile)}`) + firstLinePostfix);
+					firstLinePostfix = '';
+				}
+			}
+		}
+
+		if (this.filesWithMissingAvaImports.size > 0 || this.filesWithoutDeclaredTests.size > 0) {
+			this.lineWriter.writeLine();
+		}
+
 		if (this.stats.failedHooks > 0) {
 			this.lineWriter.writeLine(colors.error(`${this.stats.failedHooks} ${plur('hook', this.stats.failedHooks)} failed`) + firstLinePostfix);
 			firstLinePostfix = '';

test/reporters/mini.edgecases.log

@@ -3,9 +3,16 @@
 ---tty-stream-chunk-separator
 * ---tty-stream-chunk-separator
 ---tty-stream-chunk-separator
+* ✖ No tests found in test/fixture/report/edge-cases/ava-import-no-test-declaration.js---tty-stream-chunk-separator
+---tty-stream-chunk-separator
 * ✖ No tests found in test/fixture/report/edge-cases/no-ava-import.js, make sure to import "ava" at the top of your test file---tty-stream-chunk-separator
 ---tty-stream-chunk-separator
 [?25h
+  ✖ No tests found in test/fixture/report/edge-cases/no-ava-import.js, make sure to import "ava" at the top of your test file
+  ✖ No tests found in test/fixture/report/edge-cases/ava-import-no-test-declaration.js
+  ✖ No tests found in test/fixture/report/edge-cases/throws.js
+  ✖ No tests found in test/fixture/report/edge-cases/import-and-use-test-member.js
+
   2 uncaught exceptions
 
   Uncaught exception in test/fixture/report/edge-cases/throws.js

test/reporters/mini.regular.log

@@ -111,6 +111,8 @@ stderr
   1 todo---tty-stream-chunk-separator
 ---tty-stream-chunk-separator
 [?25h
+  ✖ No tests found in test/fixture/report/regular/bad-test-chain.js
+
   8 tests failed
   1 known failure
   1 test skipped

test/reporters/mini.typescript.log

@@ -4,6 +4,8 @@
 * ---tty-stream-chunk-separator
 ---tty-stream-chunk-separator
 [?25h
+  ✖ No tests found in test/fixture/report/typescript/build-error.ts
+
   1 uncaught exception
 
   Uncaught exception in test/fixture/report/typescript/build-error.ts