gsadmin/pulse
1
0
Fork 0
mirror of https://github.com/rcourtman/Pulse.git synced 2026-09-10 02:25:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

test(notifications): isolate terminal recovery paths

Browse Source
This commit is contained in:
pulse-triage[bot]
2026-08-27 16:08:53 +01:00
parent 947a26b72f
commit 30b4fbd95e
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/api/alerting/notification_queue_additional_test.go
+7 -2
View File
@@ -145,8 +145,13 @@ func TestNotificationQueueHandlers_BulkTerminalFailureRecovery(t *testing.T) {
}
}
if err := queue.UpdateStatus("notif-bulk", notifications.QueueStatusDLQ, "still unavailable"); err != nil {
t.Fatalf("mark retried notification DLQ: %v", err)
// Use a distinct terminal delivery for dismissal. The retry above is now
// pending and the real background worker is entitled to deliver it at any
// time; forcing that in-flight row back to DLQ makes this handler test race
// the queue processor instead of testing the dismissal contract.
enqueueDLQNotification(t, queue, "notif-dismiss")
if err := queue.UpdateStatus("notif-dismiss", notifications.QueueStatusDLQ, "still unavailable"); err != nil {
t.Fatalf("mark dismiss notification DLQ: %v", err)
}
assertNotificationDeliveryAlertActive(t, handler.monitor)
req = httptest.NewRequest(http.MethodPost, "/api/notifications/terminal-failures/dismiss", nil)