Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Fix Detox tests after upgrading to latest CLI (facebook#23191)
Summary: Latest changes inside CLI now require that Metro configuration is provided when building RNTester app. This is to let CLI know that instead of looking for "react-native" under "node_modules" (that is obviously not present since we are running from source), it should check the paths provided. When running "npm start", it finds the configuration at the root. However, when building through Xcode (e.g. "react-native bundle" or "xcodebuild" in Release scheme), it runs "react-native-xcode.sh" that works in different folder and makes Metro not detect the configuration file. This PR explicitly sets path to the configuration via `BUNDLE_CONFIG`. It also removes `pwd` from being prepended to all `BUNDLE_CONFIG` values. In my case, `pwd` was `/Users/grabbou` and the RNTester files where inside `/Users/grabbou/Repositories/react-native`. I was unable to point the script to correct location without making it aware of the folder structure - which is not going to work on the CI. Pull Request resolved: facebook#23191 Differential Revision: D13851741 Pulled By: hramos fbshipit-source-id: d920353fd68d39468bd33bd1ad47e03b017a7727
- Loading branch information