Skip to content

test(e2e): add public verification flow tests (#349)#439

Merged
dev-fatima-24 merged 1 commit into
dev-fatima-24:mainfrom
daree-dev:feat/349-e2e-public-verification
May 31, 2026
Merged

test(e2e): add public verification flow tests (#349)#439
dev-fatima-24 merged 1 commit into
dev-fatima-24:mainfrom
daree-dev:feat/349-e2e-public-verification

Conversation

@daree-dev
Copy link
Copy Markdown
Contributor

Summary

Adds a Playwright E2E test for the public verification flow, covering all acceptance criteria from issue #349.

Changes

  • frontend/e2e/public-verification.spec.js — new test file (4 tests)

Acceptance Criteria Coverage

# Criterion Test
1 Navigates to the Verification Page navigates to the Verification Page
2 Vaccinated wallet → green Verified badge + record details shows Verified badge with record count for vaccinated wallet
3 Unvaccinated wallet → No Records Found badge shows No Records Found badge for unvaccinated wallet
4 Invalid wallet format → client-side error shows error message for invalid wallet format

Testing

All API calls are intercepted via Playwright route mocks — no real backend required.
Tests run under npm run test:e2e (chromium/firefox/webkit).

- Navigate to /verify page
- Vaccinated wallet shows Verified badge with record count and NFT details
- Unvaccinated wallet shows No Records Found badge
- Invalid wallet format shows client-side error / blocks submission
@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented May 31, 2026

Hey @daree-dev! 👋 It looks like this PR isn't linked to any issue.

If this PR is for one of the issues assigned to you as part of a Wave, please link it to ensure your contribution is tracked properly. You can do this by adding a keyword to the PR description (e.g., Closes #123), or by clicking a button below:

Issue Title
#350 [TESTING] Write performance tests for verify endpoint under load Link to this issue
#349 [TESTING] Write end-to-end test for public verification flow Link to this issue

ℹ️ Learn more about linking PRs to issues

@dev-fatima-24 dev-fatima-24 merged commit 4016825 into dev-fatima-24:main May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants