[go: up one dir, main page]

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 Detox failure for new ignited projects #1808

Merged
merged 2 commits into from
Oct 22, 2021
Merged

Conversation

derekgreenberg
Copy link
Member
@derekgreenberg derekgreenberg commented Oct 14, 2021

Please verify the following:

  • [ x ] yarn ci:test jest tests pass with new tests, if relevant
  • [ x ] README.md has been updated with your changes, if relevant

Describe your PR

This fixes a current issue whereby running e2e via Detox failed because the default Detox config did NOT specify which OS of the iPhone 11 device is used for e2e tests. The change is to package.json. Instead of merely specifying the name of the iOS simulated device, both the name and OS version are specified. The OS version in this case must be less than 15 in order to avoid the cause of Detox tests failing on new ignited projects.

To provide more information about how users can configure the Detox settings in package.json according to their environment, the README file in the e2e directory has been updated with information about how to find the supported versions of simulators installed in their environments via Xcode itself. This will help future users modify the Detox configuration settings in the future.

Fixes #1804.

Copy link
Member
@jamonholmgren jamonholmgren left a comment

Choose a reason for hiding this comment

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

This is great, thank you @derekgreenberg!

@jamonholmgren jamonholmgren merged commit 13d9bbb into master Oct 22, 2021
@jamonholmgren jamonholmgren deleted the fix-detox-failures branch October 22, 2021 17:24
infinitered-circleci pushed a commit that referenced this pull request Oct 22, 2021
# [7.6.0](v7.5.0...v7.6.0) (2021-10-22)

### Bug Fixes

* **generators:** Update NAME.tsx.ejs to avoid require cycle  ([#1794](#1794) by [@anhtuank7c](https://github.com/anhtuank7c)) ([24d149c](24d149c))
* **tests:** Fix Detox failure for new ignited projects ([#1808](#1808) by [@derekgreenberg](https://github.com/derekgreenberg) and [@glesperance](https://github.com/glesperance)) ([13d9bbb](13d9bbb))

### Features

* **boilerplate:** Update iOS Deployment Target to 11.0 ([#1788](#1788) by @Andrewngabriel) ([e39d3cb](e39d3cb))
@infinitered-circleci
Copy link

🎉 This PR is included in version 7.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

End-to-end tests fail on iOS 15.0 with a freshly scaffolded app
3 participants