Skip to content

closes issue 136#242

Merged
phertyameen merged 1 commit into
bridgelet-org:mainfrom
yesi3:fix/136-playwright-e2e
Jul 27, 2026
Merged

closes issue 136#242
phertyameen merged 1 commit into
bridgelet-org:mainfrom
yesi3:fix/136-playwright-e2e

Conversation

@yesi3

@yesi3 yesi3 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Executive Summary
Adds Playwright end-to-end testing for the Bridgelet reference UI. Installs @playwright/test, adds a base config targeting http://localhost:3000, introduces a smoke E2E suite, wires test:e2e / test:e2e:ui npm scripts, and updates Frontend CI to run E2E against the built Next.js app (next build then next start). Documentation in TESTING.md, E2E guidelines, frontend README, and CHANGELOG is updated to match.

Linked Issues & References
Issue: Resolves #136
Related PRs: N/A
Type of Change
Category

New Feature (Non-breaking change adding functionality)

Bug Fix (Non-breaking change fixing an issue)

Performance Optimization (Code change to improve efficiency)

Refactoring (Code structure change without external API modification)

Security Patch (Fixing a vulnerability or security concern)

Documentation (Updates to README, Wiki, or inline docs)

Infrastructure (CI/CD, build scripts, Docker)
Domain

Smart Contracts (Rust/Soroban)

SDK/Backend (NestJS/TypeScript)

Frontend/UI
Technical Implementation Details
Added @playwright/test and frontend/playwright.config.ts with baseURL: http://localhost:3000
Local runs use npm run dev (or reuse an existing server); CI (CI=true) uses npm run start after npm run build
Smoke specs in frontend/e2e/ cover homepage load, navigation to /send and /claim/example-token, and send-page render
Frontend CI (.github/workflows/frontend-ci.yml) installs Chromium, runs npm run test:e2e, and uploads the Playwright report on failure
Docs updated: TESTING.md, docs/testing/E2E_GUIDELINES.md, frontend/README.md, CHANGELOG.md
Breaking Changes

No - Backward compatibility is maintained.

Yes - This change requires consumers of the SDK/Contract to update their code.
Verification & Testing Strategy
Automated Tests

Unit Tests: Added/Updated new unit tests.

Integration Tests: Verified interaction between modules.

Linting: Code follows the project's style guide.
Manual Verification

  1. Environment:

Target Environment: Local Next.js app on http://localhost:3000 (dev server via Playwright webServer)
2. Reproduction Steps:

cd frontend && npx playwright install --with-deps chromium
npm run test:e2e
Confirm all smoke specs pass (optionally npm run build && CI=true npm run test:e2e to mirror CI)
3. Proof of Execution: Chromium installed successfully. Playwright reported 4 passed (18.0s) for e2e/home.spec.ts and e2e/send.spec.ts. Production npm run build also succeeded locally before commit.

Quality Assurance Checklist

I have performed a self-review of my own code.

I have commented on my code, particularly in hard-to-understand areas.

I have updated the documentation to reflect these changes.

My changes do not generate new warnings or errors.

I have verified that new dependencies are necessary and secure.

(If Smart Contract) I have verified gas usage and storage footprints are optimized.
By submitting this Pull Request, I acknowledge that I have adhered to the Bridgelet contribution guidelines.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

@yesi3 is attempting to deploy a commit to the aminubabafatima8-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jul 24, 2026

Copy link
Copy Markdown

@yesi3 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 Jul 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bridgelet Ready Ready Preview, Comment Jul 25, 2026 6:43am

@phertyameen

Copy link
Copy Markdown
Contributor

@yesi3 Please fix CI

@phertyameen

Copy link
Copy Markdown
Contributor

My CI is designed to protect deployment. You cannot edit it or any sensitive/configuration

The vercel build will catch it
@yesi3

@phertyameen
phertyameen merged commit ace8aaa into bridgelet-org:main Jul 27, 2026
2 of 3 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

Development

Successfully merging this pull request may close these issues.

Set up Playwright for E2E testing

2 participants