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

e2e/Install, config & test #1592

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

Conversation

prury
Copy link

@prury prury commented Aug 28, 2024

Hello everyone,
Opening this PR to showcase a bit my knowledge regarding QA/Test Automation/CI.

For anyone that doesn't know me, i have been a Talisman sentinel for a long time and since the beginning of the last year shifted to QA/IT once again in my career. I have a background in IT, it is where I've started my career a long time ago(C++, PHP, delphi, networks, html, css, tech support), but then, later on i shifted to the construction area(Architecture) and now, returning, learning typescript and loving it(objects, classes, namespaces, generics, decorators).

Here's what my PR covers:

  1. Playwright installation and basic configuration. (I've setup the installation inside the config folder)

  2. Tailored a playwright fixture to install the built wallet on a chromium browser and wrote a simple on-boarding test that creates a Ethereum account.

  3. Created a playwright.yml file that installs pnpm, install dependencies, build the wallet, install a playwright browser and execute the tests inside a predefined e2e folder.

  4. The test uses the browser context instead of the page context and makes it possible to open a tab and use the extension. (The test itself takes 15-17s to run, what took most of the CI time was building the wallet itself ~3m)

link to workflow running on my fork: https://github.com/prury/talisman/actions/runs/10601904208/job/29382757340?pr=1

Future tests can be setup to run on the embed version of the extension, making it possible test every user interaction:
image

Previous PRs: https://github.com/kodadot/nft-gallery/pulls?page=2&q=is%3Apr+is%3Aclosed+author%3Aprury
Issues(245 closed): https://github.com/kodadot/nft-gallery/issues/created_by/prury

Copy link

socket-security bot commented Aug 28, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@playwright/test@1.46.1 None 0 25.4 kB dgozman-ms, mxschmitt, pavelfeldman, ...1 more
npm/@polkadot/phishing@0.23.4 network +1 51.6 kB polkadotjs

View full report↗︎

@prury prury marked this pull request as ready for review August 28, 2024 18:55
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