ui: HomeView에서 웹페이지 추가 시 툴바 위치에 LoadingView가 뜨도록 수정

Author: opficdev

Application/DevLogPresentation/Sources/Home/Home/HomeFeature+Effects.swift

@@ -76,6 +76,7 @@ extension HomeFeature {
                 trackAnalyticsEventUseCase.execute(.webPageCreate)
                 let pages = try await fetchWebPagesUseCase.execute("")
                 await send(.store(.updateWebPages(pages.map(WebPageItem.init(from:)))))
+                await send(.store(.setSheet(nil)))
             } catch {
                 await send(.store(.setAlert(isPresented: true, type: .error)))
             }

Application/DevLogPresentation/Sources/Home/Home/HomeFeature.swift

@@ -253,7 +253,6 @@ private extension HomeFeature {
                 Self.setAlert(&state, isPresented: true, type: .invalidURL)
                 return .none
             }
-            state.sheet = nil
             Self.setAlert(&state, isPresented: false, type: nil)
             return addWebPageEffect(normalizedURL)
         case .deleteWebPage(let page):

Application/DevLogPresentation/Sources/Home/Home/HomeView.swift

@@ -38,11 +38,6 @@ struct HomeView: View {
         .alert($store.scope(state: \.alert, action: \.alert))
         .sheet(item: $store.scope(state: \.sheet, action: \.sheet), content: sheetContent)
         .fullScreenCover(item: $store.scope(state: \.fullScreenCover, action: \.fullScreenCover), content: coverContent)
-        .overlay {
-            if store.isAppending {
-                LoadingView()
-            }
-        }
     }
 
     private var todoSection: some View {
@@ -234,9 +229,18 @@ struct HomeView: View {
                     .navigationTitle(Text(String(localized: "home_webpage_input_title")))
                     .navigationBarTitleDisplayMode(.inline) //  설정 안하면 섹션 위에 내비게이션 large 만큼 영역 먹음
                     .toolbar {
-                        ToolbarItem(placement: .topBarTrailing) {
-                            Button(String(localized: "home_add")) {
-                                store.send(.view(.addWebPage))
+                        if store.isAppending {
+                            if #available(iOS 26.0, *) {
+                                ToolbarSpacer(.fixed, placement: .topBarTrailing)
+                            }
+                            ToolbarItem(placement: .topBarTrailing) {
+                                ProgressView()
+                            }
+                        } else {
+                            ToolbarItem(placement: .topBarTrailing) {
+                                Button(String(localized: "home_add")) {
+                                    store.send(.view(.addWebPage))
+                                }
                             }
                         }
                     }

Application/DevLogPresentation/Tests/Home/HomeFeatureTestAssertions.swift

@@ -101,6 +101,7 @@ func verifyHomeAddWebPage(
     fetchWebPagesUseCaseSpy: FetchWebPagesUseCaseSpy,
     trackAnalyticsEventUseCaseSpy: HomeTrackAnalyticsEventUseCaseSpy
 ) async throws {
+    await adapter.setPresentation(.contentPicker, true)
     await adapter.updateWebPageURLInput("openai.com")
     await adapter.addWebPage()
 
@@ -116,9 +117,28 @@ func verifyHomeAddWebPage(
         "https://openai.com",
         "https://developer.apple.com"
     ])
+    #expect(!adapter.showContentPicker)
     #expect(!adapter.showAlert)
 }
 
+@MainActor
+func verifyHomeAddWebPageFailureKeepsSheet(
+    adapter: HomeStoreTestAdapter,
+    addWebPageUseCaseSpy: AddWebPageUseCaseSpy
+) async throws {
+    await adapter.setPresentation(.contentPicker, true)
+    await adapter.updateWebPageURLInput("openai.com")
+    await adapter.addWebPage()
+
+    await waitUntil {
+        addWebPageUseCaseSpy.calledUrlStrings == ["https://openai.com"]
+            && adapter.showAlert
+    }
+
+    #expect(adapter.showContentPicker)
+    #expect(adapter.alertType == .error)
+}
+
 struct HomeFetchDataContext {
     let fetchPreferencesUseCaseSpy: FetchTodoCategoryPreferencesUseCaseSpy
     let fetchTodosUseCaseSpy: FetchTodosUseCaseSpy

Application/DevLogPresentation/Tests/Home/HomeFeatureTests.swift

@@ -74,6 +74,22 @@ struct HomeFeatureTests {
         )
     }
 
+    @Test("HomeFeature addWebPage 실패는 입력 시트를 유지한다")
+    func HomeFeature_addWebPage_실패는_입력_시트를_유지한다() async throws {
+        let context = makeHomeAddWebPageContext()
+        context.addWebPageUseCaseSpy.error = HomeTestError.failure
+        let adapter = HomeStoreTestAdapter(
+            addWebPageUseCase: context.addWebPageUseCaseSpy,
+            fetchWebPagesUseCase: context.fetchWebPagesUseCaseSpy,
+            trackAnalyticsEventUseCase: context.trackAnalyticsEventUseCaseSpy
+        )
+
+        try await verifyHomeAddWebPageFailureKeepsSheet(
+            adapter: adapter,
+            addWebPageUseCaseSpy: context.addWebPageUseCaseSpy
+        )
+    }
+
     @Test("웹페이지를 삭제하면 항목이 즉시 숨겨지고 삭제 유스케이스가 호출된다")
     func 웹페이지를_삭제하면_항목이_즉시_숨겨지고_삭제_유스케이스가_호출된다() async throws {
         let context = makeHomeDeleteContext()
@@ -143,3 +159,7 @@ struct HomeFeatureTests {
         #expect(!adapter.isNetworkConnected)
     }
 }
+
+private enum HomeTestError: Error {
+    case failure
+}

Application/DevLogPresentation/Tests/Support/TestSupport.swift

@@ -155,10 +155,14 @@ final class UpdateTodoCategoryPreferencesUseCaseSpy: UpdateTodoCategoryPreferenc
 }
 
 final class AddWebPageUseCaseSpy: AddWebPageUseCase {
+    var error: Error?
     private(set) var calledUrlStrings: [String] = []
 
     func execute(_ urlString: String) async throws {
         calledUrlStrings.append(urlString)
+        if let error {
+            throw error
+        }
     }
 }