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

Refactor Xcode 15.5 upgrade #20497

Closed
wants to merge 9 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
29 changes: 15 additions & 14 deletions bitrise.yml
Original file line number Diff line number Diff line change
Expand Up @@ -65,18 +65,18 @@ workflows:
mkdir DerivedData

xcodebuild -resolvePackageDependencies -onlyUsePackageVersionsFromResolvedFile
xcodebuild "-project" "/Users/vagrant/git/firefox-ios/Client.xcodeproj" "-scheme" "Fennec" -configuration "Fennec" "CODE_SIGNING_ALLOWED=NO" -sdk "iphonesimulator" "-destination" "platform=iOS Simulator,name=iPhone 15,OS=17.4" "COMPILER_INDEX_STORE_ENABLE=NO" "build-for-testing" "CODE_SIGN_IDENTITY=" "CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO" -derivedDataPath "/Users/vagrant/git/DerivedData" | xcpretty | tee xcodebuild_fennec.log
xcodebuild "-project" "/Users/vagrant/git/firefox-ios/Client.xcodeproj" "-scheme" "Fennec" -configuration "Fennec" "CODE_SIGNING_ALLOWED=NO" -sdk "iphonesimulator" "-destination" "platform=iOS Simulator,name=iPhone 15,OS=17.5" "COMPILER_INDEX_STORE_ENABLE=NO" "build-for-testing" "CODE_SIGN_IDENTITY=" "CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO" -derivedDataPath "/Users/vagrant/git/DerivedData" | xcpretty | tee xcodebuild_fennec.log

