From 3383f6650c90f1151bdd4013a9cce6b5f3e880de Mon Sep 17 00:00:00 2001 From: Marcel Date: Mon, 11 May 2026 17:51:24 +0200 Subject: [PATCH] test(admin): convert .not.toThrow into merge-success-banner-absent assertion (admin/tags/[id]) Co-Authored-By: Claude Opus 4.7 --- .../src/routes/admin/tags/[id]/page.svelte.test.ts | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/frontend/src/routes/admin/tags/[id]/page.svelte.test.ts b/frontend/src/routes/admin/tags/[id]/page.svelte.test.ts index c52053bd..48a2e64f 100644 --- a/frontend/src/routes/admin/tags/[id]/page.svelte.test.ts +++ b/frontend/src/routes/admin/tags/[id]/page.svelte.test.ts @@ -143,11 +143,12 @@ describe('admin/tags/[id] page', () => { expect(formSuccessBanner.length).toBe(0); }); - it('renders without throwing when mergeSuccess is null', async () => { - expect(() => - render(AdminTagEditPage, { - props: { data: baseData({ mergeSuccess: null }), form: undefined } - }) - ).not.toThrow(); + it('hides the merge-success banner when mergeSuccess is null', async () => { + render(AdminTagEditPage, { + props: { data: baseData({ mergeSuccess: null }), form: undefined } + }); + + // The merge-success banner has a distinct testid; mergeSuccess=null keeps it absent. + expect(document.querySelector('[data-testid="tag-merge-success"]')).toBeNull(); }); });