Fix timeouts so that stdout/stderr activity does not keep a test running

fixes #3373

Author: mdouglass

lib/api.js

@@ -221,7 +221,9 @@ export default class Api extends Emittery {
 				if (record.testFile && !timedOutWorkerFiles.has(record.testFile)) {
 					// Debounce the timer whenever there is activity from workers that
 					// haven't already timed out.
-					timeoutTrigger.debounce();
+					if (record.type !== 'worker-stderr' && record.type !== 'worker-stdout') {
+						timeoutTrigger.debounce();
+					}
 				}
 
 				if (failFast && (record.type === 'hook-failed' || record.type === 'test-failed' || record.type === 'worker-failed')) {

test/test-timeouts/fixtures/console-output.js

@@ -0,0 +1,9 @@
+import test from 'ava';
+
+test('timeout with console output', async t => {
+	t.timeout(5, 'timeout despite console output');
+	for (;;) {
+		console.log('sorry for the noise');
+		await new Promise(resolve => setTimeout(resolve, 1000));
+	}
+});

test/test-timeouts/snapshots/test.js.md

@@ -18,3 +18,9 @@ Generated by [AVA](https://avajs.dev).
         label: 'Called with:',
       },
     ]
+
+## timeouts are not blocked by console output
+
+> timeout despite console output
+
+    'timeout despite console output'

test/test-timeouts/snapshots/test.js.snap

@@ -14,3 +14,9 @@ test('timeout messages must be strings', async t => {
 	t.snapshot(error.message, 'error message');
 	t.snapshot(error.formattedDetails, 'formatted details');
 });
+
+test('timeouts are not blocked by console output', async t => {
+	const result = await t.throwsAsync(fixture(['console-output.js']));
+	const error = result.stats.getError(result.stats.failed[0]);
+	t.snapshot(error.message, 'timeout despite console output');
+});