ls /Users/vagrant/git/DerivedData/Build/Products
ls /Users/vagrant/git/DerivedData
- xcode-test-without-building@0:
run_if: '{{getenv "RUN_UI_TESTS" | eq "Run_UI_Tests"}}'
timeout: 600
inputs:
- destination: platform=iOS Simulator,name=iPhone 15,OS=17.4
- destination: platform=iOS Simulator,name=iPhone 15,OS=17.5
- xcodebuild_options: '"COMPILER_INDEX_STORE_ENABLE=NO"
"CODE_SIGN_IDENTITY=" "CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO"'
- xctestrun: "/Users/vagrant/git/DerivedData/Build/Products/Fennec_UnitTest_iphonesimulator17.4-arm64.xctestrun"
- xctestrun: "/Users/vagrant/git/DerivedData/Build/Products/Fennec_UnitTest_iphonesimulator17.5-arm64.xctestrun"
- maximum_test_repetitions: 2
- save-spm-cache@1:
is_always_run: true
Expand Down Expand Up @@ -215,11 +215,12 @@ workflows:
run_if: '{{getenv "RUN_UI_TESTS" | eq "Run_UI_Tests"}}'
timeout: 1200
inputs:
- destination: platform=iOS Simulator,name=iPhone 15,OS=17.4
- destination: platform=iOS Simulator,name=iPhone 15,OS=17.5
- xcodebuild_options: '"-derivedDataPath" "/Users/vagrant/git/DerivedData"
"COMPILER_INDEX_STORE_ENABLE=NO" "CODE_SIGN_IDENTITY="
"CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO"'
- xctestrun: "Users/vagrant/git/DerivedData/Build/Products/Fennec_Smoketest${SHARD}_iphonesimulator17.4-arm64.xctestrun"
"CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO"
"-parallel-testing-enabled" "YES" "-parallel-testing-worker-count" "2"'
- xctestrun: "Users/vagrant/git/DerivedData/Build/Products/Fennec_Smoketest${SHARD}_iphonesimulator17.5-arm64.xctestrun"
- deploy-to-bitrise-io@2.1.1:
run_if: '{{getenv "RUN_UI_TESTS" | eq "Run_UI_Tests"}}'
inputs:
Expand Down Expand Up @@ -751,7 +752,7 @@ workflows:
run_if: '{{getenv "NEW_XCODE_VERSION" | eq "New_Version_Found" | or (getenv "RUN_ALL_STEPS" | eq "Run_All_Steps")}}'
inputs:
- scheme: Fennec
- destination: platform=iOS Simulator,name=iPhone 15,OS=17.4
- destination: platform=iOS Simulator,name=iPhone 15,OS=17.5
- deploy-to-bitrise-io@2.2: {}
- cache-push@2.4: {}
- slack@3.1:
Expand Down Expand Up @@ -1226,7 +1227,7 @@ workflows:
inputs:
- project_path: firefox-ios/Client.xcodeproj
- scheme: Fennec
- destination: platform=iOS Simulator,name=iPhone 15,OS=17.4
- destination: platform=iOS Simulator,name=iPhone 15,OS=17.5
- test_plan: PerformanceTestPlan
- script@1.1:
is_always_run: true
Expand Down Expand Up @@ -1608,14 +1609,14 @@ workflows:
echo "-- build-for-testing --"
mkdir DerivedData
#xcodebuild -resolvePackageDependencies -onlyUsePackageVersionsFromResolvedFile
xcodebuild "-project" "/Users/vagrant/git/focus-ios/Blockzilla.xcodeproj" "-scheme" "Focus" -configuration "FocusDebug" "CODE_SIGNING_ALLOWED=NO" "-destination" "platform=iOS Simulator,name=iPhone 15,OS=17.4" "COMPILER_INDEX_STORE_ENABLE=NO" "build-for-testing" "CODE_SIGN_IDENTITY=" "CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO" -derivedDataPath "/Users/vagrant/git/DerivedData" | xcpretty | tee xcodebuild_test.log
xcodebuild "-project" "/Users/vagrant/git/focus-ios/Blockzilla.xcodeproj" "-scheme" "Focus" -configuration "FocusDebug" "CODE_SIGNING_ALLOWED=NO" "-destination" "platform=iOS Simulator,name=iPhone 15,OS=17.5" "COMPILER_INDEX_STORE_ENABLE=NO" "build-for-testing" "CODE_SIGN_IDENTITY=" "CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO" -derivedDataPath "/Users/vagrant/git/DerivedData" | xcpretty | tee xcodebuild_test.log
- script@1.1:
title: Build for Testing Klar
inputs:
- content: |
set -euxo pipefail
echo "-- build-for-testing --"
xcodebuild "-project" "/Users/vagrant/git/focus-ios/Blockzilla.xcodeproj" "-scheme" "Klar" -configuration "KlarDebug" "CODE_SIGNING_ALLOWED=NO" "-destination" "platform=iOS Simulator,name=iPhone 15,OS=17.4" "COMPILER_INDEX_STORE_ENABLE=NO" "build-for-testing" "CODE_SIGN_IDENTITY=" "CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO" -derivedDataPath "/Users/vagrant/git/DerivedData" | xcpretty | tee xcodebuild_test.log
xcodebuild "-project" "/Users/vagrant/git/focus-ios/Blockzilla.xcodeproj" "-scheme" "Klar" -configuration "KlarDebug" "CODE_SIGNING_ALLOWED=NO" "-destination" "platform=iOS Simulator,name=iPhone 15,OS=17.5" "COMPILER_INDEX_STORE_ENABLE=NO" "build-for-testing" "CODE_SIGN_IDENTITY=" "CODE_SIGNING_REQUIRED=NO" "CODE_SIGNING_ALLOWED=NO" -derivedDataPath "/Users/vagrant/git/DerivedData" | xcpretty | tee xcodebuild_test.log
- script@1.1:
title: Compress Derived Data
is_always_run: true
Expand Down Expand Up @@ -1683,7 +1684,7 @@ workflows:
mv ./Users/vagrant/git/* .
ls -la
echo "-- test-without-building --"
xcodebuild -resultBundlePath "xcodebuild.xcresult" -derivedDataPath "/Users/vagrant/git/DerivedData" -destination "platform=iOS Simulator,name=iPhone 15,OS=17.4" test-without-building -xctestrun "/Users/vagrant/git/DerivedData/Build/Products/Focus_SmokeTest_iphonesimulator17.4-arm64.xctestrun"
xcodebuild -resultBundlePath "xcodebuild.xcresult" -derivedDataPath "/Users/vagrant/git/DerivedData" -parallel-testing-enabled YES -parallel-testing-worker-count 2 -destination "platform=iOS Simulator,name=iPhone 15,OS=17.5" test-without-building -xctestrun "/Users/vagrant/git/DerivedData/Build/Products/Focus_SmokeTest_iphonesimulator17.5-arm64.xctestrun"
- custom-test-results-export@0:
inputs:
- search_pattern: "$BITRISE_SOURCE_DIR/xcodebuild.xcresult"
Expand Down Expand Up @@ -1714,7 +1715,7 @@ workflows:
mv ./Users/vagrant/git/* .
ls -la
echo "-- test-without-building --"
xcodebuild -resultBundlePath "xcodebuild.xcresult" -derivedDataPath "/Users/vagrant/git/DerivedData" -destination "platform=iOS Simulator,name=iPhone 15,OS=17.4" test-without-building -xctestrun "/Users/vagrant/git/DerivedData/Build/Products/Klar_UnitTests_iphonesimulator17.4-arm64.xctestrun"
xcodebuild -resultBundlePath "xcodebuild.xcresult" -derivedDataPath "/Users/vagrant/git/DerivedData" -destination "platform=iOS Simulator,name=iPhone 15,OS=17.5" test-without-building -xctestrun "/Users/vagrant/git/DerivedData/Build/Products/Klar_UnitTests_iphonesimulator17.5-arm64.xctestrun"
- custom-test-results-export@0:
inputs:
- search_pattern: "$BITRISE_SOURCE_DIR/xcodebuild.xcresult"
Expand Down Expand Up @@ -1742,7 +1743,7 @@ workflows:
mv ./Users/vagrant/git/* .
ls -la
echo "-- test-without-building --"
xcodebuild -resultBundlePath "xcodebuild.xcresult" -derivedDataPath "/Users/vagrant/git/DerivedData" -destination "platform=iOS Simulator,name=iPhone 15,OS=17.4" test-without-building -xctestrun "/Users/vagrant/git/DerivedData/Build/Products/Focus_UnitTests_iphonesimulator17.4-arm64.xctestrun"
xcodebuild -resultBundlePath "xcodebuild.xcresult" -derivedDataPath "/Users/vagrant/git/DerivedData" -destination "platform=iOS Simulator,name=iPhone 15,OS=17.5" test-without-building -xctestrun "/Users/vagrant/git/DerivedData/Build/Products/Focus_UnitTests_iphonesimulator17.5-arm64.xctestrun"
- custom-test-results-export@0:
inputs:
- search_pattern: "$BITRISE_SOURCE_DIR/xcodebuild.xcresult"
Expand Down Expand Up @@ -2145,6 +2146,6 @@ trigger_map:

meta:
bitrise.io:
stack: osx-xcode-15.3.x
stack: osx-xcode-15.4.x
machine_type_id: g2-m1.8core

2 changes: 1 addition & 1 deletion firefox-ios/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ This is the subdirectory that contains the Firefox for iOS application.

## Main branch

Firefox for iOS works with [Xcode 15.3](https://developer.apple.com/download/all/?q=xcode), Swift 5.8 and supports iOS 15.0 and above.
Firefox for iOS works with [Xcode 15.4](https://developer.apple.com/download/all/?q=xcode), Swift 5.8 and supports iOS 15.0 and above.

Please make sure you aim your pull requests in the right direction.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ class L10nSuite2SnapshotTests: L10nBaseSnapshotTests {

func tapKeyboardKey(_ key: Int) {
let key = app.keyboards.keys.element(boundBy: key)
if app.buttons["Continue"].exists == true {
if app.buttons["Continue"].isHittable {
// Attempt to find and tap the Continue button
// of the keyboard onboarding screen.
app.buttons.staticTexts["Continue"].tap()
Expand Down
2 changes: 1 addition & 1 deletion firefox-ios/l10n-screenshots.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ for lang in $LOCALES; do
--skip_open_summary \
--xcargs "-maximum-parallel-testing-workers 2" \
--derived_data_path l10n-screenshots-dd \
--ios_version "17.4" \
--ios_version "17.5" \
--erase_simulator --localize_simulator \
--devices "iPhone 15" --languages "$lang" \
--output_directory "l10n-screenshots/$lang" \
Expand Down
2 changes: 1 addition & 1 deletion focus-ios/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ Download on the [App Store](https://itunes.apple.com/app/id1055677337).

## Main Branch

This branch works with Xcode 15.3 and supports iOS 15.0 and newer.
This branch works with Xcode 15.4 and supports iOS 15.0 and newer.

Pull requests should be submitted with `main` as the base branch.

Expand Down