From e6d118ccb36fffe0eb24b5c89716a3753740e3a8 Mon Sep 17 00:00:00 2001 From: Jonas Date: Fri, 25 Sep 2026 16:27:11 +0200 Subject: [PATCH] test(vitest): move `vi.mock()` to top of file Vitest 5 refuses a `vi.mock()` call nested in a describe or it block. Signed-off-by: Jonas Assisted-by: OpenCode:claude-fable-5.1 --- src/tests/services/SyncService.spec.ts | 3 ++- src/tests/services/WebsocketPolyfill.spec.ts | 15 ++++++++------- 2 files changed, 10 insertions(+), 8 deletions(-) diff --git a/src/tests/services/SyncService.spec.ts b/src/tests/services/SyncService.spec.ts index 2707f1fd0f5..d76df39243a 100644 --- a/src/tests/services/SyncService.spec.ts +++ b/src/tests/services/SyncService.spec.ts @@ -8,6 +8,8 @@ import * as connect from '../../apis/connect.ts' import { provideConnection } from '../../composables/useConnection.js' import { SyncService } from '../../services/SyncService.js' +vi.mock('../../apis/connect') + const connection = { documentId: 123, sessionId: 345, @@ -53,7 +55,6 @@ describe('Sync service', () => { getBaseVersionEtag, setBaseVersionEtag, ) - vi.mock('../../apis/connect') vi.mocked(connect.open).mockResolvedValue(openResult) const openHandler = vi.fn() const service = new SyncService({ connection, openConnection }) diff --git a/src/tests/services/WebsocketPolyfill.spec.ts b/src/tests/services/WebsocketPolyfill.spec.ts index 1cecd225ce6..1f61490130b 100644 --- a/src/tests/services/WebsocketPolyfill.spec.ts +++ b/src/tests/services/WebsocketPolyfill.spec.ts @@ -8,14 +8,15 @@ import { shallowRef } from 'vue' import { SyncService } from '../../services/SyncService.ts' import initWebSocketPolyfill from '../../services/WebSocketPolyfill.js' +vi.mock(import('../../services/SyncService.ts'), () => { + const SyncService = vi.fn() + SyncService.prototype.bus = { on: vi.fn() } + SyncService.prototype.open = vi.fn().mockImplementation(async () => ({})) + SyncService.prototype.hasActiveConnection = vi.fn() + return { SyncService } +}) + describe('Init function', () => { - vi.mock(import('../../services/SyncService.ts'), () => { - const SyncService = vi.fn() - SyncService.prototype.bus = { on: vi.fn() } - SyncService.prototype.open = vi.fn().mockImplementation(async () => ({})) - SyncService.prototype.hasActiveConnection = vi.fn() - return { SyncService } - }) const mockSyncService = () => new SyncService({ connection: shallowRef(undefined), openConnection: vi.fn(),