Skip to content

Commit 6834c5b

Browse files
committed
build(e2e, ios): integrate xcbeautify, with correct compile failure status report
the current xcode build is still very noisy previously there was an issue with combination of xcpretty and CI, such that compile failures were not detected and failing CI. xcpretty is abandoned but xcbeautify lives. The built-in `-quiet` is too quiet. I believe this combo of set -o pipefail and xcbeautify should do it. It passes local negative- and positive-testing on build status reporting
1 parent ba031b2 commit 6834c5b

File tree

3 files changed

+12
-10
lines changed

3 files changed

+12
-10
lines changed

.github/workflows/tests_e2e_ios.yml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,14 @@ jobs:
120120
- name: Start Firestore Emulator
121121
run: yarn tests:emulator:start-ci
122122

123+
- name: Install brew utilities
124+
uses: nick-invision/retry@v2
125+
with:
126+
timeout_minutes: 5
127+
retry_wait_seconds: 60
128+
max_attempts: 3
129+
command: HOMEBREW_NO_AUTO_UPDATE=1 brew tap wix/brew && HOMEBREW_NO_AUTO_UPDATE=1 brew install applesimutils xcbeautify && applesimutils --list
130+
123131
- name: Build iOS App
124132
run: |
125133
export PATH="/usr/lib/ccache:/usr/local/opt/ccache/libexec:$PATH"
@@ -135,14 +143,6 @@ jobs:
135143
ccache -s
136144
shell: bash
137145

138-
- name: Install applesimutils
139-
uses: nick-invision/retry@v2
140-
with:
141-
timeout_minutes: 5
142-
retry_wait_seconds: 60
143-
max_attempts: 3
144-
command: HOMEBREW_NO_AUTO_UPDATE=1 brew tap wix/brew && HOMEBREW_NO_AUTO_UPDATE=1 brew install applesimutils && applesimutils --list
145-
146146
- name: Metro Bundler Cache
147147
uses: actions/cache@v2
148148
with:

CONTRIBUTING.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,8 @@ cd react-native-firebase
101101
```bash
102102
yarn
103103
yarn tests:ios:pod:install
104+
brew tap wix/brew
105+
brew install applesimutils xbeautify
104106
```
105107

106108
> Note that this project is a mono-repo, so you only need to install NPM dependencies once at the root of the project with `yarn`.

tests/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,15 +58,15 @@
5858
"configurations": {
5959
"ios.sim.debug": {
6060
"binaryPath": "ios/build/Build/Products/Debug-iphonesimulator/testing.app",
61-
"build": "xcodebuild VALID_ARCHS=\"`uname -m`\" CC=clang CPLUSPLUS=clang++ LD=clang LDPLUSPLUS=clang++ -workspace ios/testing.xcworkspace -scheme testing -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build",
61+
"build": "set -o pipefail && xcodebuild VALID_ARCHS=\"`uname -m`\" CC=clang CPLUSPLUS=clang++ LD=clang LDPLUSPLUS=clang++ -workspace ios/testing.xcworkspace -scheme testing -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build | xcbeautify",
6262
"type": "ios.simulator",
6363
"device": {
6464
"type": "iPhone 11"
6565
}
6666
},
6767
"ios.sim.release": {
6868
"binaryPath": "ios/build/Build/Products/Release-iphonesimulator/testing.app",
69-
"build": "export RCT_NO_LAUNCH_PACKAGER=true && xcodebuild CC=clang CPLUSPLUS=clang++ LD=clang LDPLUSPLUS=clang++ -workspace ios/testing.xcworkspace -scheme testing -configuration Release -sdk iphonesimulator -derivedDataPath ios/build",
69+
"build": "export RCT_NO_LAUNCH_PACKAGER=true && set -o pipefail | xcodebuild CC=clang CPLUSPLUS=clang++ LD=clang LDPLUSPLUS=clang++ -workspace ios/testing.xcworkspace -scheme testing -configuration Release -sdk iphonesimulator -derivedDataPath ios/build | xcbeautify",
7070
"type": "ios.simulator",
7171
"device": {
7272
"type": "iPhone 11"

0 commit comments

Comments
 (0)