Skip to content

test: Add Stake e2e automation #13180

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

Merged
merged 37 commits into from
Feb 6, 2025
Merged

test: Add Stake e2e automation #13180

merged 37 commits into from
Feb 6, 2025

Conversation

davibroc
Copy link
Contributor

@davibroc davibroc commented Jan 24, 2025

Description

These are the Stake flows this PR will automate:

  • Stake
  • Unstake
  • Stake more
  • Claim
  • Making sure staking balance, banners, and actions are hidden for ETH assets that aren't on mainnet or holesky

To run them locally follow these steps:

  • Build the app locally and make sure you set the following ENV variables:
    export MM_MULTICHAIN_V1_ENABLED="true"
    export MM_CHAIN_PERMISSIONS="true"
    export MM_PERMISSIONS_SETTINGS_V1_ENABLED="false"
    export PORTFOLIO_VIEW="true"
    export MM_SECURITY_ALERTS_API_ENABLED="true"
    export MM_NETWORK_UI_REDESIGN_ENABLED= "false"

    and run yarn test:e2e:android:debug:build

  • Run the following command yarn test:e2e:android:debug:run e2e/specs/stake/stake-action-smoke.spec.js

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@davibroc davibroc requested review from a team as code owners January 24, 2025 20:23
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot metamaskbot added the team-qa QA team label Jan 24, 2025
@davibroc davibroc added team-stake Run Smoke E2E Requires smoke E2E testing labels Jan 28, 2025
Copy link
Contributor

github-actions bot commented Jan 28, 2025

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: ae1c314
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/cd7cbe7b-104b-47cd-b67e-c2e84ca14a81

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@davibroc davibroc force-pushed the test/stake-e2e-automation branch 2 times, most recently from 6e8a6dc to 3084376 Compare February 3, 2025 05:41
@davibroc davibroc added Run Smoke E2E Requires smoke E2E testing and removed Run Smoke E2E Requires smoke E2E testing labels Feb 3, 2025
Copy link
Contributor

github-actions bot commented Feb 3, 2025

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 696d3aa
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/16d880bd-4b7d-4a7e-a6f3-515eb0f28462

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@davibroc davibroc added Run Smoke E2E Requires smoke E2E testing and removed Run Smoke E2E Requires smoke E2E testing labels Feb 4, 2025
Copy link
Contributor

github-actions bot commented Feb 4, 2025

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 3ad8b72
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/7ebc9c24-78c2-4c2d-9ac9-3c95e6d5e448

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Copy link
Contributor

@amitabh94 amitabh94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor comment but overall LGTM!

@davibroc davibroc force-pushed the test/stake-e2e-automation branch from 81741ee to 33e519d Compare February 6, 2025 00:50
@davibroc davibroc added Run Smoke E2E Requires smoke E2E testing and removed Run Smoke E2E Requires smoke E2E testing labels Feb 6, 2025
Copy link
Contributor

github-actions bot commented Feb 6, 2025

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 33e519d
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/fd48405e-2827-4796-95df-29996ca64c5d

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

Copy link

sonarqubecloud bot commented Feb 6, 2025

@davibroc davibroc added Run Smoke E2E Requires smoke E2E testing and removed Run Smoke E2E Requires smoke E2E testing labels Feb 6, 2025
Copy link
Contributor

@amitabh94 amitabh94 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Contributor

@chrisleewilcox chrisleewilcox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@davibroc davibroc added this pull request to the merge queue Feb 6, 2025
Merged via the queue into main with commit 832a4f9 Feb 6, 2025
45 of 47 checks passed
@davibroc davibroc deleted the test/stake-e2e-automation branch February 6, 2025 17:50
@github-actions github-actions bot locked and limited conversation to collaborators Feb 6, 2025
@metamaskbot metamaskbot added the release-7.41.0 Issue or pull request that will be included in release 7.41.0 label Feb 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-7.41.0 Issue or pull request that will be included in release 7.41.0 Run Smoke E2E Requires smoke E2E testing team-qa QA team team-stake
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants