From a7484acca404b6856ac8c5abed190040356afd5b Mon Sep 17 00:00:00 2001 From: "pulse-triage[bot]" <249995291+pulse-triage[bot]@users.noreply.github.com> Date: Mon, 7 Sep 2026 21:32:33 +0100 Subject: [PATCH] style(web): restore alert recovery formatting gate Apply the repository formatter to the two alert recovery test files rejected by the protected frontend check. This is an additive formatting-only correction; reviewed commit identities and behavior remain unchanged. Change-source: pulse-maintainer --- .../alerts/AlertDeliveryHealthCard.test.tsx | 10 +++--- .../DestinationsTab.recoveryediting.test.tsx | 34 +++++++++++++------ 2 files changed, 30 insertions(+), 14 deletions(-) diff --git a/frontend-modern/src/features/alerts/AlertDeliveryHealthCard.test.tsx b/frontend-modern/src/features/alerts/AlertDeliveryHealthCard.test.tsx index 82170c438..042da4ff4 100644 --- a/frontend-modern/src/features/alerts/AlertDeliveryHealthCard.test.tsx +++ b/frontend-modern/src/features/alerts/AlertDeliveryHealthCard.test.tsx @@ -221,10 +221,12 @@ describe('recovery feedback beside delivery health', () => { afterEach(() => cleanup()); it('preserves editing focus and live-region identity when failure feedback changes', () => { const [message, setMessage] = createSignal(null); - render(() => <> - - setMessage(null)} /> - ); + render(() => ( + <> + + setMessage(null)} /> + + )); const input = screen.getByRole('textbox', { name: 'Destination name' }); const status = screen.getByRole('status'); expect(status).toHaveAttribute('aria-live', 'polite'); diff --git a/frontend-modern/src/features/alerts/__tests__/DestinationsTab.recoveryediting.test.tsx b/frontend-modern/src/features/alerts/__tests__/DestinationsTab.recoveryediting.test.tsx index 2307c007c..2c07536ca 100644 --- a/frontend-modern/src/features/alerts/__tests__/DestinationsTab.recoveryediting.test.tsx +++ b/frontend-modern/src/features/alerts/__tests__/DestinationsTab.recoveryediting.test.tsx @@ -137,12 +137,16 @@ describe('DestinationsTab recovery while editing SMTP', () => { vi.restoreAllMocks(); }); - it.each(([ - ['retryTerminalFailures', 'Retry retained deliveries', 'retry'], - ['dismissTerminalFailures', 'Dismiss retained failures', 'dismiss'], - ] as const).flatMap(([action, label, verb]) => - (['available', 'unavailable'] as const).map((refresh) => ({ action, label, verb, refresh })), - ))( + it.each( + ( + [ + ['retryTerminalFailures', 'Retry retained deliveries', 'retry'], + ['dismissTerminalFailures', 'Dismiss retained failures', 'dismiss'], + ] as const + ).flatMap(([action, label, verb]) => + (['available', 'unavailable'] as const).map((refresh) => ({ action, label, verb, refresh })), + ), + )( 'preserves unfinished edits through $action with $refresh refresh', async ({ action, label, verb, refresh }) => { const [emailConfig, setEmailConfig] = createSignal(buildEmailConfig()); @@ -222,8 +226,12 @@ describe('DestinationsTab recovery while editing SMTP', () => { deadLetter: 0, }; if (refresh === 'unavailable') { - vi.mocked(NotificationsAPI.getHealth).mockRejectedValueOnce(new Error('health unavailable')); - vi.mocked(NotificationsAPI.getDeliveryLog).mockRejectedValueOnce(new Error('history unavailable')); + vi.mocked(NotificationsAPI.getHealth).mockRejectedValueOnce( + new Error('health unavailable'), + ); + vi.mocked(NotificationsAPI.getDeliveryLog).mockRejectedValueOnce( + new Error('history unavailable'), + ); } else { vi.mocked(NotificationsAPI.getHealth).mockResolvedValueOnce(healthy); } @@ -246,8 +254,14 @@ describe('DestinationsTab recovery while editing SMTP', () => { expect(screen.getByRole('status')).toBe(status); expect(status).toBeEmptyDOMElement(); if (refresh === 'unavailable') { - expect(await screen.findByText('Notification delivery status is unavailable')).toBeInTheDocument(); - expect(await screen.findByText('Pulse could not read the delivery log, so recent delivery activity cannot be shown.')).toBeInTheDocument(); + expect( + await screen.findByText('Notification delivery status is unavailable'), + ).toBeInTheDocument(); + expect( + await screen.findByText( + 'Pulse could not read the delivery log, so recent delivery activity cannot be shown.', + ), + ).toBeInTheDocument(); expect(screen.queryByText('SMTP fixture rejected')).not.toBeInTheDocument(); } else { expect(screen.getByText('SMTP fixture rejected')).toBeInTheDocument();