chore(deps): update dependency @antfu/eslint-config to v3 (#6435)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Vladimir Sheremet <sleuths.slews0s@icloud.com>

docs/api/expect.md

@@ -1141,9 +1141,10 @@ This assertion checks if a value satisfies a certain predicate.
 
 ```ts
 import { describe, expect, it } from 'vitest'
-describe('toSatisfy()', () => {
-  const isOdd = (value: number) => value % 2 !== 0
 
+const isOdd = (value: number) => value % 2 !== 0
+
+describe('toSatisfy()', () => {
   it('pass with 0', () => {
     expect(1).toSatisfy(isOdd)
   })

eslint.config.js

@@ -117,6 +117,14 @@ export default antfu(
       'no-restricted-globals': 'off',
     },
   },
+  {
+    files: [
+      `test/${GLOB_SRC}`,
+    ],
+    rules: {
+      'unicorn/consistent-function-scoping': 'off',
+    },
+  },
   {
     files: [`packages/vite-node/${GLOB_SRC}`],
     rules: {

examples/lit/test/basic.test.ts

@@ -3,10 +3,6 @@ import { beforeEach, describe, expect, it, vi } from 'vitest'
 import '../src/my-button.js'
 
 describe('Button with increment', async () => {
-  function getInsideButton(): HTMLElement | null | undefined {
-    return document.body.querySelector('my-button')?.shadowRoot?.querySelector('button')
-  }
-
   beforeEach(() => {
     document.body.innerHTML = '<my-button name="World"></my-button>'
   })
@@ -31,3 +27,7 @@ describe('Button with increment', async () => {
     expect(spyClick).toHaveBeenCalled()
   })
 })
+
+function getInsideButton(): HTMLElement | null | undefined {
+  return document.body.querySelector('my-button')?.shadowRoot?.querySelector('button')
+}

package.json

@@ -36,7 +36,7 @@
     "test:browser:playwright": "pnpm -C test/browser run test:playwright"
   },
   "devDependencies": {
-    "@antfu/eslint-config": "^2.25.1",
+    "@antfu/eslint-config": "^3.0.0",
     "@antfu/ni": "^0.22.1",
     "@playwright/test": "^1.46.0",
     "@rollup/plugin-commonjs": "^26.0.1",

packages/browser/src/client/tester/dialog.ts

@@ -9,8 +9,8 @@ If needed, mock the \`${name}\` call manually like:
 import { expect, vi } from "vitest"
 
 vi.spyOn(window, "${name}")${
-      defaultValue ? `.mockReturnValue(${JSON.stringify(defaultValue)})` : ''
-    }
+  defaultValue ? `.mockReturnValue(${JSON.stringify(defaultValue)})` : ''
+}
 ${name}(${formatedParams})
 expect(${name}).toHaveBeenCalledWith(${formatedParams})
 \`\`\``)

packages/browser/src/client/tester/logger.ts

@@ -20,49 +20,6 @@ export function setupConsoleLogSpy() {
     count,
     countReset,
   } = console
-  const formatInput = (input: unknown) => {
-    if (typeof input === 'object') {
-      return stringify(input, undefined, {
-        printBasicPrototype: false,
-        escapeString: false,
-      })
-    }
-    return format(input)
-  }
-  const processLog = (args: unknown[]) => args.map(formatInput).join(' ')
-  const sendLog = (
-    type: 'stdout' | 'stderr',
-    content: string,
-    disableStack?: boolean,
-  ) => {
-    if (content.startsWith('[vite]')) {
-      return
-    }
-    const unknownTestId = '__vitest__unknown_test__'
-    // @ts-expect-error untyped global
-    const taskId = globalThis.__vitest_worker__?.current?.id ?? unknownTestId
-    const origin
-      = getConfig().printConsoleTrace && !disableStack
-        ? new Error('STACK_TRACE').stack?.split('\n').slice(1).join('\n')
-        : undefined
-    rpc().sendLog({
-      origin,
-      content,
-      browser: true,
-      time: Date.now(),
-      taskId,
-      type,
-      size: content.length,
-    })
-  }
-  const stdout = (base: (...args: unknown[]) => void) => (...args: unknown[]) => {
-    base(...args)
-    sendLog('stdout', processLog(args))
-  }
-  const stderr = (base: (...args: unknown[]) => void) => (...args: unknown[]) => {
-    base(...args)
-    sendLog('stderr', processLog(args))
-  }
   console.log = stdout(log)
   console.debug = stdout(debug)
   console.info = stdout(info)
