cesnimda/jobtrackingapp
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

test(settings): unbreak stale settings-view assertion blocking CI
CI and Deploy / test (push) Successful in 2m15s
Details
CI and Deploy / deploy (push) Failing after 37s
Details

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
cesnimda
2026-07-18 15:30:36 +02:00
parent fd9bd3f47a
commit 7a74311a71
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
job-tracker-ui/src/settings-view.test.tsx
+2 -1
View File
@@ -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();
});