fix!: remove deprecated APIs (#8428)

Author: sheremet-va

docs/guide/browser/config.md

@@ -295,19 +295,6 @@ export interface BrowserScript {
 }
 ```
 
-## browser.testerScripts
-
-- **Type:** `BrowserScript[]`
-- **Default:** `[]`
-
-::: danger DEPRECATED
-This API is deprecated an will be removed in Vitest 4. Please, use [`browser.testerHtmlPath`](#browser-testerhtmlpath) field instead.
-:::
-
-Custom scripts that should be injected into the tester HTML before the tests environment is initiated. This is useful to inject polyfills required for Vitest browser implementation. It is recommended to use [`setupFiles`](#setupfiles) in almost all cases instead of this.
-
-The script `src` and `content` will be processed by Vite plugins.
-
 ## browser.commands
 
 - **Type:** `Record<string, BrowserCommand>`

docs/guide/migration.md

@@ -237,6 +237,7 @@ Vitest 4.0 removes some deprecated APIs, including:
 - `environmentMatchGlobs` config option. Use [`projects`](/guide/projects) instead.
 - Reporter APIs `onCollected`, `onSpecsCollected`, `onPathsCollected`, `onTaskUpdate` and `onFinished`. See [`Reporters API`](/advanced/api/reporters) for new alternatives. These APIs were introduced in Vitest `v3.0.0`.
 - `deps.external`, `deps.inline`, `deps.fallbackCJS` config options. Use `server.deps.external`, `server.deps.inline`, or `server.deps.fallbackCJS` instead.
+- `browser.testerScripts` config option. Use [`browser.testerHtmlPath`](/guide/browser/config#browser-testerhtmlpath) instead.
 
 This release also removes all deprecated types. This finally fixes an issue where Vitest accidentally pulled in `@types/node` (see [#5481](https://github.com/vitest-dev/vitest/issues/5481) and [#6141](https://github.com/vitest-dev/vitest/issues/6141)).
 

packages/browser/src/node/commands/fs.ts

@@ -8,7 +8,7 @@ import { isFileServingAllowed } from 'vitest/node'
 function assertFileAccess(path: string, project: TestProject) {
   if (
     !isFileServingAllowed(path, project.vite)
-    && !isFileServingAllowed(path, project.vitest.server)
+    && !isFileServingAllowed(path, project.vitest.vite)
   ) {
     throw new Error(
       `Access denied to "${path}". See Vite config documentation for "server.fs": https://vitejs.dev/config/server-options.html#server-fs-strict.`,

packages/browser/src/node/plugin.ts

@@ -460,12 +460,6 @@ export default (parentServer: ParentBrowserProject, base = '/'): Plugin[] => {
           return
         }
 
-        if (!parentServer.testerScripts) {
-          const testerScripts = await parentServer.formatScripts(
-            parentServer.config.browser.testerScripts,
-          )
-          parentServer.testerScripts = testerScripts
-        }
         const stateJs = typeof parentServer.stateJs === 'string'
           ? parentServer.stateJs
           : await parentServer.stateJs
@@ -562,7 +556,6 @@ body {
                 injectTo: 'head',
               } as const
             : null,
-          ...parentServer.testerScripts,
           ...testerTags,
         ].filter(s => s != null)
       },

packages/browser/src/node/projectParent.ts

@@ -23,7 +23,6 @@ import { slash } from './utils'
 
 export class ParentBrowserProject {
   public orchestratorScripts: string | undefined
-  public testerScripts: HtmlTagDescriptor[] | undefined
 
   public faviconUrl: string
   public prefixOrchestratorUrl: string

packages/runner/src/utils/index.ts

@@ -19,6 +19,5 @@ export {
   getTests,
   hasFailed,
   hasTests,
-  isAtomTest,
   isTestCase,
 } from './tasks'

packages/runner/src/utils/tasks.ts

@@ -2,13 +2,6 @@ import type { Arrayable } from '@vitest/utils'
 import type { Suite, Task, Test } from '../types/tasks'
 import { toArray } from '@vitest/utils'
 
-/**
- * @deprecated use `isTestCase` instead
- */
-export function isAtomTest(s: Task): s is Test {
-  return isTestCase(s)
-}
-
 export function isTestCase(s: Task): s is Test {
   return s.type === 'test'
 }

packages/ui/client/composables/explorer/utils.ts

@@ -6,7 +6,7 @@ import type {
   TestTreeNode,
   UITaskTreeNode,
 } from '~/composables/explorer/types'
-import { isAtomTest } from '@vitest/runner/utils'
+import { isTestCase } from '@vitest/runner/utils'
 import { client } from '~/composables/client'
 import { explorerTree } from '~/composables/explorer/index'
 import { openedTreeItemsSet } from '~/composables/explorer/state'
@@ -117,7 +117,7 @@ export function createOrUpdateNodeTask(id: string) {
 
   const task = client.state.idMap.get(id)
   // if it is not a test just return
-  if (!task || !isAtomTest(task)) {
+  if (!task || !isTestCase(task)) {
     return
   }
 
@@ -148,7 +148,7 @@ export function createOrUpdateNode(
       taskNode.state = task.result?.state
     }
     else {
-      if (isAtomTest(task)) {
+      if (isTestCase(task)) {
         taskNode = {
           id: task.id,
           fileId: task.file.id,

packages/vitest/src/api/setup.ts

@@ -29,7 +29,7 @@ export function setup(ctx: Vitest, _server?: ViteDevServer): void {
 
   const clients = new Map<WebSocket, WebSocketRPC>()
 
-  const server = _server || ctx.server
+  const server = _server || ctx.vite
 
   server.httpServer?.on('upgrade', (request: IncomingMessage, socket, head) => {
     if (!request.url) {

packages/vitest/src/node/ast-collect.ts

@@ -197,7 +197,7 @@ export function createFailedFileTask(project: TestProject, filepath: string, err
     tasks: [],
     start: 0,
     end: 0,
-    projectName: project.getName(),
+    projectName: project.name,
     meta: {},
     pool: project.browser ? 'browser' : project.config.pool,
     file: null!,