fix(reporter): end reason 'failed' on worker crash (#11568)

Author: AriPerkkio

packages/vitest/src/node/test-run.ts

@@ -176,7 +176,7 @@ export class TestRun {
       return !this.vitest.config.passWithNoTests
     }
 
-    return modules.some((m) => !m.ok())
+    return modules.some((m) => !m.ok() || m.state() === 'pending')
   }
 
   // make sure the error always has a "stacks" property

test/e2e/test/pool-worker-exit.test.ts

@@ -1,3 +1,4 @@
+import type { TestRunEndReason } from 'vitest/node'
 import { sep } from 'node:path'
 import { resolve } from 'pathe'
 import { expect, test } from 'vitest'
@@ -169,3 +170,24 @@ test('worker that fails to start surfaces the error instead of hanging', async (
     ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯"
   `)
 })
+
+test('run with a pending module after worker death is reported as failed', async () => {
+  let reason: TestRunEndReason | undefined
+
+  const { exitCode } = await runVitest({
+    root: './fixtures/pool-worker-exit',
+    pool: 'forks',
+    include: ['1-first.test.ts', '2-crash.test.ts'],
+    dangerouslyIgnoreUnhandledErrors: true,
+    reporters: [
+      {
+        onTestRunEnd(_modules, _errors, runEndReason) {
+          reason = runEndReason
+        },
+      },
+    ],
+  })
+
+  expect(reason).toBe('failed')
+  expect(exitCode).toBe(1)
+})