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

🫠 test(cypress): /liquidity #299

Merged
merged 3 commits into from
Aug 22, 2024

Conversation

aahna-ashina
Copy link
Member

@aahna-ashina aahna-ashina commented Aug 22, 2024

Add Cypress test for /liquidity page

Related GitHub Issue

closes #298

Screenshots (if appropriate):

Before code change:

After code change:

How Has This Change Been Tested?

Tested with yarn cypress

  • All status checks pass (build, lint, e2e, test)
  • Works on Sepolia preview deployment
  • Works on Mainnet preview deployment

Are Any Admin Tasks Required?

  • No admin tasks

Add Cypress test for `/liquidity` page
@aahna-ashina aahna-ashina self-assigned this Aug 22, 2024
@aahna-ashina aahna-ashina requested a review from a team as a code owner August 22, 2024 04:06
@aahna-ashina aahna-ashina linked an issue Aug 22, 2024 that may be closed by this pull request
Copy link

vercel bot commented Aug 22, 2024

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

Name Status Preview Updated (UTC)
citizen-app-mainnet ✅ Ready (Inspect) Visit Preview Aug 22, 2024 4:14am
citizen-app-sepolia ✅ Ready (Inspect) Visit Preview Aug 22, 2024 4:14am

Copy link

codecov bot commented Aug 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 11.29%. Comparing base (0ca342b) to head (7b363c5).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #299   +/-   ##
=======================================
  Coverage   11.29%   11.29%           
=======================================
  Files          18       18           
  Lines         354      354           
  Branches       76       79    +3     
=======================================
  Hits           40       40           
  Misses        314      314           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aahna-ashina aahna-ashina enabled auto-merge (squash) August 22, 2024 04:19
@aahna-ashina aahna-ashina disabled auto-merge August 22, 2024 04:19
@aahna-ashina aahna-ashina merged commit 43ac1c4 into main Aug 22, 2024
8 checks passed
@aahna-ashina aahna-ashina deleted the 298-add-cypress-test-for-liquidity-page branch August 22, 2024 04:19
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 Cypress test for /liquidity page
1 participant