From 2163cd9634deaa3ec7b203430040b08a1ad22725 Mon Sep 17 00:00:00 2001 From: Marcel Date: Tue, 12 May 2026 07:54:33 +0200 Subject: [PATCH] test(pdf-renderer): assert init() re-throws when libLoader rejects .catch(()=>{}) swallowed the rejection, so the test passed vacuously even if a future refactor silently caught the error. rejects.toThrow() proves the propagation contract holds before asserting pdfjsReady stays false. Co-Authored-By: Claude Sonnet 4.6 --- frontend/src/lib/document/viewer/usePdfRenderer.svelte.test.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frontend/src/lib/document/viewer/usePdfRenderer.svelte.test.ts b/frontend/src/lib/document/viewer/usePdfRenderer.svelte.test.ts index c6b40f81..a5647cde 100644 --- a/frontend/src/lib/document/viewer/usePdfRenderer.svelte.test.ts +++ b/frontend/src/lib/document/viewer/usePdfRenderer.svelte.test.ts @@ -83,7 +83,7 @@ describe('createPdfRenderer', () => { it('leaves pdfjsReady false when libLoader rejects', async () => { const failingLoader = vi.fn().mockRejectedValue(new Error('load failed')); const r = createPdfRenderer(failingLoader); - await r.init().catch(() => {}); + await expect(r.init()).rejects.toThrow('load failed'); expect(r.pdfjsReady).toBe(false); }); });