Support parallel runs that do not need to run test files

Author: micaelmbagira

lib/reporters/verbose.js

@@ -90,6 +90,7 @@ class VerboseReporter {
 		this.failFastEnabled = plan.failFastEnabled;
 		this.matching = plan.matching;
 		this.previousFailures = plan.previousFailures;
+		this.emptyParallelRun = plan.status.emptyParallelRun;
 
 		if (this.watching || plan.files.length > 1) {
 			this.prefixTitle = (testFile, title) => prefixTitle(plan.filePathPrefix, testFile, title);
@@ -320,6 +321,12 @@ class VerboseReporter {
 	}
 
 	endRun() { // eslint-disable-line complexity
+		if (this.emptyParallelRun) {
+			this.lineWriter.writeLine('No files tested in this parallel run');
+			this.lineWriter.writeLine();
+			return;
+		}
+
 		if (!this.stats) {
 			this.lineWriter.writeLine(colors.error(`${figures.cross} Couldn’t find any files to test`));
 			this.lineWriter.writeLine();

lib/run-status.js

@@ -8,6 +8,11 @@ class RunStatus extends Emittery {
 
 		this.pendingTests = new Map();
 
+		this.emptyParallelRun = parallelRuns &&
+			parallelRuns.currentFileCount === 0 &&
+			parallelRuns.totalRuns > 1 &&
+			files > 0;
+
 		this.stats = {
 			byFile: new Map(),
 			declaredTests: 0,
@@ -143,6 +148,10 @@ class RunStatus extends Emittery {
 	}
 
 	suggestExitCode(circumstances) {
+		if (this.emptyParallelRun) {
+			return 0;
+		}
+
 		if (circumstances.matching && this.stats.selectedTests === 0) {
 			return 1;
 		}

test-tap/fixture/parallel-runs/2.js renamed to test-tap/fixture/parallel-runs/less-files-than-ci-total/2.js

@@ -1,4 +1,4 @@
-const test = require('../../..');
+const test = require('../../../..');
 
 test('at expected index', t => {
 	t.is(process.env.CI_NODE_INDEX, '0');

test-tap/fixture/parallel-runs/less-files-than-ci-total/9.js

@@ -0,0 +1,5 @@
+const test = require('../../../..');
+
+test('at expected index', t => {
+	t.is(process.env.CI_NODE_INDEX, '1');
+});

test-tap/fixture/parallel-runs/package.json renamed to test-tap/fixture/parallel-runs/less-files-than-ci-total/package.json

@@ -1,4 +1,4 @@
-const test = require('../../..');
+const test = require('../../../..');
 
 test('at expected index', t => {
 	t.is(process.env.CI_NODE_INDEX, '1');

test-tap/fixture/parallel-runs/10.js renamed to test-tap/fixture/parallel-runs/more-files-than-ci-total/10.js

@@ -0,0 +1,5 @@
+const test = require('../../../..');
+
+test('at expected index', t => {
+	t.is(process.env.CI_NODE_INDEX, '0');
+});

test-tap/fixture/parallel-runs/more-files-than-ci-total/2.js

@@ -1,4 +1,4 @@
-const test = require('../../..');
+const test = require('../../../..');
 
 test('at expected index', t => {
 	t.is(process.env.CI_NODE_INDEX, '0');

test-tap/fixture/parallel-runs/2a.js renamed to test-tap/fixture/parallel-runs/more-files-than-ci-total/2a.js

@@ -1,4 +1,4 @@
-const test = require('../../..');
+const test = require('../../../..');
 
 test('at expected index', t => {
 	t.is(process.env.CI_NODE_INDEX, '0');

test-tap/fixture/parallel-runs/9.js renamed to test-tap/fixture/parallel-runs/more-files-than-ci-total/9.js

@@ -1,5 +1,5 @@
 /* eslint-disable unicorn/filename-case */
-const test = require('../../..');
+const test = require('../../../..');
 
 test('at expected index', t => {
 	t.is(process.env.CI_NODE_INDEX, '1');