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

Remove unneeded tests for StoreKit2 with JWS #3747

Merged
merged 6 commits into from
Mar 5, 2024
Merged

Conversation

joshdholtz
Copy link
Member

@joshdholtz joshdholtz commented Mar 5, 2024

Motivation

Unlock shipping by skipping tests that are flaky unneeded tests

Description

Removing StoreKit2JWS tests from BackendIntegrationTests-SK2.xctestplan

⚠️ The functionality in StoreKit2JWSIntegrationTests (which is the usesStoreKit2JWS dangerous setting), is not publicly available in iOS v4. Its intention was intended only for iOS v5. These tests are safe to skip since they have no production use.

⚠️ It is important that these are not skipped for iOS v5.

@joshdholtz joshdholtz added the ci label Mar 5, 2024
@joshdholtz joshdholtz requested a review from a team March 5, 2024 18:57
@joshdholtz joshdholtz marked this pull request as ready for review March 5, 2024 19:19
@joshdholtz joshdholtz requested a review from aboedo March 5, 2024 19:21
@joshdholtz joshdholtz changed the title SK2 JWS Flaky Tests Remove unneeded tests for StoreKit2 with JWS Mar 5, 2024
@joshdholtz joshdholtz added test and removed ci labels Mar 5, 2024
@joshdholtz joshdholtz merged commit b259d2f into main Mar 5, 2024
24 checks passed
@joshdholtz joshdholtz deleted the sk2-jws-flaky-tests branch March 5, 2024 22:06
joshdholtz pushed a commit that referenced this pull request Mar 6, 2024
**This is an automatic release.**

### Bugfixes
* Fix for passing `TargetingContext` when using
`currentOffering(forPlacement:)` (#3751) via Josh Holtz (@joshdholtz)
### Other Changes
* Remove unneeded tests for StoreKit2 with JWS (#3747) via Josh Holtz
(@joshdholtz)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants