[E2E] Add Detox tests#4182
Open
m-bert wants to merge 112 commits into
Open
Conversation
…ied pinch gesture boundary check
nd versioned docs to also not rely on the SrcFiles
…re-handler merge main
m-bert
commented
May 20, 2026
Collaborator
Author
There was a problem hiding this comment.
Needed because yarn suddenly stopped hoisting react-native
m-bert
commented
May 20, 2026
Collaborator
Author
There was a problem hiding this comment.
Changed to match root
m-bert
commented
May 22, 2026
Collaborator
Author
There was a problem hiding this comment.
Note that I've tested only release configuration. I'll leave the other to be tested later
Comment on lines
+68
to
+72
| description: 'androidTestImplementation for Detox', | ||
| find: ` if (hermesEnabled.toBoolean()) {`, | ||
| replace: ` androidTestImplementation('com.wix:detox:+') | ||
|
|
||
| if (hermesEnabled.toBoolean()) {`, |
Collaborator
Author
There was a problem hiding this comment.
I don't think we have to do this for now
Comment on lines
+158
to
+164
| '<?xml version="1.0" encoding="utf-8"?>', | ||
| '<network-security-config>', | ||
| ' <domain-config cleartextTrafficPermitted="true">', | ||
| ' <domain includeSubdomains="true">10.0.2.2</domain>', | ||
| ' <domain includeSubdomains="true">localhost</domain>', | ||
| ' </domain-config>', | ||
| '</network-security-config>', |
Collaborator
Author
There was a problem hiding this comment.
This one comes from detox docs
Also, this is for tests
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…-gesture-handler into @mbert/e2e
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Huge thanks to @relextm19 for investigating this problem and helping with setup ❤️
Description
This PR adds Detox E2E tests to our example app. Sources for examples are placed in
common-app/e2e_screens, tests are located inexpo-example/e2e. This PR adds only one simple test forPangesture, more tests will be added in the follow-ups.Tests are also run on CI when pushing to
mainor modifyinge2esources mentioned above.Since our example app uses
expoprebuilds, e2e setup requires additional script on android that will configure app for tests.Important
You may need to adjust device name in
.detoxrc.jsfile.Running E2E
This PR also adds few yarn scripts:
yarn build-detox-android(also runs preparation script for android)yarn build-detox-iosyarn build-detox(combines both of the above)To run tests you can use the following command:
yarn detox test -c ios.sim.release <test>yarn detox test -c android.emu.release <test>yarn test:iosandyarn test:androidare added, in the future they'll run all tests. For now they just runPantest.Note
For now I've only tested release build, but other configurations are left to be checked
Test plan
Well well well...
yarn build-detoxyarn test:iosyarn test:android