refactor: AlertState 적용

Author: opficdev

Application/DevLogPresentation/Sources/Root/RootFeature.swift

@@ -21,9 +21,7 @@ struct RootFeature {
 
     @ObservableState
     struct State: Equatable {
-        var showAlert = false
-        var alertTitle = ""
-        var alertMessage = ""
+        @Presents var alert: AlertState<Never>?
         var isNetworkConnected = true
         var signIn: Bool?
         var theme: SystemTheme = .automatic
@@ -33,12 +31,12 @@ struct RootFeature {
     }
 
     enum Action: Equatable {
+        case alert(PresentationAction<Never>)
         case view(ViewAction)
         case store(StoreAction)
 
         enum ViewAction: Equatable {
             case onAppear
-            case setAlert(Bool)
         }
 
         enum StoreAction: Equatable {
@@ -57,6 +55,8 @@ struct RootFeature {
     var body: some ReducerOf<Self> {
         Reduce { state, action in
             switch action {
+            case .alert:
+                break
             case .view(.onAppear):
                 var effect = clearApplicationBadgeCountEffect()
 
@@ -76,13 +76,11 @@ struct RootFeature {
                 }
 
                 return effect
-            case .view(.setAlert(let isPresented)):
-                Self.setAlert(&state, isPresented: isPresented)
             case .store(.networkStatusChanged(let isConnected)):
                 let wasConnected = state.isNetworkConnected
                 state.isNetworkConnected = isConnected
                 if wasConnected && !isConnected {
-                    Self.setAlert(&state, isPresented: true)
+                    state.alert = Self.alertState()
                 }
             case .store(.setTheme(let theme)):
                 state.theme = theme
@@ -95,6 +93,7 @@ struct RootFeature {
 
             return .none
         }
+        .ifLet(\.$alert, action: \.alert)
     }
 }
 
@@ -154,9 +153,15 @@ private extension RootFeature {
         }
     }
 
-    static func setAlert(_ state: inout State, isPresented: Bool) {
-        state.alertTitle = String(localized: "root_network_disconnected_title")
-        state.alertMessage = String(localized: "root_network_disconnected_message")
-        state.showAlert = isPresented
+    static func alertState() -> AlertState<Never> {
+        AlertState {
+            TextState(String(localized: "root_network_disconnected_title"))
+        } actions: {
+            ButtonState(role: .cancel) {
+                TextState(String(localized: "common_close"))
+            }
+        } message: {
+            TextState(String(localized: "root_network_disconnected_message"))
+        }
     }
 }

Application/DevLogPresentation/Sources/Root/RootView.swift

@@ -79,14 +79,7 @@ public struct RootView: View {
                 break
             }
         }
-        .alert(store.alertTitle, isPresented: Binding(
-            get: { store.showAlert },
-            set: { store.send(.view(.setAlert($0))) }
-        )) {
-            Button(String(localized: "common_close"), role: .cancel) { }
-        } message: {
-            Text(store.alertMessage)
-        }
+        .alert($store.scope(state: \.alert, action: \.alert))
         .sheet(item: $selectedRoute) { route in
             switch route {
             case .todoDetail(let todoId):

Application/DevLogPresentation/Tests/Root/RootFeatureTestSupport.swift

@@ -24,9 +24,8 @@ protocol RootStateDriving {
 }
 
 struct RootStateSnapshot: Equatable {
-    let showAlert: Bool
-    let alertTitle: String
-    let alertMessage: String
+    let alertTitle: String?
+    let alertMessage: String?
     let isNetworkConnected: Bool
     let signIn: Bool?
     let theme: SystemTheme
@@ -38,9 +37,8 @@ struct RootStoreTestAdapter: RootStateDriving {
 
     var snapshot: RootStateSnapshot {
         RootStateSnapshot(
-            showAlert: store.state.showAlert,
-            alertTitle: store.state.alertTitle,
-            alertMessage: store.state.alertMessage,
+            alertTitle: store.state.alert.map { String(state: $0.title) },
+            alertMessage: store.state.alert?.message.map { String(state: $0) },
             isNetworkConnected: store.state.isNetworkConnected,
             signIn: store.state.signIn,
             theme: store.state.theme
@@ -78,7 +76,11 @@ struct RootStoreTestAdapter: RootStateDriving {
     }
 
     func setAlert(_ isPresented: Bool) async {
-        await store.send(.view(.setAlert(isPresented)))
+        if isPresented {
+            await store.send(.store(.networkStatusChanged(false)))
+        } else {
+            await store.send(.alert(.dismiss))
+        }
     }
 
     func networkStatusChanged(_ isConnected: Bool) async {
@@ -107,7 +109,6 @@ func verifyNetworkDisconnectedAlert(adapter: some RootStateDriving) async {
     #expect(
         adapter.snapshot
             == RootStateSnapshot(
-                showAlert: true,
                 alertTitle: String(localized: "root_network_disconnected_title"),
                 alertMessage: String(localized: "root_network_disconnected_message"),
                 isNetworkConnected: false,
@@ -125,9 +126,8 @@ func verifySetAlert(adapter: some RootStateDriving) async {
     #expect(
         adapter.snapshot
             == RootStateSnapshot(
-                showAlert: false,
-                alertTitle: String(localized: "root_network_disconnected_title"),
-                alertMessage: String(localized: "root_network_disconnected_message"),
+                alertTitle: nil,
+                alertMessage: nil,
                 isNetworkConnected: false,
                 signIn: nil,
                 theme: .automatic
@@ -140,7 +140,7 @@ func verifyThemeUpdate(adapter: some RootStateDriving) async {
     await adapter.setTheme(.dark)
 
     #expect(adapter.snapshot.theme == .dark)
-    #expect(!adapter.snapshot.showAlert)
+    #expect(adapter.snapshot.alertTitle == nil)
 }
 
 @MainActor
@@ -176,13 +176,12 @@ func verifyObservedInitialValues(adapter: some RootStateDriving) async {
         return snapshot.signIn == false
             && !snapshot.isNetworkConnected
             && snapshot.theme == .dark
-            && snapshot.showAlert
+            && snapshot.alertTitle == String(localized: "root_network_disconnected_title")
     }
 
     #expect(
         adapter.snapshot
             == RootStateSnapshot(
-                showAlert: true,
                 alertTitle: String(localized: "root_network_disconnected_title"),
                 alertMessage: String(localized: "root_network_disconnected_message"),
                 isNetworkConnected: false,