Skip to content

ci: run playwright wallet/transaction matrix on chromium firefox webkit - #236

Open
cybermax4200 wants to merge 1 commit into
wraith-protocol:developfrom
cybermax4200:feat/ci-playwright-matrix
Open

cybermax4200 wants to merge 1 commit into
wraith-protocol:developfrom
cybermax4200:feat/ci-playwright-matrix

Conversation

@cybermax4200

Copy link
Copy Markdown

Summary

Adds a GitHub Actions workflow that runs the Playwright E2E matrix (wallet, send, receive, vault, and failure-path tests) across Chromium, Firefox, and WebKit.

Changes

  • .github/workflows/e2e.yml: new CI job running on push/PR to main/develop:
    • Matrix: chromium, firefox, webkit (fail-fast: false)
    • Installs Playwright browsers per project (npx playwright install --with-deps)
    • Builds the app, then runs pnpm playwright test --project=${{ matrix.project }} with CI: true
    • Uploads the Playwright HTML report on every run (always())
    • Uploads traces only on failure (if: failure()), per the requirement to keep traces out of passing runs

Test coverage

Existing specs under tests/ and e2e/ cover:

  • Wallet connect/picker (Stellar Freighter/Albedo, connection failure paths)
  • Send / receive flows
  • Vault deposit/refund/claim (simulate → assemble → sign → send → poll ordering, retry/failure paths)

Browser-specific expectations are already explicit in specs (e.g. Chromium-only clipboard assertions skip on Firefox/WebKit via browserName).

Verification

  • pnpm format:check: passed
  • YAML validated locally
  • Chromium binary installed and tests/vault-deposit.spec.ts: 7 passed locally

Closes #210

@drips-wave

drips-wave Bot commented Oct 17, 2026

Copy link
Copy Markdown

@cybermax4200 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 Oct 17, 2026

Copy link
Copy Markdown

@cybermax4200 is attempting to deploy a commit to the truthixify's projects Team on Vercel.

A member of the Team first needs to authorize it.

@truthixify truthixify left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All three browser jobs fail before tests because pnpm/action-setup specifies version 10 while package.json pins pnpm@10.28.2. Use the packageManager field or the exact same version. Also upload Playwright's actual trace location (test-results/**/trace.zip or the configured output directory); trace/ is not produced by the current config. Rerun the Chromium, Firefox, and WebKit matrix successfully.

This branch has not been deployed

No deployments
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.

[Wave 10][Carryover] Run the Playwright wallet and transaction matrix in CI

2 participants