David/WFDASH-WEBAPP
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(test): drop stale triggerSend comment in approvals page test

Browse source 
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 <noreply@anthropic.com>
This commit is contained in:
Dave 2026-08-27 00:48:14 +02:00
parent 27f60e14ec
commit c0eca10b51
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/app/(app)/quotes/approvals/page.test.tsx
View file

@ -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(() => {