From a693f07eca1abc47a027c7d555b170d126b38a21 Mon Sep 17 00:00:00 2001 From: Marcel Date: Tue, 12 May 2026 09:36:47 +0200 Subject: [PATCH] refactor(test): convert TextLayerMock to class syntax in PdfViewer spec MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Prototype-style assignment was a vi.mock hoisting artifact from the old version of the file. Rest of the codebase uses class syntax — aligning. Addresses Felix Brandt round-4 suggestion on PR #536. Co-Authored-By: Claude Sonnet 4.6 --- .../src/lib/document/viewer/PdfViewer.svelte.spec.ts | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts b/frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts index 3a1eed5f..6067481d 100644 --- a/frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts +++ b/frontend/src/lib/document/viewer/PdfViewer.svelte.spec.ts @@ -7,9 +7,12 @@ import PdfViewer from './PdfViewer.svelte'; afterEach(cleanup); function makeFakePdfjsLib() { - function TextLayerMock() {} - TextLayerMock.prototype.render = () => Promise.resolve(); - TextLayerMock.prototype.cancel = () => {}; + class TextLayerMock { + render() { + return Promise.resolve(); + } + cancel() {} + } return { GlobalWorkerOptions: { workerSrc: '' },