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

Detox e2e tests #182

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Detox e2e tests #182

wants to merge 3 commits into from

Conversation

alexrisch
Copy link
Contributor

@alexrisch alexrisch commented Dec 15, 2023

Setup Detox
Added documentation for detox
Added testIds
Added setup tests for Launch Screen, Home Screen, Conversation Screen Added helper utils for tests

Running:
detox build --configuration ios.sim.debug
detox test --configuration ios.sim.debug
or
detox build --configuration android.emu.debug
detox test --configuration android.emu.debug

@alexrisch alexrisch force-pushed the user/alexrisch/detox-setup branch from 69f1289 to 38bec7e Compare December 21, 2023 02:42
@alexrisch alexrisch force-pushed the user/alexrisch/detox-setup branch from 38bec7e to 4484049 Compare February 15, 2024 15:30
@alexrisch alexrisch marked this pull request as ready for review February 15, 2024 15:30
@alexrisch alexrisch requested review from humanagent and a team as code owners February 15, 2024 15:30
Copy link
Contributor

@nplasterer nplasterer left a comment

Choose a reason for hiding this comment

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

If you're removing the MainActivityTest than can you update the GitHub action for the android test to run the detox test instead? or at the very least the github action should be removed. https://github.com/xmtp/xmtp-react-native/blob/main/.github/workflows/androidTests.yml

@alexrisch
Copy link
Contributor Author

Closes #162

@alexrisch alexrisch force-pushed the user/alexrisch/detox-setup branch from 4484049 to d80e792 Compare March 2, 2024 01:48
Alex Risch added 3 commits March 15, 2024 16:44
Merged main changes
Updated Android CI Tests
Setup Detox
Added documentation for detox
Added testIds
Added setup tests for Launch Screen, Home Screen, Conversation Screen Added helper utils for tests

Draft PR while finalizing Android Setup
Corrected Android build
@alexrisch alexrisch force-pushed the user/alexrisch/detox-setup branch from d80e792 to 4350985 Compare March 15, 2024 23:00
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.

2 participants