Skip to content

Commit 8861ea4

Browse files
committed
build(tests, ios): unify iOS debug build target, remove build filters
react-native 0.64 includes "CodeGen" on the iOS platform, and it was not running reliably locally for iOS tests (`yarn tests:ios:build`) while xcpretty and quiet were in use, not sure why. Removing them means it runs reliably though, so this removes them also devs should use what CI uses philosophically, so unify those targets
1 parent 2378d75 commit 8861ea4

File tree

2 files changed

+7
-15
lines changed

2 files changed

+7
-15
lines changed

.github/workflows/tests_e2e.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,7 @@ jobs:
301301
export RCT_NO_LAUNCH_PACKAGER=1
302302
cd tests
303303
set -o pipefail
304-
./node_modules/.bin/detox build --configuration ios.ci
304+
./node_modules/.bin/detox build --configuration ios.sim.debug
305305
shell: bash
306306

307307
- name: Install applesimutils
@@ -334,7 +334,7 @@ jobs:
334334
timeout-minutes: 40
335335
run: |
336336
cd tests
337-
./node_modules/.bin/nyc ./node_modules/.bin/detox test --loglevel trace --debug-synchronization 200 --configuration ios.ci --cleanup
337+
./node_modules/.bin/nyc ./node_modules/.bin/detox test --loglevel trace --debug-synchronization 200 --configuration ios.sim.debug --cleanup
338338
339339
- name: Compress Simulator Log
340340
if: always()

tests/package.json

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -52,15 +52,15 @@
5252
"configurations": {
5353
"ios.sim.debug": {
5454
"binaryPath": "ios/build/Build/Products/Debug-iphonesimulator/testing.app",
55-
"build": "set -o pipefail && xcodebuild -workspace ios/testing.xcworkspace -scheme testing -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build -UseModernBuildSystem=YES -quiet | xcpretty -k",
55+
"build": "xcodebuild -workspace ios/testing.xcworkspace -scheme testing -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build -UseModernBuildSystem=YES",
5656
"type": "ios.simulator",
5757
"device": {
58-
"type": "iPhone 8"
58+
"type": "iPhone 11"
5959
}
6060
},
61-
"ios.ci": {
62-
"binaryPath": "ios/build/Build/Products/Debug-iphonesimulator/testing.app",
63-
"build": "set -o pipefail && xcodebuild -workspace ios/testing.xcworkspace -scheme testing -configuration Debug -sdk iphonesimulator -derivedDataPath ios/build -UseModernBuildSystem=YES | xcpretty -k",
61+
"ios.sim.release": {
62+
"binaryPath": "ios/build/Build/Products/Release-iphonesimulator/testing.app",
63+
"build": "export RCT_NO_LAUNCH_PACKAGER=true && xcodebuild -workspace ios/testing.xcworkspace -scheme testing -configuration Release -sdk iphonesimulator -derivedDataPath ios/build -UseModernBuildSystem=YES",
6464
"type": "ios.simulator",
6565
"device": {
6666
"type": "iPhone 11"
@@ -74,14 +74,6 @@
7474
"avdName": "TestingAVD"
7575
}
7676
},
77-
"ios.sim.release": {
78-
"binaryPath": "ios/build/Build/Products/Release-iphonesimulator/testing.app",
79-
"build": "set -o pipefail && export RCT_NO_LAUNCH_PACKAGER=true && xcodebuild -workspace ios/testing.xcworkspace -scheme testing -configuration Release -sdk iphonesimulator -derivedDataPath ios/build -UseModernBuildSystem=YES -quiet | xcpretty -k",
80-
"type": "ios.simulator",
81-
"device": {
82-
"type": "iPhone X"
83-
}
84-
},
8577
"android.emu.release": {
8678
"binaryPath": "android/app/build/outputs/apk/release/app-release.apk",
8779
"build": "cd android && ./gradlew assembleRelease assembleAndroidTest -DtestBuildType=release && cd ..",

0 commit comments

Comments
 (0)