Skip to content

feat(frontend): add comprehensive component unit and snapshot tests f… - #1625

Merged
emdevelopa merged 1 commit into
emdevelopa:mainfrom
LawalRahman:feature/fe-add-comprehensive-component-unit-and-snapshot-tests-for-checkout-qr-modal
Oct 20, 2026
Merged

emdevelopa merged 1 commit into
emdevelopa:mainfrom
LawalRahman:feature/fe-add-comprehensive-component-unit-and-snapshot-tests-for-checkout-qr-modal

Conversation

@LawalRahman

Copy link
Copy Markdown
Contributor

…or Checkout QR Modal

  • Visibility tests: isOpen true/false render states
  • QR code tests: value prop, error-correction level H, size 512, re-render on prop change
  • Download button tests: success path (toDataURL + anchor click), filename uses paymentId, error toast when canvas absent, error toast when toDataURL throws
  • Close behaviour: close button click, backdrop click, Escape key
  • Accessibility: role=dialog, aria-modal, aria-labelledby, close button label, backdrop aria-hidden, keyboard tab order
  • Snapshot tests: open/closed/custom qrValue/custom paymentId

Closes #1532
Closes #1533
Closes #1535
Closes #1537

…or Checkout QR Modal

- Visibility tests: isOpen true/false render states
- QR code tests: value prop, error-correction level H, size 512, re-render on prop change
- Download button tests: success path (toDataURL + anchor click), filename uses paymentId,
  error toast when canvas absent, error toast when toDataURL throws
- Close behaviour: close button click, backdrop click, Escape key
- Accessibility: role=dialog, aria-modal, aria-labelledby, close button label,
  backdrop aria-hidden, keyboard tab order
- Snapshot tests: open/closed/custom qrValue/custom paymentId

Closes emdevelopa#1532
@vercel

vercel Bot commented Oct 20, 2026

Copy link
Copy Markdown

@LawalRahman is attempting to deploy a commit to the Emmanuel's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Oct 20, 2026

Copy link
Copy Markdown

@LawalRahman Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@emdevelopa
emdevelopa merged commit d885a30 into emdevelopa:main Oct 20, 2026
1 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants