refactor(runner): deprecate custom type in favour of "test"

packages/runner/src/types/tasks.ts

@@ -218,7 +218,13 @@ export interface Test<ExtraContext = object> extends TaskPopulated {
   context: TaskContext<Test> & ExtraContext & TestContext
 }
 
+/**
+ * @deprecated Use `Test` instead. `type: 'custom'` will be removed in Vitest 3.0
+ */
 export interface Custom<ExtraContext = object> extends TaskPopulated {
+  /**
+   * @deprecated use `test` instead. `custom` will be removed in Vitest 3.0
+   */
   type: 'custom'
   /**
    * Task context that will be passed to the test function.
@@ -228,6 +234,9 @@ export interface Custom<ExtraContext = object> extends TaskPopulated {
 
 export type Task = Test | Suite | Custom | File
 
+/**
+ * @deprecated Vitest doesn't provide `done()` anymore
+ */
 export type DoneCallback = (error?: any) => void
 export type TestFunction<ExtraContext = object> = (
   context: ExtendedContext<Test> & ExtraContext

packages/runner/src/utils/tasks.ts

@@ -1,20 +1,27 @@
 import type { Custom, Suite, Task, Test } from '../types/tasks'
 import { type Arrayable, toArray } from '@vitest/utils'
 
+/**
+ * @deprecated use `isTestCase` instead
+ */
 export function isAtomTest(s: Task): s is Test | Custom {
+  return isTestCase(s)
+}
+
+export function isTestCase(s: Task): s is Test | Custom {
   return s.type === 'test' || s.type === 'custom'
 }
 
 export function getTests(suite: Arrayable<Task>): (Test | Custom)[] {
   const tests: (Test | Custom)[] = []
   const arraySuites = toArray(suite)
   for (const s of arraySuites) {
-    if (isAtomTest(s)) {
+    if (isTestCase(s)) {
       tests.push(s)
     }
     else {
       for (const task of s.tasks) {
-        if (isAtomTest(task)) {
+        if (isTestCase(task)) {
           tests.push(task)
         }
         else {
@@ -31,7 +38,7 @@ export function getTests(suite: Arrayable<Task>): (Test | Custom)[] {
 
 export function getTasks(tasks: Arrayable<Task> = []): Task[] {
   return toArray(tasks).flatMap(s =>
-    isAtomTest(s) ? [s] : [s, ...getTasks(s.tasks)],
+    isTestCase(s) ? [s] : [s, ...getTasks(s.tasks)],
   )
 }
 
@@ -43,7 +50,7 @@ export function getSuites(suite: Arrayable<Task>): Suite[] {
 
 export function hasTests(suite: Arrayable<Suite>): boolean {
   return toArray(suite).some(s =>
-    s.tasks.some(c => isAtomTest(c) || hasTests(c)),
+    s.tasks.some(c => isTestCase(c) || hasTests(c)),
   )
 }