From c0eca10b51dc698223f5f890c3582bc00d1fe2b2 Mon Sep 17 00:00:00 2001 From: Dave Date: Thu, 27 Aug 2026 00:48:14 +0200 Subject: [PATCH] fix(test): drop stale triggerSend comment in approvals page test The fix commit (27f60e1) removed the triggerSend hoisted spy and wired the modal stub's button to call props.onSend directly, but left a two-line comment in test 3 that still referenced the deleted symbol. Replace the comment with one that describes the actual mechanism so future readers don't go hunting for a triggerSend identifier that no longer exists. Co-Authored-By: Claude --- src/app/(app)/quotes/approvals/page.test.tsx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/app/(app)/quotes/approvals/page.test.tsx b/src/app/(app)/quotes/approvals/page.test.tsx index 0ba4eb2..768b413 100644 --- a/src/app/(app)/quotes/approvals/page.test.tsx +++ b/src/app/(app)/quotes/approvals/page.test.tsx @@ -163,8 +163,8 @@ describe("ApprovalsPage — Approve & send from list", () => { const buttons = await screen.findAllByRole("button", { name: /approve & send/i }); fireEvent.click(buttons[0]); await waitFor(() => expect(captureModalProps).toHaveBeenCalled()); - // The modal stub's "Submit modal" button fires triggerSend which is - // wired up as the page's onSend callback below. + // The modal stub's "Submit modal" button calls props.onSend directly, + // which is the page's sendApprove callback. const submitBtn = await screen.findByRole("button", { name: /submit modal/i }); fireEvent.click(submitBtn); await waitFor(() => {