refactor: use AuthServiceError for all errors

Author: russellwheatley

FirebaseSwiftUI/FirebaseAuthSwiftUI/Sources/AuthServiceError.swift

@@ -38,6 +38,9 @@ public enum AuthServiceError: LocalizedError {
   case accountMergeConflict(context: AccountMergeConflictContext)
   case providerNotFound(String)
   case multiFactorAuth(String)
+  case rootViewControllerNotFound(String)
+  case providerAuthenticationFailed(String)
+  case signInCancelled(String)
 
   public var errorDescription: String? {
     switch self {
@@ -59,6 +62,8 @@ public enum AuthServiceError: LocalizedError {
     // Use when failed to sign-in with provider (e.g. Google, Facebook, etc.)
     case let .providerAuthenticationFailed(description):
       return description
+    case let .signInCancelled(description):
+      return description
     case let .accountMergeConflict(context):
       return context.errorDescription
     case let .providerNotFound(description):

FirebaseSwiftUI/FirebaseAuthSwiftUI/Sources/Services/AccountService+Email.swift

@@ -96,7 +96,8 @@ public final class PasswordPromptCoordinator {
 
   func cancel() {
     continuation?
-      .resume(throwing: AuthServiceError.reauthenticationRequired("Password entry cancelled"))
+      .resume(throwing: AuthServiceError
+        .signInCancelled("Password entry cancelled for Email provider"))
     cleanup()
   }
 

FirebaseSwiftUI/FirebaseFacebookSwiftUI/Sources/Services/FacebookProviderAuthUI.swift

@@ -19,14 +19,6 @@ import FirebaseAuth
 import FirebaseAuthSwiftUI
 import SwiftUI
 
-public enum FacebookProviderError: Error {
-  case signInCancelled(String)
-  case configurationInvalid(String)
-  case limitedLoginNonce(String)
-  case accessToken(String)
-  case authenticationToken(String)
-}
-
 public class FacebookProviderSwift: AuthProviderSwift, DeleteUserSwift {
   let scopes: [String]
   let providerId = "facebook.com"
@@ -60,8 +52,8 @@ public class FacebookProviderSwift: AuthProviderSwift, DeleteUserSwift {
         )
       }
     }() else {
-      throw FacebookProviderError
-        .configurationInvalid("Failed to create Facebook login configuration")
+      throw AuthServiceError
+        .providerAuthenticationFailed("Failed to create Facebook login configuration")
     }
 
     let result = try await withCheckedThrowingContinuation { (continuation: CheckedContinuation<
@@ -74,7 +66,8 @@ public class FacebookProviderSwift: AuthProviderSwift, DeleteUserSwift {
         switch result {
         case .cancelled:
           continuation
-            .resume(throwing: FacebookProviderError.signInCancelled("User cancelled sign-in"))
+            .resume(throwing: AuthServiceError
+              .signInCancelled("User cancelled sign-in for Facebook"))
         case let .failed(error):
           continuation.resume(throwing: error)
         case .success:
@@ -97,8 +90,8 @@ public class FacebookProviderSwift: AuthProviderSwift, DeleteUserSwift {
 
       return credential
     } else {
-      throw FacebookProviderError
-        .accessToken(
+      throw AuthServiceError
+        .providerAuthenticationFailed(
           "Access token has expired or not available. Please sign-in with Facebook before attempting to create a Facebook provider credential"
         )
     }
@@ -107,16 +100,18 @@ public class FacebookProviderSwift: AuthProviderSwift, DeleteUserSwift {
   private func limitedLogin() throws -> AuthCredential {
     if let idToken = AuthenticationToken.current {
       guard let nonce = rawNonce else {
-        throw FacebookProviderError
-          .limitedLoginNonce("`rawNonce` has not been generated for Facebook limited login")
+        throw AuthServiceError
+          .providerAuthenticationFailed(
+            "`rawNonce` has not been generated for Facebook limited login"
+          )
       }
       let credential = OAuthProvider.credential(withProviderID: providerId,
                                                 idToken: idToken.tokenString,
                                                 rawNonce: nonce)
       return credential
     } else {
-      throw FacebookProviderError
-        .authenticationToken(
+      throw AuthServiceError
+        .providerAuthenticationFailed(
           "Authentication is not available. Please sign-in with Facebook before attempting to create a Facebook provider credential"
         )
     }

FirebaseSwiftUI/FirebaseFacebookSwiftUI/Sources/Views/SignInWithFacebookButton.swift

@@ -66,17 +66,7 @@ extension SignInWithFacebookButton: View {
     VStack {
       Button(action: {
         Task {
-          do {
-            try await authService.signIn(facebookProvider)
-          } catch {
-            switch error {
-            case FacebookProviderError.signInCancelled:
-              showCanceledAlert = true
-            default:
-              // Error already handled by AuthService
-              break
-            }
-          }
+          try? await authService.signIn(facebookProvider)
         }
       }) {
         HStack {
@@ -113,12 +103,6 @@ extension SignInWithFacebookButton: View {
         .toggleStyle(SwitchToggleStyle(tint: .green))
       }
     }
-    .alert(isPresented: $showCanceledAlert) {
-      Alert(
-        title: Text(authService.string.facebookLoginCancelledLabel),
-        dismissButton: .default(Text(authService.string.okButtonLabel))
-      )
-    }
     .alert(isPresented: $showUserTrackingAlert) {
       Alert(
         title: Text(authService.string.authorizeUserTrackingLabel),

FirebaseSwiftUI/FirebasePhoneAuthSwiftUI/Sources/Views/PhoneAuthView.swift

@@ -51,11 +51,8 @@ extension PhoneAuthView: View {
         HStack {
           Spacer()
           Button(action: {
-            completion(.failure(NSError(
-              domain: "PhoneAuthError",
-              code: -1,
-              userInfo: [NSLocalizedDescriptionKey: "User cancelled"]
-            )))
+            completion(.failure(AuthServiceError
+                .signInCancelled("User cancelled sign-in for Phone")))
             dismiss()
           }) {
             Image(systemName: "xmark.circle.fill")