From 7a74311a71a679e361f5d36a182a4d60b9bd6d61 Mon Sep 17 00:00:00 2001 From: cesnimda Date: Sat, 18 Jul 2026 15:30:36 +0200 Subject: [PATCH] test(settings): unbreak stale settings-view assertion blocking CI MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Notifications tab dropped the old "check system status" link; SMTP status now lives under Admin → System → Settings (settingsNotificationsDelivery). The test still asserted the removed text, failing the frontend CI job and blocking deploy. Point the assertion at the current delivery caption. Full suite 88/88. Co-Authored-By: Claude Opus 4.8 --- job-tracker-ui/src/settings-view.test.tsx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/job-tracker-ui/src/settings-view.test.tsx b/job-tracker-ui/src/settings-view.test.tsx index 4842a3b..8538c26 100644 --- a/job-tracker-ui/src/settings-view.test.tsx +++ b/job-tracker-ui/src/settings-view.test.tsx @@ -79,7 +79,8 @@ test('settings view has no accent picker and uses one follow-up section, one not fireEvent.click(screen.getByRole('tab', { name: /notifications/i })); expect(screen.getByText(/notification settings/i)).toBeInTheDocument(); - expect(screen.getAllByText(/check system status/i).length).toBe(1); + // SMTP status now lives under Admin → System → Settings (the old "check system status" link was removed). + expect(screen.getAllByText(/smtp delivery and test mail live under/i).length).toBe(1); expect(screen.getByLabelText(/email reminders for follow-ups/i)).toBeInTheDocument(); expect(screen.getByLabelText(/email alerts for ghosted jobs/i)).toBeInTheDocument(); });