From 6d1587531fd0ffbcadb914b363c1528fb4bc776a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= Date: Tue, 2 Jun 2026 17:04:40 +0200 Subject: [PATCH] test(settings): expect the shortened notifications section label The settings section reverts to the single-line "Notifications" / "Benachrichtigungen"; update the shell title assertions to match. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../settings/__tests__/settings-shell.test.tsx | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/src/components/settings/__tests__/settings-shell.test.tsx b/src/components/settings/__tests__/settings-shell.test.tsx index 2a2752820..829205233 100644 --- a/src/components/settings/__tests__/settings-shell.test.tsx +++ b/src/components/settings/__tests__/settings-shell.test.tsx @@ -145,10 +145,10 @@ describe("", () => { const html = renderShell({ active: "account", locale: "en" }); expect(html).toContain("Account"); expect(html).toContain("Integrations"); - // v1.4.33 IW7 — section renamed from "Notifications" to - // "Notification channels" so it doesn't collide with the inbox at - // `/notifications` ("Notification Center"). - expect(html).toContain("Notification channels"); + // v1.9.0 — the section label is back to the shorter "Notifications" + // (single-line; the longer "Notification channels" wrapped). The + // `/notifications` inbox is still "Notification Center" — distinct enough. + expect(html).toContain("Notifications"); expect(html).toContain("Dashboard"); expect(html).toContain("Insights"); // The ampersand is HTML-escaped by React SSR — assert on the encoded @@ -173,10 +173,10 @@ describe("", () => { const html = renderShell({ active: "account", locale: "de" }); expect(html).toContain("Konto"); expect(html).toContain("Integrationen"); - // v1.4.33 IW7 — section renamed from "Benachrichtigungen" to - // "Benachrichtigungs-Kanäle" so it doesn't collide with the inbox at - // `/notifications` ("Benachrichtigungs-Center"). - expect(html).toContain("Benachrichtigungs-Kanäle"); + // v1.9.0 — back to the shorter "Benachrichtigungen" (single-line; the + // compound "Benachrichtigungs-Kanäle" wrapped). The `/notifications` + // inbox stays "Benachrichtigungs-Center" — distinct enough. + expect(html).toContain("Benachrichtigungen"); // v1.4.3: the Settings sub-section formerly labelled "Übersicht" is now // "Dashboard" (matching the term users see in the main nav). The // per-metric overrides moved out into their own "Persönliche Zielwerte"