Skip to content

Add E2E tests for WalletConnect using Playwright#172

Open
rnbrady wants to merge 12 commits intocashonize:mainfrom
rnbrady:feat/127-wc-e2e-tests
Open

Add E2E tests for WalletConnect using Playwright#172
rnbrady wants to merge 12 commits intocashonize:mainfrom
rnbrady:feat/127-wc-e2e-tests

Conversation

@rnbrady
Copy link
Contributor

@rnbrady rnbrady commented Feb 12, 2026

Adds:

  • a minimal Vue-based test dApp with WalletConnect
  • Playwright as dev dependency
  • E2E tests for walletconnect using Playwright

If the environment or .env contains E2E_SEED_PHRASE this will be used to test transaction signing. Otherwise that test will be skipped.

Correction: Turns out you can send an arbitrary tx to cashonize for signing with broadcast: false and it will sign it. So we don't need a funded chipnet seed phrase for the tests. However, such the seed phrase will be useful for future e2e tests so I have left support for it in an simply removed the condition so that signTransaction is tested either way.

@rnbrady rnbrady changed the title Add EE for WalletConnect using Palywright Add E2E for WalletConnect using Palywright Feb 12, 2026
@rnbrady rnbrady changed the title Add E2E for WalletConnect using Palywright Add E2E tests for WalletConnect using Palywright Feb 12, 2026
@rnbrady rnbrady marked this pull request as ready for review February 12, 2026 11:27
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a Playwright-based E2E test setup for WalletConnect by introducing a minimal Vue test dApp, Playwright configuration, and CI wiring to run the E2E suite alongside existing lint/type/unit tests.

Changes:

  • Add Playwright E2E tests covering WalletConnect connect/sign/cancel/disconnect flows.
  • Add a standalone Vue/Vite “test dApp” used by the E2E tests.
  • Wire E2E execution into package.json scripts and GitHub Actions CI.

Reviewed changes

Copilot reviewed 14 out of 20 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
yarn.lock Adds Playwright packages to the root Yarn v1 lockfile.
package.json Adds test:e2e and test:e2e:ui scripts and @playwright/test dev dependency.
vitest.config.ts Excludes test/e2e/** from Vitest runs.
tsconfig.json Formatting-only change.
quasar.config.ts Excludes test/e2e/test-dapp from Quasar-generated TS config checking.
test/e2e/playwright.config.ts New Playwright config that starts the wallet dev server and the test dApp server.
test/e2e/walletconnect.test.ts New Playwright E2E test suite for WalletConnect flows.
test/e2e/test-dapp/package.json New minimal Vue/Vite test dApp package definition.
test/e2e/test-dapp/vite.config.ts Vite config for the test dApp (fixed port).
test/e2e/test-dapp/tsconfig.json TypeScript config for the test dApp.
test/e2e/test-dapp/index.html Test dApp HTML entrypoint.
test/e2e/test-dapp/src/main.ts Vue bootstrap for the test dApp.
test/e2e/test-dapp/src/App.vue Test dApp UI + WalletConnect request logic and TX fixture.
test/e2e/test-dapp/README.md Usage notes for running the test dApp standalone.
test/e2e/test-dapp/.gitignore Ignores test dApp build artifacts and deps.
test/e2e/test-dapp/yarn.lock Adds a separate lockfile for the test dApp (currently Yarn Berry format).
.gitignore Ignores .env* and Playwright test-results/.
.github/workflows/ci.yml Adds yarn test:e2e to CI.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rnbrady rnbrady changed the title Add E2E tests for WalletConnect using Palywright Add E2E tests for WalletConnect using Playwright Feb 12, 2026
@rnbrady
Copy link
Contributor Author

rnbrady commented Feb 12, 2026

Closes #127

@mr-zwets mr-zwets linked an issue Feb 12, 2026 that may be closed by this pull request
6 tasks
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.

Add WalletConnect integration tests

1 participant