Skip to content

Commit

Permalink
Xcode 15.3: fix warnings on Integration Tests (#3603)
Browse files Browse the repository at this point in the history
Follow up to #3599 and #3600.

These `@MainActor` annotations don't work anymore because of this:
> Main actor-isolated class '...' has different actor isolation from
nonisolated superclass 'TestCase'; this is an error in Swift 6

They were necessary with an older version of Xcode (likely 13.x), but
not anymore.

---------

Co-authored-by: Mark Villacampa <m@markvillacampa.com>
  • Loading branch information
NachoSoto and MarkVillacampa committed Mar 4, 2024
1 parent da11a79 commit 0bb44cc
Show file tree
Hide file tree
Showing 14 changed files with 154 additions and 87 deletions.
64 changes: 32 additions & 32 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ aliases:
branches:
only: main
non-patch-release-branches: &non-patch-release-branches
filters:
filters:
tags:
ignore: /.*/
branches:
Expand Down Expand Up @@ -99,7 +99,7 @@ commands:
key: v2-gem-cache-{{ checksum "Gemfile.lock" }}-{{ arch }}
paths:
- vendor/bundle

install-dependencies:
parameters:
directory:
Expand All @@ -122,7 +122,7 @@ commands:
- /usr/local/Cellar/swiftlint/
- /usr/local/Cellar/xcbeautify/
- /Users/$USER/Library/Caches/Homebrew/

install-brew-dependency:
parameters:
dependency_name:
Expand Down Expand Up @@ -308,7 +308,7 @@ commands:
command: bundle exec fastlane backend_integration_tests test_plan:"<< parameters.test_plan >>"
no_output_timeout: 5m
environment:
SCAN_DEVICE: iPhone 14 (17.2.0)
SCAN_DEVICE: iPhone 14 (17.4.0)
- compress_result_bundle:
directory: fastlane/test_output/xctest/ios
bundle_name: BackendIntegrationTests
Expand Down Expand Up @@ -359,7 +359,7 @@ jobs:
name: SPM Receipt Parser
command: swift build -c release --target ReceiptParser
no_output_timeout: 30m

spm-xcode-14-1:
<<: *base-job
steps:
Expand Down Expand Up @@ -427,7 +427,7 @@ jobs:
path: fastlane/test_output
destination: scan-test-output
- run:
condition:
condition:
- not: << pipeline.parameters.generate_revenuecatui_snapshots >>
name: RevenueCatUI API Tests
command: bundle exec fastlane build_revenuecatui_api_tester
Expand All @@ -447,7 +447,7 @@ jobs:
command: bundle exec fastlane test_revenuecatui
no_output_timeout: 15m
environment:
SCAN_DEVICE: iPhone 15 Pro,OS=17.2
SCAN_DEVICE: iPhone 15 Pro,OS=17.4
- create-snapshot-pr-if-needed:
condition: << pipeline.parameters.generate_revenuecatui_snapshots >>
job: "create_snapshots_repo_pr"
Expand All @@ -460,7 +460,7 @@ jobs:
- store_artifacts:
path: fastlane/test_output
destination: scan-test-output

spm-revenuecat-ui-watchos:
<<: *base-job
steps:
Expand All @@ -483,7 +483,7 @@ jobs:
- store_artifacts:
path: fastlane/test_output
destination: scan-test-output

run-test-macos:
<<: *base-job
steps:
Expand All @@ -505,7 +505,7 @@ jobs:
- store_artifacts:
path: fastlane/test_output/xctest
destination: scan-test-output

run-test-ios-17:
<<: *base-job
steps:
Expand All @@ -516,7 +516,7 @@ jobs:
command: bundle exec fastlane test_ios
no_output_timeout: 15m
environment:
SCAN_DEVICE: iPhone 15 (17.2.0)
SCAN_DEVICE: iPhone 15 (17.4.0)
- compress_result_bundle:
directory: fastlane/test_output/xctest/ios
bundle_name: RevenueCat
Expand Down Expand Up @@ -992,7 +992,7 @@ jobs:
command: bundle exec fastlane backend_integration_tests test_plan:"BackendIntegrationTests-All-CI"
no_output_timeout: 5m
environment:
SCAN_DEVICE: iPhone 14 (17.2.0)
SCAN_DEVICE: iPhone 14 (17.4.0)
- compress_result_bundle:
directory: fastlane/test_output/xctest/ios
bundle_name: BackendIntegrationTests
Expand All @@ -1016,7 +1016,7 @@ jobs:
- compress_result_bundle:
directory: fastlane/test_output/xctest/ios
bundle_name: v3LoadShedderIntegration

deploy-purchase-tester:
<<: *base-job
steps:
Expand Down Expand Up @@ -1083,7 +1083,7 @@ workflows:
when: << pipeline.parameters.generate_snapshots >>
jobs:
- run-test-ios-17:
xcode_version: '15.2'
xcode_version: '15.3'
- run-test-ios-16:
xcode_version: '14.3.0'
- run-test-ios-15:
Expand All @@ -1095,7 +1095,7 @@ workflows:
- run-test-ios-12:
xcode_version: '14.2.0'
- run-test-macos:
xcode_version: '15.2'
xcode_version: '15.3'

generate_revenuecatui_snapshots:
when: << pipeline.parameters.generate_revenuecatui_snapshots >>
Expand All @@ -1105,8 +1105,8 @@ workflows:
- spm-revenuecat-ui-ios-16:
xcode_version: '14.3.0'
- spm-revenuecat-ui-ios-17:
xcode_version: '15.2'
xcode_version: '15.3'

build-test:
when:
and:
Expand All @@ -1120,7 +1120,7 @@ workflows:
- spm-release-build-xcode-14:
xcode_version: '14.3.0'
- spm-release-build-xcode-15:
xcode_version: '15.2'
xcode_version: '15.3'
- spm-xcode-14-1:
xcode_version: '14.1.0'
- spm-custom-entitlement-computation-build:
Expand All @@ -1132,13 +1132,13 @@ workflows:
- spm-revenuecat-ui-ios-16:
xcode_version: '14.3.0'
- spm-revenuecat-ui-ios-17:
xcode_version: '15.2'
xcode_version: '15.3'
- spm-revenuecat-ui-watchos:
xcode_version: '14.3.0'
- run-test-macos:
xcode_version: '15.2'
xcode_version: '15.3'
- run-test-ios-17:
xcode_version: '15.2'
xcode_version: '15.3'
- run-test-ios-16:
xcode_version: '14.3.0'
- run-test-ios-15:
Expand All @@ -1160,31 +1160,31 @@ workflows:
- build-tv-watch-and-macos:
xcode_version: '14.3.0'
- build-visionos:
xcode_version: '15.2'
xcode_version: '15.3'
- backend-integration-tests-SK1:
xcode_version: '15.2'
xcode_version: '15.3'
filters:
branches:
# Forked pull requests have CIRCLE_BRANCH set to pull/XXX
ignore: /pull\/[0-9]+/
- backend-integration-tests-SK2:
xcode_version: '15.2'
xcode_version: '15.3'
filters:
branches:
# Forked pull requests have CIRCLE_BRANCH set to pull/XXX
ignore: /pull\/[0-9]+/
- backend-integration-tests-other:
xcode_version: '15.2'
xcode_version: '15.3'
filters:
branches:
# Forked pull requests have CIRCLE_BRANCH set to pull/XXX
ignore: /pull\/[0-9]+/
- backend-integration-tests-offline:
xcode_version: '15.2'
xcode_version: '15.3'
# These tests are flaky due to FB13133387 so we don't want the noise in every PR
<<: *release-branches-and-main
- backend-integration-tests-custom-entitlements:
xcode_version: '15.2'
xcode_version: '15.3'
filters:
branches:
# Forked pull requests have CIRCLE_BRANCH set to pull/XXX
Expand Down Expand Up @@ -1235,12 +1235,12 @@ workflows:
xcode_version: '14.3.0'
<<: *release-tags
- push-revenuecat-pod:
xcode_version: '15.2'
xcode_version: '15.3'
requires:
- make-release
<<: *release-tags
- push-revenuecatui-pod:
xcode_version: '15.2'
xcode_version: '15.3'
requires:
- make-release
- push-revenuecat-pod
Expand Down Expand Up @@ -1283,10 +1283,10 @@ workflows:
- equal: [ "load_shedder_integration_tests", << pipeline.schedule.name >> ]
jobs:
- loadshedder-integration-tests-v3:
xcode_version: '15.2'
xcode_version: '15.3'
- integration-tests-all:
xcode_version: '15.2'
xcode_version: '15.3'

manual-trigger-bump:
when:
equal: [ bump, << pipeline.parameters.action >> ]
Expand Down
1 change: 1 addition & 0 deletions Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,5 @@ gem 'cocoapods'
gem 'cocoapods-trunk'
gem 'danger'
gem 'rest-client'
gem 'readline-ext'
eval_gemfile("fastlane/Pluginfile")
2 changes: 2 additions & 0 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -277,6 +277,7 @@ GEM
public_suffix (4.0.7)
rake (13.1.0)
rchardet (1.8.0)
readline-ext (0.2.0)
representable (3.2.0)
declarative (< 0.1.0)
trailblazer-option (>= 0.1.1, < 0.2.0)
Expand Down Expand Up @@ -346,6 +347,7 @@ DEPENDENCIES
fastlane
fastlane-plugin-create_xcframework
fastlane-plugin-revenuecat_internal!
readline-ext
rest-client

BUNDLED WITH
Expand Down
2 changes: 1 addition & 1 deletion Package.swift
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ let shouldIncludeDocCPlugin = environmentVariables["INCLUDE_DOCC_PLUGIN"] == "tr
var dependencies: [Package.Dependency] = [
.package(url: "git@github.com:Quick/Nimble.git", from: "10.0.0"),
// SST requires iOS 13 starting from version 1.13.0
.package(url: "git@github.com:pointfreeco/swift-snapshot-testing.git", .upToNextMinor(from: "1.12.0"))
.package(url: "git@github.com:revenuecat/swift-snapshot-testing.git", branch: "1.12.0-fix-test-observer")
]
if shouldIncludeDocCPlugin {
dependencies.append(.package(url: "https://github.com/apple/swift-docc-plugin", from: "1.0.0"))
Expand Down
2 changes: 1 addition & 1 deletion Package@swift-5.7.swift
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ let shouldIncludeDocCPlugin = environmentVariables["INCLUDE_DOCC_PLUGIN"] == "tr

var dependencies: [Package.Dependency] = [
.package(url: "git@github.com:Quick/Nimble.git", from: "10.0.0"),
.package(url: "git@github.com:pointfreeco/swift-snapshot-testing.git", from: "1.11.0")
.package(url: "git@github.com:revenuecat/swift-snapshot-testing.git", branch: "1.12.0-fix-test-observer")
]
if shouldIncludeDocCPlugin {
dependencies.append(.package(url: "https://github.com/apple/swift-docc-plugin", from: "1.0.0"))
Expand Down
2 changes: 1 addition & 1 deletion Package@swift-5.8.swift
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ let shouldIncludeDocCPlugin = environmentVariables["INCLUDE_DOCC_PLUGIN"] == "tr

var dependencies: [Package.Dependency] = [
.package(url: "git@github.com:Quick/Nimble.git", from: "10.0.0"),
.package(url: "git@github.com:pointfreeco/swift-snapshot-testing.git", from: "1.11.0")
.package(url: "git@github.com:revenuecat/swift-snapshot-testing.git", branch: "1.12.0-fix-test-observer")
]
if shouldIncludeDocCPlugin {
dependencies.append(.package(url: "https://github.com/apple/swift-docc-plugin", from: "1.0.0"))
Expand Down
Loading

0 comments on commit 0bb44cc

Please sign in to comment.