carrot-kpi / monorepo

A monorepo for Carrot v1.
GNU General Public License v3.0
5 stars 2 forks source link

feat(qa): custom metamask setup #571

Closed MilanVojnovic95 closed 9 months ago

MilanVojnovic95 commented 9 months ago

Trying to figure out the work-around since the sypress setup that we are using is not working as expected. Installation and wallet import work but when Metamask notification pop-up for connecting the Carrot with Metamask i.e. acceptAccess() method is being called; the notification window is stuck on loader - if I refresh the noticifation window manually then the test passes.

I've tried with different versions of Metamask. But, Metamask version that is being used by the Synpress setup is 10.25.0 for which the method acceptAccess() should be working. If I try to install any version newer than v10.26.0 the method acceptAccess() is not going to work because the user flow is changed. For example there is a checkbox that should be checked before initiating wallet import.

changeset-bot[bot] commented 9 months ago

⚠️ No Changeset found

Latest commit: 57a77e24012dee8f37576f108de14a78457d8f2c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

vercel[bot] commented 9 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
host-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 22, 2024 4:40pm
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 22, 2024 4:40pm
MilanVojnovic95 commented 9 months ago

@guerrap I've tried different things, but I think it would not work with Cypress. By the Cypress docs before:browser:launch API cannot load extensions in headless Chrome which makes it useless in our case.

I will close this PR.

The idea is

I think this can be done with Playwright like we had so I'll try that.