-
Notifications
You must be signed in to change notification settings - Fork 319
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
Xcode 15.3
: fix warnings on Integration Tests
#3603
Conversation
Unfortunately removing these
|
Oh no never mind, it's because this is broken in private class CleanCounterBetweenTestCases: NSObject, XCTestObservation {
private static var registered = false
private static var registerQueue = DispatchQueue(label: "co.pointfree.SnapshotTesting.testObserver")
static func registerIfNeeded() {
registerQueue.sync {
if !registered {
registered = true
XCTestObservationCenter.shared.addTestObserver(CleanCounterBetweenTestCases())
}
}
}
func testCaseDidFinish(_ testCase: XCTestCase) {
counterQueue.sync {
counterMap = [:]
}
}
} |
Blocked by this: pointfreeco/swift-snapshot-testing#661 Updating |
80eb253
to
527f4bb
Compare
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.
77e6820
to
1435b74
Compare
1435b74
to
34939db
Compare
Xcode 15.3 beta 1
: fix warnings on Integration TestsXcode 15.3
: fix warnings on Integration Tests
Submitted a PR to the Meanwhile, we can use the fork at https://github.com/RevenueCat/swift-snapshot-testing/tree/1.12.0-fix-test-observer |
…required by fastlane See: #3729
This reverts commit 0bb44cc.
Reverts #3603 I _think_ this is causing a bunch of failures in SK2 tests
Follow up to #3599 and #3600.
These
@MainActor
annotations don't work anymore because of this:They were necessary with an older version of Xcode (likely 13.x), but not anymore.