Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce e2e testing #275

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from
Open

Conversation

tombeckenham
Copy link
Collaborator

Related Issue

Closes #243

Summary of Changes

Added Playwright that can be run by developers
Created one test that loads the extension and ensures welcome page is shown
Does not run tests on CI

Need Regression Testing

No code changes that affect the build

  • Yes
  • [ X ] No

Risk Assessment

  • [ X] Low
  • Medium
  • High

Additional Notes

More will follow later but will need bigger changes such as adding the emulator

- Added Playwright test dependencies to package.json and pnpm-lock.yaml.
- Updated .gitignore to exclude test results and Playwright reports.

Closes #243
- Updated ESLint configuration to include e2e tests.
- Changed Playwright test directory from './tests' to './e2e'.
- Adjusted TypeScript configuration to include e2e files.
- Removed obsolete test files: example.spec.ts and wallet.spec.ts.

Closes #243
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.

1 participant