Skip to content

Tests: disabled iOS 11.x tests to fix Xcode 15 tests#2720

Merged
NachoSoto merged 3 commits into
mainfrom
disable-ios-11-tests
Jun 28, 2023
Merged

Tests: disabled iOS 11.x tests to fix Xcode 15 tests#2720
NachoSoto merged 3 commits into
mainfrom
disable-ios-11-tests

Conversation

@NachoSoto

@NachoSoto NachoSoto commented Jun 27, 2023

Copy link
Copy Markdown
Contributor

We weren't running iOS 11.x tests anyway because it's no longer possible to install those simulators.
This simply changes the deployment target of only test targets to iOS 12.0 so that running those in Xcode 15.0 doesn't fail.

Note that iOS 17 tests are still disabled in CI since we still have a few failures (see #2606).

@NachoSoto NachoSoto requested a review from a team June 27, 2023 17:29
Comment thread .circleci/config.yml Outdated
@NachoSoto NachoSoto force-pushed the disable-ios-11-tests branch 2 times, most recently from 33c7350 to f67fb06 Compare June 27, 2023 18:43

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The tvOS deployment target of API testers was set to iOS 14, so we weren't checking these.

We weren't running iOS 11.x tests anyway because it's no longer possible to install those simulators.
This simply changes the deployment target of **_only_ test targets** to iOS 12.0 so that running those in `Xcode 15.0` doesn't fail.
@NachoSoto NachoSoto removed the ci label Jun 27, 2023
@NachoSoto NachoSoto force-pushed the disable-ios-11-tests branch from f67fb06 to 37ff22d Compare June 27, 2023 20:23
@NachoSoto NachoSoto changed the title Tests: disabled iOS 11.x tests to fix Xcode 15 tests Tests: disabled iOS 11.x tests to fix Xcode 15 tests Jun 27, 2023
@codecov

codecov Bot commented Jun 27, 2023

Copy link
Copy Markdown

Codecov Report

Merging #2720 (37ff22d) into main (3b34874) will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2720      +/-   ##
==========================================
+ Coverage   86.43%   86.47%   +0.04%     
==========================================
  Files         212      212              
  Lines       15048    15048              
==========================================
+ Hits        13006    13013       +7     
+ Misses       2042     2035       -7     

see 6 files with indirect coverage changes

@NachoSoto NachoSoto merged commit 1cbc974 into main Jun 28, 2023
@NachoSoto NachoSoto deleted the disable-ios-11-tests branch June 28, 2023 14:07
NachoSoto added a commit that referenced this pull request Jun 30, 2023
**This is an automatic release.**

### New Features
* New `TestStoreProduct` for creating mock `StoreProduct`s and
`Offering`s (#2711) via NachoSoto (@NachoSoto)

✨ With this new API you can now create fake products for unit tests and
SwiftUI previews! 🎉
#### Example:
```swift
let product = TestStoreProduct(
    localizedTitle: "PRO monthly",
    price: 3.99,
    localizedPriceString: "$3.99",
    productIdentifier: "com.revenuecat.product",
    productType: .autoRenewableSubscription,
    localizedDescription: "Description",
    subscriptionGroupIdentifier: "group",
    subscriptionPeriod: .init(value: 1, unit: .month)
)
let offering = Offering(
    identifier: "offering",
    serverDescription: "Main offering",
    metadata: [:],
    availablePackages: [
        .init(
            identifier: "monthly",
            packageType: .monthly,
            storeProduct: product.toStoreProduct(),
            offeringIdentifier: offering
        )
    ]
)
```

### Dependency Updates
* Bump fastlane-plugin-revenuecat_internal from `13773d2` to `b2108fb`
(#2706) via dependabot[bot] (@dependabot[bot])
### Other Changes
* `VerificationResult: CustomDebugStringConvertible` (#2739) via
NachoSoto (@NachoSoto)
* Refactor: simplified `PurchasesOrchestrator.syncPurchases` (#2731) via
NachoSoto (@NachoSoto)
* `Trusted Entitlements`: add integration tests to verify `CustomerInfo`
cache invalidation (#2730) via NachoSoto (@NachoSoto)
* `SystemInfo.identifierForVendor`: add tests (#2732) via NachoSoto
(@NachoSoto)
* `Tests`: disabled `iOS 11.x` tests to fix `Xcode 15` tests (#2720) via
NachoSoto (@NachoSoto)
* `DebugViewSwiftUITests`: create separate snapshots for each OS version
(#2721) via NachoSoto (@NachoSoto)
* `Integration Tests`: fix clearing `UserDefaults` before each test
(#2719) via NachoSoto (@NachoSoto)
* Remove unused `Signing.loadPublicKey(with:)` (#2714) via NachoSoto
(@NachoSoto)
* Add `UInt32(littleEndian32Bits:)` and `UInt32.littleEndianData`
(#2713) via NachoSoto (@NachoSoto)
* `TimingUtil`: added synchronous API (#2716) via NachoSoto (@NachoSoto)
* `XCFramework`: sign archive for `Xcode 15` (#2709) via NachoSoto
(@NachoSoto)
* `CI`: removed `carthage_archive` from `release` lane (#2710) via
NachoSoto (@NachoSoto)
* `PriceFormatterProvider.priceFormatterForSK2`: enable on all versions
(#2712) via NachoSoto (@NachoSoto)
* `xrOS`: add support for `debugRevenueCatOverlay` (#2702) via NachoSoto
(@NachoSoto)
* Refactor method to get product ID including plan ID in android
purchases (#2708) via Toni Rico (@tonidero)
* `Purchases.restoreLogHandler` (#2699) via NachoSoto (@NachoSoto)
* Remove alpha from purchase tester icon to upload to testflight (#2705)
via Toni Rico (@tonidero)

---------

Co-authored-by: NachoSoto <ignaciosoto90@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants