fix(reporter)!: mark onTaskUpdate as @internal

Author: AriPerkkio

packages/vitest/src/node/types/reporter.ts

@@ -16,9 +16,7 @@ export interface Reporter {
    * @experimental
    */
   onBrowserInit?: (project: TestProject) => Awaitable<void>
-  /**
-   * @deprecated use `onTestModuleQueued`, `onTestModuleStart`, `onTestModuleEnd`, `onTestCaseReady`, `onTestCaseResult` instead
-   */
+  /** @internal   */
   onTaskUpdate?: (packs: TaskResultPack[], events: TaskEventPack[]) => Awaitable<void>
   onTestRemoved?: (trigger?: string) => Awaitable<void>
   onWatcherStart?: (files?: File[], errors?: unknown[]) => Awaitable<void>

test/browser/specs/runner.test.ts

@@ -36,7 +36,6 @@ describe('running browser tests', async () => {
           onTestRunStart: noop,
           onTestModuleCollected: noop,
           onTestRunEnd: noop,
-          onTaskUpdate: noop,
           onTestRemoved: noop,
           onWatcherStart: noop,
           onWatcherRerun: noop,

test/cli/test/public-api.test.ts

@@ -1,4 +1,4 @@
-import type { RunnerTaskResultPack } from 'vitest'
+import type { TaskMeta } from '@vitest/runner'
 import type { TestModule, TestUserConfig } from 'vitest/node'
 import { resolve } from 'pathe'
 import { expect, it } from 'vitest'
@@ -15,7 +15,9 @@ it.each([
     },
   },
 ] as TestUserConfig[])('passes down metadata when $name', { timeout: 60_000, retry: 1 }, async (config) => {
-  const taskUpdate: RunnerTaskResultPack[] = []
+  const finishedTestCaseMetas: TaskMeta[] = []
+  const finishedTestModuleMetas: TaskMeta[] = []
+
   const finishedTestModules: TestModule[] = []
   const collectedTestModules: TestModule[] = []
   const { ctx, stdout, stderr } = await runVitest({
@@ -24,8 +26,11 @@ it.each([
     reporters: [
       ['verbose', { isTTY: false }],
       {
-        onTaskUpdate(packs) {
-          taskUpdate.push(...packs.filter(i => i[1]?.state === 'pass'))
+        onTestCaseResult(testCase) {
+          finishedTestCaseMetas.push(testCase.meta())
+        },
+        onTestModuleEnd(testModule) {
+          finishedTestModuleMetas.push(testModule.meta())
         },
         onTestRunEnd(testModules) {
           finishedTestModules.push(...testModules)
@@ -46,27 +51,16 @@ it.each([
   const suiteMeta = { done: true }
   const testMeta = { custom: 'some-custom-hanlder' }
 
-  expect(taskUpdate).toHaveLength(4)
+  expect(finishedTestCaseMetas).toHaveLength(3)
+  expect(finishedTestModuleMetas).toHaveLength(1)
   expect(finishedTestModules).toHaveLength(1)
 
   const files = ctx?.state.getFiles() || []
   expect(files).toHaveLength(1)
 
-  expect(taskUpdate).toContainEqual(
-    [
-      expect.any(String),
-      expect.anything(),
-      suiteMeta,
-    ],
-  )
+  expect(finishedTestModuleMetas).toContainEqual(suiteMeta)
 
-  expect(taskUpdate).toContainEqual(
-    [
-      expect.any(String),
-      expect.anything(),
-      testMeta,
-    ],
-  )
+  expect(finishedTestCaseMetas).toContainEqual(testMeta)
 
   const test = finishedTestModules[0].children.tests().next().value!
 

test/reporters/tests/console.test.ts

@@ -7,7 +7,6 @@ import { runVitest } from '../../test-utils'
 
 class LogReporter extends DefaultReporter {
   isTTY = true
-  onTaskUpdate() {}
 }
 
 test('should print logs correctly', async () => {

test/reporters/tests/silent.test.ts

@@ -101,5 +101,4 @@ Log from failed suite`,
 
 class LogReporter extends DefaultReporter {
   isTTY = true
-  onTaskUpdate() {}
 }