Skip to content

feat(e2e) add data testid attributes#217

Open
nomadbitcoin wants to merge 4 commits intomainfrom
e2e-tests/add-data-testid-attributes
Open

feat(e2e) add data testid attributes#217
nomadbitcoin wants to merge 4 commits intomainfrom
e2e-tests/add-data-testid-attributes

Conversation

@nomadbitcoin
Copy link
Copy Markdown
Contributor

Added data-testid attributes to all interactive components so E2E tests can reliably select elements regardless of styling changes.

All review comments from PR #204 have been addressed.

Add data-testid attributes to all interactive components across
navigation, wallet modal, bidders, candidates, members, payouts
pages, and transaction feedback toasts to enable reliable E2E
test element selection.
- Replace data-testid with data-test convention (cypress-realworld-app)
- Use type extensions instead of custom test props on components
- Rename abbreviated selectors (btn → button) for discoverability
- Add cy.getBySel() and cy.getBySelLike() Cypress custom commands
- Break long line in BidVouch submit button
- Remove unused htmlProps destructuring in TimeRemaining
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 15, 2026

Deploy Preview for dreamy-gates-ad44c0 ready!

Name Link
🔨 Latest commit 24dc3ee
🔍 Latest deploy log https://app.netlify.com/projects/dreamy-gates-ad44c0/deploys/69df369c8c53e60008be9947
😎 Deploy Preview https://deploy-preview-217--dreamy-gates-ad44c0.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant