From b0cf35cf0650e1ae65c11df95f8e54f9c9548d02 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Thu, 14 May 2026 15:24:27 +0200
Subject: [PATCH] fix(test): replace toBeAttached() with querySelector not-null
 check for spinner

toBeAttached() is not in the vitest-browser matcher set; toBeVisible() was
previously ruled out because the spinner is 0x0 px. Mirror the querySelector
pattern already used for the negative case in the same file.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../src/routes/admin/system/ImportStatusCard.svelte.test.ts   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/frontend/src/routes/admin/system/ImportStatusCard.svelte.test.ts b/frontend/src/routes/admin/system/ImportStatusCard.svelte.test.ts
index 13538b9b..b347f5ef 100644
--- a/frontend/src/routes/admin/system/ImportStatusCard.svelte.test.ts
+++ b/frontend/src/routes/admin/system/ImportStatusCard.svelte.test.ts
@@ -20,14 +20,14 @@ const makeStatus = (overrides: Partial<ImportStatus> = {}): ImportStatus => ({
 
 describe('ImportStatusCard', () => {
 	it('shows spinner while state is RUNNING', async () => {
-		const { getByTestId } = render(ImportStatusCard, {
+		render(ImportStatusCard, {
 			props: {
 				importStatus: makeStatus({ state: 'RUNNING', statusCode: 'IMPORT_RUNNING', processed: 3 }),
 				ontrigger: () => {}
 			}
 		});
 
-		await expect.element(getByTestId('spinner')).toBeAttached();
+		expect(document.querySelector('[data-testid="spinner"]')).not.toBeNull();
 	});
 
 	it('shows processed count at text-base while RUNNING', async () => {