Don't report filtered tests as pending

Author: vancouverwill

lib/run-status.js

@@ -59,7 +59,6 @@ class RunStatus extends Emittery {
 			case 'declared-test':
 				stats.declaredTests++;
 				fileStats.declaredTests++;
-				this.addPendingTest(event);
 				break;
 			case 'hook-failed':
 				stats.failedHooks++;
@@ -84,6 +83,7 @@ class RunStatus extends Emittery {
 				} else {
 					stats.remainingTests++;
 					fileStats.remainingTests++;
+					this.addPendingTest(event);
 				}
 
 				break;

test/fixture/report/timeoutwithmatch/a.js

@@ -0,0 +1,15 @@
+import test from '../../../..';
+
+test('passes needle', t => t.pass());
+
+test.cb('slow needle', t => {
+	setTimeout(t.end, 15000);
+});
+test.cb('slow two', t => {
+	setTimeout(t.end, 15000);
+});
+test.cb('slow three needle', t => {
+	setTimeout(t.end, 15000);
+});
+
+test('passes two', t => t.pass());

test/fixture/report/timeoutwithmatch/package.json

@@ -0,0 +1 @@
+{}

test/helper/report.js

@@ -81,7 +81,7 @@ exports.sanitizers = {
 	version: str => replaceString(str, `v${pkg.version}`, 'v1.0.0-beta.5.1')
 };
 
-const run = (type, reporter) => {
+const run = (type, reporter, match = []) => {
 	const projectDir = path.join(__dirname, '../fixture/report', type.toLowerCase());
 
 	const options = {
@@ -96,7 +96,7 @@ const run = (type, reporter) => {
 		require: [],
 		cacheEnabled: true,
 		compileEnhancements: true,
-		match: [],
+		match,
 		babelConfig: {testOptions: {}},
 		resolveTestsFrom: projectDir,
 		projectDir,
@@ -144,6 +144,7 @@ exports.failFast2 = reporter => run('failFast2', reporter);
 exports.only = reporter => run('only', reporter);
 exports.timeoutInSingleFile = reporter => run('timeoutInSingleFile', reporter);
 exports.timeoutInMultipleFiles = reporter => run('timeoutInMultipleFiles', reporter);
+exports.timeoutWithMatch = reporter => run('timeoutWithMatch', reporter, ['*needle*']);
 exports.watch = reporter => run('watch', reporter);
 exports.typescript = reporter => run('typescript', reporter);
 exports.edgeCases = reporter => run('edge-cases', reporter);

test/reporters/verbose.js

@@ -44,5 +44,6 @@ test('verbose reporter - timeout', t => {
 
 	t.test('single file run', run('timeoutInSingleFile'));
 	t.test('multiple files run', run('timeoutInMultipleFiles'));
+	t.test('single file with only certain tests matched run', run('timeoutWithMatch'));
 	t.end();
 });

test/reporters/verbose.timeoutwithmatch.log

@@ -0,0 +1,17 @@
+
+---tty-stream-chunk-separator
+  ✔ passes needle
+---tty-stream-chunk-separator
+  
+  ✖ Timed out while running tests
+
+  2 tests were pending in ~/test/fixture/report/timeoutwithmatch/a.js
+
+  ◌ slow needle
+  ◌ slow three needle
+
+---tty-stream-chunk-separator
+
+  1 test passed
+
+---tty-stream-chunk-separator