✨ feat: Add keplr wallet support and shared utils package

Seroxdesign commented 1 month ago

Motivation and context

We want to support keplr wallet and the cosmos ecosystem, as well as Phantom to support the Solana ecosystem. Additionally we want to set a standard for multi-wallet cache support

Does it fix any issue?

Allows us to use multiple wallets with cache
Adds Keplr/cosmos support

Other useful info

I removed test/unit from wallets since we removed the duplicate prepareExtension wallet solely in sypress-cache and all the units are already tested in cache
Modified CLI to take a flag to decide on a wallet to use for cache
Added over all the necessary functions/pages/and tests to use as a base for keplr
Added 2 setup functions in keplr (Note: the cache library needs 2 setup files to work properly, consistently and produce the correct hash in cache and wallet)

Quality checklist

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough e2e tests.

⚠️👆 Delete any section you see irrelevant before submitting the pull request 👆⚠️

vercel[bot] commented 1 month ago

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

Name	Status	Preview	Comments	Updated (UTC)
synpress	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 5, 2024 0:39am

Seroxdesign commented 1 month ago

extra everything into utils
recall definewalletsetup

Seroxdesign commented 1 month ago

Cypress overrides the chromium profile making it impossible for us to fetch the extensions/cache
We could run Cypress fixtures separately with no cache
Some concerns around /utils package were raised by @matstyler mainly that most of the utils right now are helpers for /cache There are more shared functions such as fixtures and generic playwright commands that could go in there as well

Seroxdesign commented 1 month ago

e2e tests and build cache are working fine. I am having some issues with build in release & non-e2e testing.

Seroxdesign commented 3 weeks ago

Things to change:

Change the randomly generated ClassNames to somethings more stable
Maybe select all elements of a certain type and filter through them
Priority should be for text, or xpath, IDs, or test-ids

Seroxdesign commented 2 weeks ago

NOTE:

We need to update synpress npm packages and versioning in package.json before we merge this

Synthetixio / synpress