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(e2e): making the e2e testing suite more reliable with detox and local backend #2971

Closed
wants to merge 6 commits into from

Conversation

sandipndev
Copy link
Member

@sandipndev sandipndev commented Dec 30, 2023

For a very long time now, e2e tests on mobile has been very unreliable and eventually we have gotten into the habit of ignoring the circleci test results.

This PR takes this problem head on. I have found out:

  • Appium is not really ideal and often what happens is that it misses out points (example - double tap to remove keyboard)
  • Appium is not "native"

This is a draft PR at the moment and I have introduced Detox as an alternative to Appium. The detox tests will run on iOS Simulator or Android Emulator and they can just directly be run on CI.

BlueWallet uses Detox: https://github.com/BlueWallet/BlueWallet/blob/master/.github/workflows/ci.yml

@sandipndev sandipndev force-pushed the detox-setup branch 30 times, most recently from 1ed3ec1 to 888b0a0 Compare January 3, 2024 11:12
@sandipndev sandipndev force-pushed the detox-setup branch 6 times, most recently from 96be14a to cf005f1 Compare January 31, 2024 08:58
@sandipndev
Copy link
Member Author

sandipndev commented Feb 2, 2024

With GitHub Actions, the time taken is a lot. That too, the docker environment is not having enough RAM for the Detox test to even able to launch the app. It's not worth it to use GitHub Actions for this because of all this - even if they're free.

Screenshot 2024-02-02 at 11 52 48 AM

@sandipndev sandipndev force-pushed the detox-setup branch 3 times, most recently from feb112f to d1afdd0 Compare February 2, 2024 08:54
@sandipndev sandipndev force-pushed the detox-setup branch 10 times, most recently from 3f1122e to 1ddc91c Compare February 8, 2024 06:35
@sandipndev
Copy link
Member Author

Will split this PR into multiple for better review.

@sandipndev sandipndev closed this Feb 8, 2024
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.

1 participant