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

fix: fix old signature e2e by mocking launch darkly api #13002

Merged
merged 17 commits into from
Jan 15, 2025
Merged

Conversation

jpuri
Copy link
Contributor

@jpuri jpuri commented Jan 15, 2025

Description

Fix old signature pages e2e. by mocking launch darkly api.

Related issues

Fixes: https://github.com/MetaMask/MetaMask-planning/issues/3916

Manual testing steps

NA

Screenshots/Recordings

NA

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.

@jpuri jpuri requested review from a team as code owners January 15, 2025 14:47
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-confirmations Push issues to confirmations team label Jan 15, 2025
@jpuri jpuri marked this pull request as draft January 15, 2025 14:48
@jpuri jpuri changed the title E2e fix fix: fix old signature e2e by mocking launch darkly api Jan 15, 2025
@jpuri jpuri marked this pull request as ready for review January 15, 2025 17:43
@jpuri jpuri added the Run Smoke E2E Triggers smoke e2e on Bitrise label Jan 15, 2025
Copy link
Contributor

github-actions bot commented Jan 15, 2025

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 2dc7aa9
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/4a7e8198-b918-4d05-99d0-b7fb1d314160

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

@jpuri jpuri removed the Run Smoke E2E Triggers smoke e2e on Bitrise label Jan 15, 2025
@jpuri jpuri added the Run Smoke E2E Triggers smoke e2e on Bitrise label Jan 15, 2025
Copy link
Member

@cortisiko cortisiko left a comment

Choose a reason for hiding this comment

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

Copy link
Contributor

github-actions bot commented Jan 15, 2025

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 1227bb4
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/a84d3039-621d-43ba-91fa-47849e9ea0d8

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

@jpuri
Copy link
Contributor Author

jpuri commented Jan 15, 2025

Seems like https://github.com/MetaMask/metamask-mobile/blob/e2e_fix/e2e/specs/confirmations/signatures/ethereum-sign.spec.js was missed?

Hey @cortisiko : eth sign is deprecated and not covered by new designs.

@jpuri jpuri requested a review from cortisiko January 15, 2025 18:11
@jpuri jpuri added the Run Smoke E2E Triggers smoke e2e on Bitrise label Jan 15, 2025
Copy link
Contributor

github-actions bot commented Jan 15, 2025

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: cf8d4da
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/be58bce9-f0b0-4f55-a188-907565f9d1bb

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

cortisiko
cortisiko previously approved these changes Jan 15, 2025
Copy link
Member

@cortisiko cortisiko left a comment

Choose a reason for hiding this comment

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

Thanks for addressing @jpuri , we will get the test name/variables updated to reflect its true intent

@jpuri jpuri enabled auto-merge January 15, 2025 19:13
digiwand
digiwand previously approved these changes Jan 15, 2025
Copy link
Contributor

@digiwand digiwand left a comment

Choose a reason for hiding this comment

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

discussed w/ @jpuri about removing REDESIGN_SIGNATURE_REQUEST flag from jest.config.js. not a blocker and will be done in a follow-up PR

Lgtm!

@jpuri jpuri added this pull request to the merge queue Jan 15, 2025
@cortisiko cortisiko removed this pull request from the merge queue due to a manual request Jan 15, 2025
@cortisiko
Copy link
Member

wait, it seems like the ios_api_specs workflow is consistently failing on this PR: https://app.bitrise.io/app/be69d4368ee7e86d?branch=e2e_fix can we take a look @jpuri if we merge this is going to break main

My recommendation is trying running the tests locally to get an understanding of what is going on

@jpuri jpuri added this pull request to the merge queue Jan 15, 2025
@jpuri jpuri removed this pull request from the merge queue due to a manual request Jan 15, 2025
@jpuri jpuri dismissed stale reviews from digiwand and cortisiko via 24de5c5 January 15, 2025 19:55
@jpuri jpuri added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Jan 15, 2025
@jpuri jpuri requested review from cortisiko and digiwand January 15, 2025 19:57
Copy link
Contributor

github-actions bot commented Jan 15, 2025

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 24de5c5
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/62915f2a-c64c-4850-8d7b-aba93c7c0667

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

@cortisiko cortisiko enabled auto-merge January 15, 2025 20:30
@cortisiko cortisiko added this pull request to the merge queue Jan 15, 2025
Merged via the queue into main with commit 1f2c1a2 Jan 15, 2025
36 of 37 checks passed
@cortisiko cortisiko deleted the e2e_fix branch January 15, 2025 23:10
@github-actions github-actions bot locked and limited conversation to collaborators Jan 15, 2025
@metamaskbot metamaskbot added the release-7.39.0 Issue or pull request that will be included in release 7.39.0 label Jan 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-7.39.0 Issue or pull request that will be included in release 7.39.0 Run Smoke E2E Triggers smoke e2e on Bitrise team-confirmations Push issues to confirmations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants