refactor: providerIDs가 currentUser가 nil인 경우 빈 배열을 반환하도록 개선

opficdev · opficdev · commit 81f0c999cb35 · 2026-03-11T12:44:39.000+09:00
diff --git a/DevLog/Data/Repository/AuthDataRepositoryImpl.swift b/DevLog/Data/Repository/AuthDataRepositoryImpl.swift
@@ -33,7 +33,7 @@ final class AuthDataRepositoryImpl: AuthDataRepository {
     }
     
     func fetchAllProviders() async throws -> [AuthProvider] {
-        let providerStrings = authService.providerIDs ?? []
+        let providerStrings = authService.providerIDs
         return providerStrings.compactMap { AuthProvider(rawValue: $0) }
     }
     
diff --git a/DevLog/Data/Repository/AuthenticationRepositoryImpl.swift b/DevLog/Data/Repository/AuthenticationRepositoryImpl.swift
@@ -67,7 +67,7 @@ final class AuthenticationRepositoryImpl: AuthenticationRepository {
             throw AuthError.notAuthenticated
         }
 
-        let providers = (authService.providerIDs ?? []).compactMap { AuthProvider(rawValue: $0) }
+        let providers = authService.providerIDs.compactMap { AuthProvider(rawValue: $0) }
 
         for provider in providers {
             switch provider {
diff --git a/DevLog/Infra/Service/AuthService.swift b/DevLog/Infra/Service/AuthService.swift
@@ -20,8 +20,8 @@ final class AuthService {
         Auth.auth().currentUser?.uid
     }
 
-    var providerIDs: [String]? {
-        Auth.auth().currentUser?.providerData.map { $0.providerID }
+    var providerIDs: [String] {
+        Auth.auth().currentUser?.providerData.map { $0.providerID } ?? []
     }
 
     func getProviderID() async throws -> String? {

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ final class AuthDataRepositoryImpl: AuthDataRepository {`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`func fetchAllProviders() async throws -> [AuthProvider] {`
`36`		`- let providerStrings = authService.providerIDs ?? []`
	`36`	`+ let providerStrings = authService.providerIDs`
`37`	`37`	`return providerStrings.compactMap { AuthProvider(rawValue: $0) }`
`38`	`38`	`}`
`39`	`39`
Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ final class AuthenticationRepositoryImpl: AuthenticationRepository {`
`67`	`67`	`throw AuthError.notAuthenticated`
`68`	`68`	`}`
`69`	`69`
`70`		`- let providers = (authService.providerIDs ?? []).compactMap { AuthProvider(rawValue: $0) }`
	`70`	`+ let providers = authService.providerIDs.compactMap { AuthProvider(rawValue: $0) }`
`71`	`71`
`72`	`72`	`for provider in providers {`
`73`	`73`	`switch provider {`
Original file line number	Diff line number	Diff line change
`@@ -20,8 +20,8 @@ final class AuthService {`
`20`	`20`	`Auth.auth().currentUser?.uid`
`21`	`21`	`}`
`22`	`22`
`23`		`- var providerIDs: [String]? {`
`24`		`- Auth.auth().currentUser?.providerData.map { $0.providerID }`
	`23`	`+ var providerIDs: [String] {`
	`24`	`+ Auth.auth().currentUser?.providerData.map { $0.providerID } ?? []`
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`func getProviderID() async throws -> String? {`