Skip to content

Commit 6035541

Browse files
committed
Run CI tests in Release configuration as well
1 parent 7cefa20 commit 6035541

File tree

4 files changed

+30
-9
lines changed

4 files changed

+30
-9
lines changed

.circleci/config.yml

Lines changed: 23 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -340,6 +340,9 @@ jobs:
340340
verify_localization:
341341
type: boolean
342342
default: true
343+
configuration:
344+
type: string
345+
default: "Debug"
343346
macos:
344347
xcode: << parameters.xcode >>
345348
environment:
@@ -374,7 +377,7 @@ jobs:
374377
xcode: << parameters.xcode >>
375378
- run:
376379
name: MapboxCoreNavigation
377-
command: xcodebuild -sdk iphonesimulator -destination 'platform=iOS Simulator,OS=<< parameters.iOS >>,name=<< parameters.device >>' -project MapboxNavigation.xcodeproj -scheme MapboxCoreNavigation clean build <<# parameters.test >>test <</ parameters.test >> <<# parameters.codecoverage >>-enableCodeCoverage YES<</ parameters.codecoverage >>
380+
command: xcodebuild -sdk iphonesimulator -destination 'platform=iOS Simulator,OS=<< parameters.iOS >>,name=<< parameters.device >>' -project MapboxNavigation.xcodeproj -scheme MapboxCoreNavigation -configuration << parameters.configuration >> clean build <<# parameters.test >>test <</ parameters.test >> <<# parameters.codecoverage >>-enableCodeCoverage YES<</ parameters.codecoverage >> ENABLE_TESTABILITY=YES
378381
- when:
379382
condition: << parameters.spm >>
380383
steps:
@@ -383,7 +386,7 @@ jobs:
383386
command: rm -rf *.xcodeproj
384387
- run:
385388
name: MapboxNavigation-Package
386-
command: xcodebuild -sdk iphonesimulator -destination 'platform=iOS Simulator,OS=<< parameters.iOS >>,name=<< parameters.device >>' -scheme MapboxNavigation-Package build <<# parameters.test >>test <</ parameters.test >> <<# parameters.codecoverage >>-enableCodeCoverage YES<</ parameters.codecoverage >>
389+
command: xcodebuild -sdk iphonesimulator -destination 'platform=iOS Simulator,OS=<< parameters.iOS >>,name=<< parameters.device >>' -scheme MapboxNavigation-Package -configuration << parameters.configuration >> build <<# parameters.test >>test <</ parameters.test >> <<# parameters.codecoverage >>-enableCodeCoverage YES<</ parameters.codecoverage >> ENABLE_TESTABILITY=YES
387390
- when:
388391
condition: << parameters.codecoverage >>
389392
steps:
@@ -677,19 +680,35 @@ workflows:
677680
main-workflow:
678681
jobs:
679682
- build-job:
680-
name: "Xcode_12.5.1_iOS_14.5"
683+
name: "Xcode_12.5.1_iOS_14.5_Debug"
684+
xcode: "12.5.1"
685+
iOS: "14.5"
686+
device: "iPhone 12 Pro Max"
687+
context: Slack Orb
688+
- build-job:
689+
name: "Xcode_12.5.1_iOS_14.5_SPM_Debug"
690+
xcode: "12.5.1"
691+
iOS: "14.5"
692+
device: "iPhone 12 Pro Max"
693+
spm: true
694+
codecoverage: false
695+
context: Slack Orb
696+
- build-job:
697+
name: "Xcode_12.5.1_iOS_14.5_Release"
681698
xcode: "12.5.1"
682699
iOS: "14.5"
683700
device: "iPhone 12 Pro Max"
684701
context: Slack Orb
702+
configuration: "Release"
685703
- build-job:
686-
name: "Xcode_12.5.1_iOS_14.5_SPM"
704+
name: "Xcode_12.5.1_iOS_14.5_SPM_Release"
687705
xcode: "12.5.1"
688706
iOS: "14.5"
689707
device: "iPhone 12 Pro Max"
690708
spm: true
691709
codecoverage: false
692710
context: Slack Orb
711+
configuration: "Release"
693712
- pod-job:
694713
name: "Xcode_12.5_iOS_14.5_CP_install"
695714
update: false
@@ -720,7 +739,6 @@ workflows:
720739
filters:
721740
tags:
722741
only: /^v\d+\.\d+\.\d+(-.+)?$/
723-
724742
- ios-trigger-metrics:
725743
filters:
726744
branches:

Tests/MapboxNavigationTests/NavigationCameraTests.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -606,7 +606,7 @@ class NavigationCameraTests: TestCase {
606606
XCTAssertFalse(animatorPitch.isRunning, "Pitch animator should not be running.")
607607
}
608608

609-
#if arch(x86_64)
609+
#if arch(x86_64) && DEBUG
610610
func testNavigationCameraFollowingCameraOptionsZoomRanges() {
611611
let navigationMapView = NavigationMapView(frame: .zero)
612612
guard let navigationViewportDataSource = navigationMapView.navigationCamera.viewportDataSource as? NavigationViewportDataSource else {
@@ -628,7 +628,7 @@ class NavigationCameraTests: TestCase {
628628
}
629629
#endif
630630

631-
#if arch(x86_64)
631+
#if arch(x86_64) && DEBUG
632632
// NOTE: We are running this test only on arch(x86_64) because `throwAssertion` doesn't work on ARM chips.
633633
func testNavigationCameraOverviewCameraOptionsMaximumZoomLevel() {
634634
let navigationMapView = NavigationMapView(frame: .zero)

Tests/MapboxNavigationTests/SKUTests.swift

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import MapboxSpeech
66
import MapboxNavigation
77
import MapboxCommon_Private
88

9+
#if DEBUG
910
class SKUTests: TestCase {
1011
private var navigator: MapboxCoreNavigation.Navigator? = nil
1112

@@ -18,7 +19,7 @@ class SKUTests: TestCase {
1819
super.tearDown()
1920
navigator = nil
2021
}
21-
22+
2223
func testDirectionsSKU() {
2324
let expected: String = UUID().uuidString
2425
billingServiceMock.onGetSKUTokenIfValid = { _ in
@@ -66,3 +67,4 @@ class SKUTests: TestCase {
6667
XCTAssertEqual(viewController.directionsToken, skuToken)
6768
}
6869
}
70+
#endif

Tests/MapboxNavigationTests/TokenTestViewController.swift

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ import MapboxNavigation
66
import MapboxDirections
77
import MapboxSpeech
88

9-
9+
#if DEBUG
1010
class TokenTestViewController: UIViewController {
1111

1212
var mapViewToken: String?
@@ -61,3 +61,4 @@ class TokenTestViewController: UIViewController {
6161
}
6262
}
6363
}
64+
#endif

0 commit comments

Comments
 (0)