test: update notification sender attempt mocks

- Exercise identity-aware delivery attempt recording\n- Keep self-delivery assertions explicit

src/lib/notifications/__tests__/client-managed-apns.test.ts

@@ -137,7 +137,8 @@ describe("dispatchNotification — client-managed APNs suppression", () => {
       },
     );
     expect(recordPushAttemptMock).toHaveBeenCalledWith({
-      userId: "u-1",
+      recordUserId: "u-1",
+      recipientUserId: "u-1",
       channel: "APNS",
       eventType: "MEDICATION_REMINDER",
       result: "skipped",

src/lib/notifications/__tests__/web-push-dose-clear.test.ts

@@ -45,6 +45,7 @@ vi.mock("@/lib/logging/context", () => ({
 
 vi.mock("@/lib/notifications/senders/push-attempt-record", () => ({
   recordPushAttempt: vi.fn(),
+  recordPushAttemptForPayload: vi.fn(),
 }));
 
 vi.mock("@/lib/db", () => ({

src/lib/notifications/__tests__/web-push-egress-pin.test.ts

@@ -38,6 +38,7 @@ vi.mock("@/lib/logging/context", () => ({
 
 vi.mock("@/lib/notifications/senders/push-attempt-record", () => ({
   recordPushAttempt: vi.fn(),
+  recordPushAttemptForPayload: vi.fn(),
 }));
 
 vi.mock("@/lib/db", () => ({

src/lib/notifications/senders/__tests__/email.test.ts

@@ -31,6 +31,11 @@ vi.mock("@/lib/logging/context", () => ({
 
 vi.mock("@/lib/notifications/senders/push-attempt-record", () => ({
   recordPushAttempt: (...args: unknown[]) => recordPushAttemptMock(...args),
+  recordPushAttemptForPayload: (
+    _payload: unknown,
+    _recipientUserId: string,
+    attempt: unknown,
+  ) => recordPushAttemptMock(attempt),
 }));
 
 import { sendViaEmail, resetEmailTransporterForTesting } from "../email";

src/lib/notifications/senders/__tests__/telegram.test.ts

@@ -42,6 +42,7 @@ vi.mock("@/lib/logging/context", () => ({
 
 vi.mock("@/lib/notifications/senders/push-attempt-record", () => ({
   recordPushAttempt: vi.fn(),
+  recordPushAttemptForPayload: vi.fn(),
 }));
 
 const scheduleTelegramAutoDeleteMock = vi.fn();

src/lib/notifications/senders/__tests__/urgency.test.ts

@@ -35,6 +35,11 @@ vi.mock("@/lib/logging/context", () => ({
 
 vi.mock("@/lib/notifications/senders/push-attempt-record", () => ({
   recordPushAttempt: (...args: unknown[]) => recordPushAttemptMock(...args),
+  recordPushAttemptForPayload: (
+    _payload: unknown,
+    _recipientUserId: string,
+    attempt: unknown,
+  ) => recordPushAttemptMock(attempt),
 }));
 
 vi.mock("@/lib/db", () => ({

src/lib/notifications/senders/__tests__/webhook.test.ts

@@ -27,6 +27,11 @@ vi.mock("@/lib/logging/context", () => ({
 
 vi.mock("@/lib/notifications/senders/push-attempt-record", () => ({
   recordPushAttempt: (...args: unknown[]) => recordPushAttemptMock(...args),
+  recordPushAttemptForPayload: (
+    _payload: unknown,
+    _recipientUserId: string,
+    attempt: unknown,
+  ) => recordPushAttemptMock(attempt),
 }));
 
 import { sendViaWebhook } from "../webhook";