fix(e2e): improve types for fetch and mockLogger

Author: danielroe

src/core/mock.ts

@@ -1,16 +1,17 @@
 import { consola } from 'consola'
+import type { LogType } from 'consola'
 import { useTestContext } from './context'
 
 export function mockFn () {
   const ctx = useTestContext()
   return ctx.mockFn
 }
 
-export function mockLogger (): Record<string, Function> {
-  const mocks: any = {}
+export function mockLogger (): Record<LogType, (...args: any[]) => void> {
+  const mocks: Partial<Record<LogType, (...args: any[]) => void>> = {}
   consola.mockTypes((type) => {
     mocks[type] = mockFn()
-    return mocks[type]
+    return mocks[type]!
   })
-  return mocks
+  return mocks as Record<LogType, (...args: any[]) => void>
 }

src/core/server.ts

@@ -73,7 +73,7 @@ export async function stopServer () {
   }
 }
 
-export function fetch (path: string, options?: any) {
+export function fetch (path: string, options?: RequestInit) {
   return _fetch(url(path), options)
 }
 

src/core/types.ts

@@ -32,7 +32,7 @@ export interface TestContext {
   browser?: Browser
   url?: string
   serverProcess?: ExecaChildProcess
-  mockFn?: Function
+  mockFn?: (...args: any[]) => unknown
   /**
    * Functions to run on the vitest `afterAll` hook.
    * Useful for removing anything created during the test.