From 073b6cb45d67306852f85ffaec4f5785eac88d8b Mon Sep 17 00:00:00 2001 From: Marcel Date: Mon, 11 May 2026 22:49:34 +0200 Subject: [PATCH] refactor(test): move PdfViewer import to top and annotate partial fake cast MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - import PdfViewer left mid-file from vi.mock hoisting — no longer needed (Sara/Felix) - adds one-line comment explaining as unknown as cast is an intentional partial fake (Felix) Co-Authored-By: Claude Sonnet 4.6 --- frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts b/frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts index 0de1dcd7..39e61c45 100644 --- a/frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts +++ b/frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts @@ -2,6 +2,7 @@ import { vi, describe, it, expect, afterEach } from 'vitest'; import { cleanup, render } from 'vitest-browser-svelte'; import { page } from 'vitest/browser'; import type { createPdfRenderer } from '$lib/document/viewer/usePdfRenderer.svelte'; +import PdfViewer from './PdfViewer.svelte'; afterEach(cleanup); @@ -23,6 +24,7 @@ function makeFakePdfjsLib() { }) }), TextLayer: TextLayerMock + // Partial fake: only the methods used by usePdfRenderer are implemented } as unknown as typeof import('pdfjs-dist'); } @@ -31,8 +33,6 @@ function makeFakeLibLoader(): Parameters[0] { return vi.fn().mockResolvedValue([fakePdfjs, { default: '' }] as const); } -import PdfViewer from './PdfViewer.svelte'; - describe('PdfViewer', () => { it('shows previous and next page navigation buttons', async () => { render(PdfViewer, { url: '/api/documents/test-id/file', libLoader: makeFakeLibLoader() });