feat: introduce the new reporter API

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

@@ -71,9 +71,6 @@ export class TestRun {
             this.queuedTestModules.add(entity.id)
             queuedTestModules.push(entity)
           }
-
-          // TODO: Check if skipped TestCases of finished module were reported.
-          // If not, get TestCases here and push them to TestCase arrays+sets.
         }
 
         if (state !== 'pending' && state !== 'queued' && !this.finishedTestModules.has(entity.id)) {
@@ -90,6 +87,14 @@ export class TestRun {
             this.runningTestModules.add(entity.id)
             runningTestModules.push(entity)
           }
+
+          // Skipped tests need to be reported manually once test module has finished
+          for (const test of entity.children.tests()) {
+            if (!this.finishedTestCases.has(test.id)) {
+              this.finishedTestCases.add(test.id)
+              finishedTestCases.push(test)
+            }
+          }
         }
       }
 
@@ -118,7 +123,8 @@ export class TestRun {
     await Promise.all(finishedTestCases.map(testCase => this.vitest.report('onTestCaseFinished', testCase)))
     await Promise.all(finishedTestModules.map(module => this.vitest.report('onTestModuleFinished', module)))
 
-    await Promise.all(queuedTestModules.map(module => this.vitest.report('onTestModuleQueued', module)))
+    // TODO: Not needed as RPC has onQueued already. Do we want to remove that and centralize resolving here instead?
+    // await Promise.all(queuedTestModules.map(module => this.vitest.report('onTestModuleQueued', module)))
     await Promise.all(runningTestModules.map(module => this.vitest.report('onTestModulePrepare', module)))
     await Promise.all(runningTestCases.map(testCase => this.vitest.report('onTestCasePrepare', testCase)))
   }

test/reporters/tests/test-run.test.ts

@@ -12,17 +12,16 @@
     config: false,
     include: ['./fixtures/task-parser-tests/*.test.ts'],
     fileParallelism: false,
     reporters: [reporter],
     sequence: { sequencer: Sorter },
   })
 
   expect(stdout).toBe('')
   expect(stderr).toBe('')
 
-  // TODO: Missing skipped tests, missing hooks, queued is duplicated
+  // TODO: Missing hooks, queued is duplicated
   expect(reporter.calls).toMatchInlineSnapshot(`
     [
-      "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| queued",
       "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| queued",
       "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| start",
       "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| RUN some test",
@@ -33,9 +32,9 @@
       "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| RUN parallel slow tests 1.2",
       "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| DONE parallel slow tests 1.1",
       "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| DONE parallel slow tests 1.2",
+      "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| DONE Skipped test 1",
       "|<process-cwd>/fixtures/task-parser-tests/example-1.test.ts| finish",
       "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| queued",
-      "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| queued",
       "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| start",
       "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| RUN some test",
       "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| DONE some test",
@@ -45,6 +44,7 @@
       "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| RUN parallel slow tests 2.2",
       "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| DONE parallel slow tests 2.1",
       "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| DONE parallel slow tests 2.2",
+      "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| DONE Skipped test 1",
       "|<process-cwd>/fixtures/task-parser-tests/example-2.test.ts| finish",
     ]
   `)