Run CI tests in Release configuration #3741
Conversation
.circleci/config.yml
Outdated
| @@ -677,19 +680,35 @@ workflows: | |||
| main-workflow: | |||
| jobs: | |||
| - build-job: | |||
| name: "Xcode_12.5.1_iOS_14.5" | |||
| name: "Xcode_12.5.1_iOS_14.5_Debug" | |||
There was a problem hiding this comment.
Should we take this as an opportunity to start using Xcode v13.x for some build jobs?
There was a problem hiding this comment.
Also thought about it, I'll try updating to Xcode 13, if we won't need a lot of changes I'll make it in this PR
There was a problem hiding this comment.
Xcode 13 support is being added in another PR: #3728
azarovalex
force-pushed
the
azarovalex/tests-in-release-configuration
branch
from
395b04e to
6035541
Compare
|
No breaking changes detected in MapboxCoreNavigation |
|
No breaking changes detected in MapboxNavigation |
… as in other tests
| @@ -6,6 +6,7 @@ import MapboxSpeech | |||
| import MapboxNavigation | |||
| import MapboxCommon_Private | |||
|
|
|||
| #if DEBUG | |||
There was a problem hiding this comment.
Could you also add a comment on why we don't run some tests in the Release configuration?
There was a problem hiding this comment.
Because the billing API used in these tests was already wrapped in #if DEBUG, for security reasons.
Run CI jobs on Xcode 13.2.1 and iOS 15.2
|
No breaking changes detected in MapboxCoreNavigation |
|
No breaking changes detected in MapboxNavigation |
Fixes: #3742
This PR adds two new jobs that run tests in Release configuration with SPM and Carthage integrations respectively.
Some tests had to be disabled since they can only run in Debug configuration.