Skip to content

test: add comprehensive test suites for payment features - #641

Open
janipauwels-sys wants to merge 4 commits into
Stellar-split:mainfrom
janipauwels-sys:fix/issues-593-596
Open

test: add comprehensive test suites for payment features#641
janipauwels-sys wants to merge 4 commits into
Stellar-split:mainfrom
janipauwels-sys:fix/issues-593-596

Conversation

@janipauwels-sys

Copy link
Copy Markdown
Contributor

Summary

This PR adds comprehensive test suites for four payment-related features:

Test Coverage

Issue #593 - Payment Reference Copy Button

  • Copy button renders adjacent to payment reference
  • Clipboard API called when button is clicked
  • "Copied!" confirmation message appears for 2 seconds

Issue #594 - Arrow Key Navigation

  • ArrowDown/ArrowUp keys navigate between options
  • Enter and Space keys confirm selection
  • Focus wraps around (circular navigation)
  • Proper ARIA roles and keyboard-only accessibility

Issue #595 - Print Stylesheet

  • Print-only styling with @media print
  • Navigation and interactive elements hidden when printing
  • Font sizes suitable for print output
  • Content width constrained for A4/Letter page sizes

Issue #596 - Wallet Connection State

  • "Pay All" button disabled when wallet disconnected
  • Tooltip shows "Connect your wallet to continue"
  • Button enabled when wallet connected
  • Button state changes when wallet connects/disconnects

Testing

All tests follow the existing patterns in the codebase using Vitest and React Testing Library. Each feature has dedicated test cases covering the requirements from the GitHub issues.

Closes #596
Closes #595
Closes #594
Closes #593

…nt reference

- Test copy button renders adjacent to payment reference
- Test clipboard API is called when button clicked
- Test 'Copied!' confirmation appears for 2 seconds
- Implements feature from issue Stellar-split#593
…ayModal

- Test 'Pay All' button is disabled when wallet disconnected
- Test disabled button shows tooltip 'Connect your wallet to continue'
- Test button enabled when wallet connected
- Test button state changes when wallet connects/disconnects
- Implements feature from issue Stellar-split#596
…icate

- Test print-only styling with @media print
- Test navigation hidden when printing
- Test font sizes suitable for print output
- Test content width constrained for A4/Letter page sizes
- Test all recipients and payments displayed in table format
- Test QR code included for verification
- Implements feature from issue Stellar-split#595
…thodSelector

- Test ArrowDown moves focus to next option
- Test ArrowUp moves focus to previous option
- Test Enter and Space keys confirm selection
- Test focus wraps around (circular navigation)
- Test proper ARIA roles (listbox, option)
- Test keyboard-only accessibility without mouse
- Test saves preference to localStorage
- Implements feature from issue Stellar-split#594
@drips-wave

drips-wave Bot commented Aug 27, 2026

Copy link
Copy Markdown

@janipauwels-sys 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

@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

@primeforce is attempting to deploy a commit to the kingsman-99's projects Team on Vercel.

A member of the Team first needs to authorize it.

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