Skip to content

Commit e5933a0

Browse files
authored
[Auth] Fix 'PhoneAuthProviderFake.swift' following Swift 6 changes (#14895)
1 parent c8a6c0a commit e5933a0

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

.github/workflows/combine.yml

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,13 @@ on:
3333
# Rebuild on Ruby infrastructure changes.
3434
- 'Gemfile'
3535

36-
# Dependencies (Disabled to avoid building Firestore in presubmits)
37-
# - 'FirebaseCore/**'
38-
# - 'FirebaseAuth/**'
39-
# - 'FirebaseFunctions/**'
40-
# - 'Firestore/**'
41-
# - 'FirebaseStorage/**'
36+
# Dependencies
37+
- 'FirebaseCore/**'
38+
- 'FirebaseTestingSupport/**'
39+
- 'FirebaseAuth/**'
40+
- 'FirebaseFunctions/**'
41+
- 'FirebaseStorage/**'
42+
# - 'Firestore/**' # (Disabled to avoid building Firestore in presubmits)
4243

4344
schedule:
4445
# Run every day at 11pm (PST) - cron uses UTC times

FirebaseTestingSupport/Auth/Sources/PhoneAuthProviderFake.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,16 +16,16 @@
1616
import Foundation
1717

1818
/// A fake object to replace a real `AuthAPNSTokenManager` in tests.
19-
public class PhoneAuthProviderFake: PhoneAuthProvider {
19+
public class PhoneAuthProviderFake: PhoneAuthProvider, @unchecked Sendable {
2020
override init(auth: Auth) {
2121
super.init(auth: auth)
2222
}
2323

24-
var verifyPhoneNumberHandler: (((String?, Error?) -> Void) -> Void)?
24+
var verifyPhoneNumberHandler: ((@MainActor (String?, Error?) -> Void) -> Void)?
2525

2626
override public func verifyPhoneNumber(_ phoneNumber: String,
2727
uiDelegate: AuthUIDelegate? = nil,
28-
completion: ((_: String?, _: Error?) -> Void)?) {
28+
completion: (@MainActor (String?, Error?) -> Void)?) {
2929
if let verifyPhoneNumberHandler,
3030
let completion {
3131
verifyPhoneNumberHandler(completion)

0 commit comments

Comments
 (0)