@@ -136,3 +93,56 @@ export function setupConsoleLogSpy() {
     countLabels[label] = 0
   }
 }
+
+function stdout(base: (...args: unknown[]) => void) {
+  return (...args: unknown[]) => {
+    base(...args)
+    sendLog('stdout', processLog(args))
+  }
+}
+function stderr(base: (...args: unknown[]) => void) {
+  return (...args: unknown[]) => {
+    base(...args)
+    sendLog('stderr', processLog(args))
+  }
+}
+
+function formatInput(input: unknown) {
+  if (typeof input === 'object') {
+    return stringify(input, undefined, {
+      printBasicPrototype: false,
+      escapeString: false,
+    })
+  }
+  return format(input)
+}
+
+function processLog(args: unknown[]) {
+  return args.map(formatInput).join(' ')
+}
+
+function sendLog(
+  type: 'stdout' | 'stderr',
+  content: string,
+  disableStack?: boolean,
+) {
+  if (content.startsWith('[vite]')) {
+    return
+  }
+  const unknownTestId = '__vitest__unknown_test__'
+  // @ts-expect-error untyped global
+  const taskId = globalThis.__vitest_worker__?.current?.id ?? unknownTestId
+  const origin
+    = getConfig().printConsoleTrace && !disableStack
+      ? new Error('STACK_TRACE').stack?.split('\n').slice(1).join('\n')
+      : undefined
+  rpc().sendLog({
+    origin,
+    content,
+    browser: true,
+    time: Date.now(),
+    taskId,
+    type,
+    size: content.length,
+  })
+}

packages/browser/src/client/tester/state.ts

@@ -63,10 +63,6 @@ function createCdp() {
 
   const listeners: Record<string, Function[]> = {}
 
-  const error = (err: unknown) => {
-    window.dispatchEvent(new ErrorEvent('error', { error: err }))
-  }
-
   const cdp = {
     send(method: string, params?: Record<string, any>) {
       return rpc().sendCdpEvent(contextId, method, params)
@@ -113,3 +109,7 @@ function createCdp() {
 
   return cdp
 }
+
+function error(err: unknown) {
+  window.dispatchEvent(new ErrorEvent('error', { error: err }))
+}

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

@@ -14,16 +14,6 @@ export const keyboard: UserEventCommand<(text: string, state: KeyboardState) =>
   text,
   state,
 ) => {
-  function focusIframe() {
-    if (
-      !document.activeElement
-      || document.activeElement.ownerDocument !== document
-      || document.activeElement === document.body
-    ) {
-      window.focus()
-    }
-  }
-
   if (context.provider instanceof PlaywrightBrowserProvider) {
     const frame = await context.frame()
     await frame.evaluate(focusIframe)
@@ -40,12 +30,6 @@ export const keyboard: UserEventCommand<(text: string, state: KeyboardState) =>
     context.contextId,
     text,
     async () => {
-      function selectAll() {
-        const element = document.activeElement as HTMLInputElement
-        if (element && element.select) {
-          element.select()
-        }
-      }
       if (context.provider instanceof PlaywrightBrowserProvider) {
         const frame = await context.frame()
         await frame.evaluate(selectAll)
@@ -165,3 +149,20 @@ export async function keyboardImplementation(
     pressed,
   }
 }
+
+function focusIframe() {
+  if (
+    !document.activeElement
+    || document.activeElement.ownerDocument !== document
+    || document.activeElement === document.body
+  ) {
+    window.focus()
+  }
+}
+
+function selectAll() {
+  const element = document.activeElement as HTMLInputElement
+  if (element && element.select) {
+    element.select()
+  }
+}

packages/browser/src/node/pool.ts

@@ -6,19 +6,19 @@ import { createDebugger } from 'vitest/node'
 
 const debug = createDebugger('vitest:browser:pool')
 
+async function waitForTests(
+  method: 'run' | 'collect',
+  contextId: string,
+  project: WorkspaceProject,
+  files: string[],
+) {
+  const context = project.browser!.state.createAsyncContext(method, contextId, files)
+  return await context
+}
+
 export function createBrowserPool(ctx: Vitest): ProcessPool {
   const providers = new Set<BrowserProvider>()
 
-  const waitForTests = async (
-    method: 'run' | 'collect',
-    contextId: string,
-    project: WorkspaceProject,
-    files: string[],
-  ) => {
-    const context = project.browser!.state.createAsyncContext(method, contextId, files)
-    return await context
-  }
-
   const executeTests = async (method: 'run' | 'collect', project: WorkspaceProject, files: string[]) => {
     ctx.state.clearFiles(project, files)
     const browser = project.browser!