Do not count writes to stdout/stderr as non-idling activity for timeouts

Fixes #3373. The idle guard will now force a test worker to exit even if it's continuously writing to stdout/stderr.

Co-authored-by: Mark Wubben <[email protected]>

Author: mdouglass

lib/api.js

@@ -218,9 +218,8 @@ export default class Api extends Emittery {
 			}
 
 			runStatus.on('stateChange', record => {
-				if (record.testFile && !timedOutWorkerFiles.has(record.testFile)) {
-					// Debounce the timer whenever there is activity from workers that
-					// haven't already timed out.
+				if (record.testFile && !timedOutWorkerFiles.has(record.testFile) && record.type !== 'worker-stderr' && record.type !== 'worker-stdout') {
+					// Debounce the timer whenever there is test-related activity from workers that haven't already timed out.
 					timeoutTrigger.debounce();
 				}
 

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

@@ -0,0 +1,13 @@
+import {setTimeout as delay} from 'node:timers/promises';
+import test from 'ava';
+
+test('timeout with console output', async t => {
+	t.timeout(1000, 'timeout despite console output');
+	for (let i = 0; await delay(100, true); i++) {
+		if (i % 2 === 0) {
+			console.log('stdout');
+		} else {
+			console.error('stderr');
+		}
+	}
+});

test/idle-timeouts/fixtures/package.json

@@ -0,0 +1,9 @@
+{
+	"type": "module",
+	"ava": {
+		"files": [
+			"*.js"
+		],
+		"timeout": "500ms"
+	}
+}

test/idle-timeouts/test.js

@@ -0,0 +1,9 @@
+import test from '@ava/test';
+
+import {fixture} from '../helpers/exec.js';
+
+test('idle 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.is(error.message, 'timeout despite console output');
+});