From d8bd0c3577b7f9aebf76a6aac5e276562f6e3ec4 Mon Sep 17 00:00:00 2001
From: Marcel <marcel@familienarchiv>
Date: Sun, 10 May 2026 01:48:29 +0200
Subject: [PATCH] test(document): cover DocumentRow branches

Title rendering with originalFilename fallback, sender vs unknown
placeholder, tag buttons per document tag, bulk-select checkbox gated
on canWrite, archive chips visibility, snippet/summary visibility,
em-dash for missing date.

11 tests covering ~30 of the row's branches.

Refs #496.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../lib/document/DocumentRow.svelte.test.ts   | 150 ++++++++++++++++++
 1 file changed, 150 insertions(+)
 create mode 100644 frontend/src/lib/document/DocumentRow.svelte.test.ts

diff --git a/frontend/src/lib/document/DocumentRow.svelte.test.ts b/frontend/src/lib/document/DocumentRow.svelte.test.ts
new file mode 100644
index 00000000..704428bc
--- /dev/null
+++ b/frontend/src/lib/document/DocumentRow.svelte.test.ts
@@ -0,0 +1,150 @@
+import { describe, it, expect, vi, afterEach } from 'vitest';
+import { cleanup, render } from 'vitest-browser-svelte';
+import { page } from 'vitest/browser';
+
+vi.mock('$app/navigation', () => ({
+	beforeNavigate: () => {},
+	afterNavigate: () => {},
+	goto: vi.fn(),
+	invalidate: vi.fn(),
+	invalidateAll: vi.fn(),
+	preloadCode: vi.fn(),
+	preloadData: vi.fn(),
+	pushState: vi.fn(),
+	replaceState: vi.fn(),
+	disableScrollHandling: vi.fn(),
+	onNavigate: () => () => {}
+}));
+
+const { default: DocumentRow } = await import('./DocumentRow.svelte');
+
+afterEach(cleanup);
+
+const sender = { id: 's1', displayName: 'Anna Schmidt' };
+const receiver = { id: 'r1', displayName: 'Bert Meier' };
+
+const makeDoc = (overrides: Record<string, unknown> = {}) => ({
+	id: 'd1',
+	title: 'Brief 1923',
+	originalFilename: 'b.pdf',
+	documentDate: '1923-04-15',
+	sender,
+	receivers: [receiver],
+	tags: [],
+	thumbnailUrl: null,
+	contentType: 'application/pdf',
+	summary: null,
+	archiveBox: null,
+	archiveFolder: null,
+	location: null,
+	...overrides
+});
+
+const baseItem = (docOverrides: Record<string, unknown> = {}) => ({
+	document: makeDoc(docOverrides),
+	matchData: null,
+	completionPercentage: 0,
+	contributors: []
+});
+
+describe('DocumentRow', () => {
+	it('renders the title', async () => {
+		render(DocumentRow, { props: { item: baseItem() } });
+
+		await expect
+			.element(page.getByRole('heading', { level: 3, name: /brief 1923/i }))
+			.toBeVisible();
+	});
+
+	it('falls back to originalFilename when title is null', async () => {
+		render(DocumentRow, { props: { item: baseItem({ title: null }) } });
+
+		await expect.element(page.getByRole('heading', { level: 3, name: /b\.pdf/i })).toBeVisible();
+	});
+
+	it('renders the sender name in the metadata column', async () => {
+		render(DocumentRow, { props: { item: baseItem() } });
+
+		await expect.element(page.getByText('Anna Schmidt')).toBeVisible();
+	});
+
+	it('renders the unknown placeholder when sender is null', async () => {
+		render(DocumentRow, { props: { item: baseItem({ sender: null }) } });
+
+		const unknownTexts = document.querySelectorAll('.italic');
+		const hasUnknown = Array.from(unknownTexts).some((el) => el.textContent?.includes('Unbekannt'));
+		expect(hasUnknown).toBe(true);
+	});
+
+	it('renders one tag button per document tag', async () => {
+		render(DocumentRow, {
+			props: {
+				item: baseItem({
+					tags: [
+						{ id: 't1', name: 'Familie', color: null },
+						{ id: 't2', name: 'Reise', color: '#ffaabb' }
+					]
+				})
+			}
+		});
+
+		await expect.element(page.getByRole('button', { name: 'Familie' })).toBeVisible();
+		await expect.element(page.getByRole('button', { name: 'Reise' })).toBeVisible();
+	});
+
+	it('renders the bulk-select checkbox when canWrite is true', async () => {
+		render(DocumentRow, { props: { item: baseItem(), canWrite: true } });
+
+		const checkbox = document.querySelector('input[type="checkbox"]');
+		expect(checkbox).not.toBeNull();
+	});
+
+	it('hides the bulk-select checkbox when canWrite is false', async () => {
+		render(DocumentRow, { props: { item: baseItem(), canWrite: false } });
+
+		const checkbox = document.querySelector('input[type="checkbox"]');
+		expect(checkbox).toBeNull();
+	});
+
+	it('renders archive chips when archive metadata is present', async () => {
+		render(DocumentRow, {
+			props: {
+				item: baseItem({ archiveBox: 'Box 1', archiveFolder: 'Mappe A', location: 'Berlin' })
+			}
+		});
+
+		await expect.element(page.getByText('Box 1')).toBeVisible();
+		await expect.element(page.getByText('Mappe A')).toBeVisible();
+		await expect.element(page.getByText('Berlin')).toBeVisible();
+	});
+
+	it('renders the snippet when matchData provides a transcriptionSnippet', async () => {
+		render(DocumentRow, {
+			props: {
+				item: {
+					document: makeDoc(),
+					matchData: { transcriptionSnippet: 'Hello world snippet' },
+					completionPercentage: 50,
+					contributors: []
+				}
+			}
+		});
+
+		await expect.element(page.getByTestId('search-snippet')).toBeVisible();
+	});
+
+	it('renders the summary when present', async () => {
+		render(DocumentRow, {
+			props: { item: baseItem({ summary: 'Brief über die Reise nach Berlin' }) }
+		});
+
+		await expect.element(page.getByTestId('doc-summary')).toBeVisible();
+	});
+
+	it('renders an em-dash for missing documentDate', async () => {
+		render(DocumentRow, { props: { item: baseItem({ documentDate: null }) } });
+
+		// Multiple em-dashes possible; just ensure at least one is rendered
+		expect(document.body.textContent).toContain('—');
+	});
+});