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: Fix detox test run inconsistencies #10055

Merged
merged 6 commits into from
Jun 21, 2024

Conversation

SamuelSalas
Copy link
Member

Description

Lately, some of the test are failing both locally and on Bitrise. Most of this errors are regarding some inconsistencies found on the viewHelper.js file. Steps closing onboarding modals are being call multiple times. So the best solution would be unifying all these steps in just one method, so any change to modals can be just change in one place and affect all test. As well, we need to fix some assertions parameters that are not following the correct definitions from the jsdocs and update the deprecated methods with their current implementations.

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Smoke test run: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/9129dcc3-e46d-4572-b627-9f90c3feab96
Regression Test run:

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.

Copy link

sonarcloud bot commented Jun 21, 2024

@SamuelSalas
Copy link
Member Author

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

@SamuelSalas SamuelSalas merged commit 3a6f656 into main Jun 21, 2024
31 checks passed
@SamuelSalas SamuelSalas deleted the test/1840-fix-detox-test-run-inconsistencies branch June 21, 2024 18:20
@github-actions github-actions bot locked and limited conversation to collaborators Jun 21, 2024
@metamaskbot metamaskbot added release-7.27.0 Issue or pull request that will be included in release 7.27.0 release-7.26.0 Issue or pull request that will be included in release 7.26.0 and removed release-7.27.0 Issue or pull request that will be included in release 7.27.0 labels Jun 21, 2024
@metamaskbot
Copy link
Collaborator

Missing release label release-7.26.0 on PR. Adding release label release-7.26.0 on PR and removing other release labels(release-7.27.0), as PR was cherry-picked in branch 7.26.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-7.26.0 Issue or pull request that will be included in release 7.26.0 team-mobile-platform
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants