+ {#if showRightColumn}
+
+ {#if data.canWrite}
+
+ {/if}
+
+
+
+
+
+ {/if}
-
-
-
-
-
-
+
{:else}
diff --git a/frontend/src/routes/admin/groups/layout.svelte.spec.ts b/frontend/src/routes/admin/groups/layout.svelte.spec.ts
index 05c48a81..68a35cf5 100644
--- a/frontend/src/routes/admin/groups/layout.svelte.spec.ts
+++ b/frontend/src/routes/admin/groups/layout.svelte.spec.ts
@@ -109,10 +109,12 @@ describe('GroupsListPanel — collapse toggle', () => {
});
it('persists collapse state using the groups-specific localStorage key', async () => {
- const setSpy = vi.spyOn(Storage.prototype, 'setItem');
render(GroupsListPanel, { groups });
+ const setSpy = vi.spyOn(Storage.prototype, 'setItem');
document.querySelector
('[aria-label="Liste einklappen"]')!.click();
- expect(setSpy).toHaveBeenCalledWith('admin_groups_list_collapsed', 'true');
+ await vi.waitFor(() =>
+ expect(setSpy).toHaveBeenCalledWith('admin_groups_list_collapsed', 'true')
+ );
setSpy.mockRestore();
});
});
diff --git a/frontend/src/routes/admin/tags/layout.svelte.spec.ts b/frontend/src/routes/admin/tags/layout.svelte.spec.ts
index 8bf7eb61..c3edbbad 100644
--- a/frontend/src/routes/admin/tags/layout.svelte.spec.ts
+++ b/frontend/src/routes/admin/tags/layout.svelte.spec.ts
@@ -88,10 +88,12 @@ describe('TagsListPanel — collapse toggle', () => {
});
it('persists collapse state using the tags-specific localStorage key', async () => {
- const setSpy = vi.spyOn(Storage.prototype, 'setItem');
render(TagsListPanel, { tags });
+ const setSpy = vi.spyOn(Storage.prototype, 'setItem');
document.querySelector('[aria-label="Liste einklappen"]')!.click();
- expect(setSpy).toHaveBeenCalledWith('admin_tags_list_collapsed', 'true');
+ await vi.waitFor(() =>
+ expect(setSpy).toHaveBeenCalledWith('admin_tags_list_collapsed', 'true')
+ );
setSpy.mockRestore();
});
});
diff --git a/frontend/src/routes/admin/users/layout.svelte.spec.ts b/frontend/src/routes/admin/users/layout.svelte.spec.ts
index 865ddbe6..84164697 100644
--- a/frontend/src/routes/admin/users/layout.svelte.spec.ts
+++ b/frontend/src/routes/admin/users/layout.svelte.spec.ts
@@ -131,10 +131,12 @@ describe('UsersListPanel — collapse toggle', () => {
});
it('persists collapse state using the users-specific localStorage key', async () => {
- const setSpy = vi.spyOn(Storage.prototype, 'setItem');
render(UsersListPanel, { users });
+ const setSpy = vi.spyOn(Storage.prototype, 'setItem');
document.querySelector('[aria-label="Liste einklappen"]')!.click();
- expect(setSpy).toHaveBeenCalledWith('admin_users_list_collapsed', 'true');
+ await vi.waitFor(() =>
+ expect(setSpy).toHaveBeenCalledWith('admin_users_list_collapsed', 'true')
+ );
setSpy.mockRestore();
});
});
diff --git a/frontend/src/routes/conversations/page.svelte.spec.ts b/frontend/src/routes/conversations/page.svelte.spec.ts
index 85f18e63..ebc7c4b3 100644
--- a/frontend/src/routes/conversations/page.svelte.spec.ts
+++ b/frontend/src/routes/conversations/page.svelte.spec.ts
@@ -48,9 +48,9 @@ const withDocs = {
// ─── Empty state ──────────────────────────────────────────────────────────────
describe('Conversations page – empty state', () => {
- it('shows the "select two persons" prompt when no persons are selected', async () => {
+ it('shows the empty-state heading when no persons are selected', async () => {
render(Page, { data: baseData });
- await expect.element(page.getByText(/Wählen Sie zwei Personen aus/i)).toBeInTheDocument();
+ await expect.element(page.getByText(/Korrespondenz durchsuchen/i)).toBeInTheDocument();
});
it('hides the swap button when no persons are selected', async () => {
diff --git a/frontend/src/routes/login/page.svelte.spec.ts b/frontend/src/routes/login/page.svelte.spec.ts
index 02b4d5e0..6cf420a8 100644
--- a/frontend/src/routes/login/page.svelte.spec.ts
+++ b/frontend/src/routes/login/page.svelte.spec.ts
@@ -10,7 +10,9 @@ afterEach(cleanup);
describe('Login page – rendering', () => {
it('renders the page title', async () => {
render(LoginPage, {});
- await expect.element(page.getByRole('link', { name: 'Familienarchiv' })).toBeInTheDocument();
+ await expect
+ .element(page.getByRole('link', { name: 'Familienarchiv' }).first())
+ .toBeInTheDocument();
await page.screenshot({ path: 'test-results/screenshots/login-default.png' });
});
diff --git a/frontend/src/routes/page.server.spec.ts b/frontend/src/routes/page.server.spec.ts
index 63d7abd6..24b2e0ad 100644
--- a/frontend/src/routes/page.server.spec.ts
+++ b/frontend/src/routes/page.server.spec.ts
@@ -22,14 +22,14 @@ function makeUrl(params: Record = {}) {
// ─── dashboard mode (no search filters) ──────────────────────────────────────
describe('home page load — dashboard mode', () => {
- it('sets isDashboard true and fetches all three widget APIs', async () => {
+ it('sets isDashboard true and fetches stats, incomplete, and recent APIs', async () => {
const mockGet = vi
.fn()
.mockResolvedValueOnce({ response: { ok: true, status: 200 }, data: [] }) // persons
.mockResolvedValueOnce({
response: { ok: true },
- data: { content: [{ id: 'n1' }] }
- }) // notifications
+ data: { totalDocuments: 42, totalPersons: 7 }
+ }) // stats
.mockResolvedValueOnce({ response: { ok: true }, data: [{ id: 'd1' }] }) // incomplete
.mockResolvedValueOnce({ response: { ok: true }, data: [{ id: 'd2' }] }); // recent
vi.mocked(createApiClient).mockReturnValue({ GET: mockGet } as ReturnType<
@@ -39,17 +39,20 @@ describe('home page load — dashboard mode', () => {
const result = await load({ url: makeUrl(), fetch: vi.fn() as unknown as typeof fetch });
expect(result.isDashboard).toBe(true);
- expect(result.mentions).toHaveLength(1);
+ expect(result.stats).toEqual({ totalDocuments: 42, totalPersons: 7 });
expect(result.incompleteDocs).toHaveLength(1);
expect(result.recentDocs).toHaveLength(1);
expect(result.documents).toEqual([]);
});
- it('defaults mentions to [] when notifications API rejects', async () => {
+ it('returns stats with totalDocuments from /api/stats', async () => {
const mockGet = vi
.fn()
.mockResolvedValueOnce({ response: { ok: true, status: 200 }, data: [] }) // persons
- .mockRejectedValueOnce(new Error('network')) // notifications
+ .mockResolvedValueOnce({
+ response: { ok: true },
+ data: { totalDocuments: 248, totalPersons: 34 }
+ }) // stats
.mockResolvedValueOnce({ response: { ok: true }, data: [] }) // incomplete
.mockResolvedValueOnce({ response: { ok: true }, data: [] }); // recent
vi.mocked(createApiClient).mockReturnValue({ GET: mockGet } as ReturnType<
@@ -58,7 +61,24 @@ describe('home page load — dashboard mode', () => {
const result = await load({ url: makeUrl(), fetch: vi.fn() as unknown as typeof fetch });
- expect(result.mentions).toEqual([]);
+ expect(result.stats?.totalDocuments).toBe(248);
+ expect(result.stats?.totalPersons).toBe(34);
+ });
+
+ it('returns stats: null when /api/stats rejects', async () => {
+ const mockGet = vi
+ .fn()
+ .mockResolvedValueOnce({ response: { ok: true, status: 200 }, data: [] }) // persons
+ .mockRejectedValueOnce(new Error('network')) // stats
+ .mockResolvedValueOnce({ response: { ok: true }, data: [] }) // incomplete
+ .mockResolvedValueOnce({ response: { ok: true }, data: [] }); // recent
+ vi.mocked(createApiClient).mockReturnValue({ GET: mockGet } as ReturnType<
+ typeof createApiClient
+ >);
+
+ const result = await load({ url: makeUrl(), fetch: vi.fn() as unknown as typeof fetch });
+
+ expect(result.stats).toBeNull();
});
it('defaults incompleteDocs to [] when incomplete API rejects', async () => {
@@ -81,7 +101,10 @@ describe('home page load — dashboard mode', () => {
const mockGet = vi
.fn()
.mockResolvedValueOnce({ response: { ok: true, status: 200 }, data: [] }) // persons
- .mockResolvedValueOnce({ response: { ok: true }, data: { content: [] } }) // notifications
+ .mockResolvedValueOnce({
+ response: { ok: true },
+ data: { totalDocuments: 0, totalPersons: 0 }
+ }) // stats
.mockResolvedValueOnce({ response: { ok: true }, data: [] }) // incomplete
.mockRejectedValueOnce(new Error('network')); // recent
vi.mocked(createApiClient).mockReturnValue({ GET: mockGet } as ReturnType<
@@ -113,7 +136,7 @@ describe('home page load — search mode', () => {
expect(result.isDashboard).toBe(false);
expect(result.documents).toHaveLength(1);
- expect(result.mentions).toEqual([]);
+ expect(result.stats).toBeNull();
expect(result.incompleteDocs).toEqual([]);
expect(result.recentDocs).toEqual([]);
// Only two API calls — no widget calls
diff --git a/frontend/src/routes/page.svelte.spec.ts b/frontend/src/routes/page.svelte.spec.ts
index 09956fc7..8a280c1a 100644
--- a/frontend/src/routes/page.svelte.spec.ts
+++ b/frontend/src/routes/page.svelte.spec.ts
@@ -21,8 +21,12 @@ const emptyData = {
user: undefined,
canWrite: true,
canAnnotate: false,
+ isDashboard: false,
filters: { q: '', from: '', to: '', senderId: '', receiverId: '', tags: [] },
documents: [],
+ incompleteDocs: [],
+ recentDocs: [],
+ stats: null,
incompleteCount: 0,
initialValues: { senderName: '', receiverName: '' },
error: null
@@ -189,6 +193,42 @@ describe('Home page – search input keystroke preservation', () => {
});
});
+// ─── Dashboard mode ───────────────────────────────────────────────────────────
+
+describe('Home page – dashboard mode', () => {
+ const dashboardData = {
+ ...emptyData,
+ isDashboard: true,
+ canWrite: false,
+ incompleteDocs: [],
+ recentDocs: []
+ };
+
+ it('hides the right column when canWrite is false and incompleteDocs is empty', async () => {
+ render(Page, { data: dashboardData });
+ const rightCol = page.getByTestId('dashboard-right-column');
+ await expect.element(rightCol).not.toBeInTheDocument();
+ });
+
+ it('shows the right column when canWrite is true', async () => {
+ render(Page, { data: { ...dashboardData, canWrite: true } });
+ const rightCol = page.getByTestId('dashboard-right-column');
+ await expect.element(rightCol).toBeInTheDocument();
+ });
+
+ it('shows the right column when incompleteDocs is non-empty', async () => {
+ render(Page, {
+ data: {
+ ...dashboardData,
+ canWrite: false,
+ incompleteDocs: [{ id: 'd1', title: 'Taufschein' }]
+ }
+ });
+ const rightCol = page.getByTestId('dashboard-right-column');
+ await expect.element(rightCol).toBeInTheDocument();
+ });
+});
+
// ─── Error state ──────────────────────────────────────────────────────────────
describe('Home page – error state', () => {
diff --git a/frontend/src/routes/persons/page.svelte.spec.ts b/frontend/src/routes/persons/page.svelte.spec.ts
index 9fbb8f7f..113df55d 100644
--- a/frontend/src/routes/persons/page.svelte.spec.ts
+++ b/frontend/src/routes/persons/page.svelte.spec.ts
@@ -64,7 +64,7 @@ describe('Persons page – rendering', () => {
it('shows alias in italic when provided', async () => {
render(Page, { data: { ...emptyData, persons: [makePerson({ alias: 'Maxi' })] } });
- await expect.element(page.getByText('"Maxi"')).toBeInTheDocument();
+ await expect.element(page.getByText('„Maxi"')).toBeInTheDocument();
});
it('shows life date range when birthYear is provided', async () => {
diff --git a/frontend/src/routes/profile/PasswordChangeForm.svelte b/frontend/src/routes/profile/PasswordChangeForm.svelte
index 32145da3..6c7e0b42 100644
--- a/frontend/src/routes/profile/PasswordChangeForm.svelte
+++ b/frontend/src/routes/profile/PasswordChangeForm.svelte
@@ -70,6 +70,7 @@ let {