Additional changes to align to the Obj C API (#61)

* user is not optional * add missing getIDToken variant * add missing delete variant
thebrowsercompany · Apr 5, 2024 · 25dac03 · 25dac03
1 parent 03fe03a
commit 25dac03
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/Sources/FirebaseAuth/FirebaseAuthResult+Swift.swift b/Sources/FirebaseAuth/FirebaseAuthResult+Swift.swift
@@ -12,9 +12,8 @@ public final class AuthDataResult {
     self.impl = impl
   }
 
-  public var user: User? {
-    guard impl.user.is_valid() else { return nil }
-    return .init(impl.user)
+  public var user: User {
+    .init(impl.user)
   }
 
   // public var additionalUserInfo: AdditionalUserInfo? {

diff --git a/Sources/FirebaseAuth/FirebaseUser+Swift.swift b/Sources/FirebaseAuth/FirebaseUser+Swift.swift
@@ -144,6 +144,10 @@ public final class User {
     return try await AuthTokenResult(idTokenForcingRefresh(forceRefresh))
   }
 
+  public func getIDToken(completion: ((String?, Error?) -> Void)?) {
+    idTokenForcingRefresh(false, completion: completion)
+  }
+
   public func getIDToken() async throws -> String {
     return try await idTokenForcingRefresh(false)
   }
@@ -230,6 +234,10 @@ public final class User {
   //   fatalError("\(#function) not yet implemented")
   // }
 
+  public func delete(completion: ((Error?) -> Void)?) {
+    fatalError("\(#function) not yet implemented")
+  }
+
   public func delete() async throws {
     fatalError("\(#function) not yet implemented")
   }