refactor: 과도한 refresh 요청 방지를 위한 cancelId 추가

Author: opficdev

Application/DevLogPresentation/Sources/Home/List/TodoListFeature.swift

@@ -89,6 +89,7 @@ struct TodoListFeature {
 
     enum CancelID: Hashable {
         case debounce
+        case fetch
         case request
     }
 
@@ -261,12 +262,15 @@ private extension TodoListFeature {
                     ))
                     await send(.setHasMore(page.items.count == query.pageSize && page.nextCursor != nil))
                     await send(.loading(.end(target: .default, mode: .delayed)))
+                } catch is CancellationError {
+                    return
                 } catch {
                     await send(.setAlert(true))
                     await send(.loading(.end(target: .default, mode: .delayed)))
                 }
             }
         )
+        .cancellable(id: CancelID.fetch, cancelInFlight: true)
     }
 
     func setSearchTextEffect(state: inout State) -> Effect<Action> {

Application/DevLogPresentation/Tests/Home/TodoListFeatureTestDoubles.swift

@@ -194,6 +194,50 @@ final class TodoListFetchTodosUseCaseSpy: FetchTodosUseCase {
     }
 }
 
+actor TodoListDelayedFirstFetchTodosUseCaseSpy: FetchTodosUseCase {
+    private let pages: [TodoPage]
+    private var queries = [TodoQuery]()
+    private var cursors = [TodoCursor?]()
+    private var cancelledCallIndices = [Int]()
+
+    init(pages: [TodoPage]) {
+        self.pages = pages
+    }
+
+    func execute(_ query: TodoQuery, cursor: TodoCursor?) async throws -> TodoPage {
+        let callIndex = queries.count
+        queries.append(query)
+        cursors.append(cursor)
+
+        if callIndex == 0 {
+            do {
+                try await Task.sleep(for: .seconds(1))
+            } catch is CancellationError {
+                cancelledCallIndices.append(callIndex)
+                throw CancellationError()
+            }
+        }
+
+        if pages.count <= callIndex {
+            return pages.last ?? TodoPage(items: [], nextCursor: nil)
+        }
+
+        return pages[callIndex]
+    }
+
+    func calledQueries() -> [TodoQuery] {
+        queries
+    }
+
+    func calledCursors() -> [TodoCursor?] {
+        cursors
+    }
+
+    func cancelledCalls() -> [Int] {
+        cancelledCallIndices
+    }
+}
+
 final class TodoListFetchTodoByIdUseCaseSpy: FetchTodoByIdUseCase {
     var todos: [Todo]
     var error: Error?

Application/DevLogPresentation/Tests/Home/TodoListFeatureTests.swift

@@ -63,6 +63,32 @@ struct TodoListFeatureTests {
         #expect(!adapter.hasMore)
     }
 
+    @Test("새 목록 조회는 이전 요청을 취소하고 마지막 응답만 반영한다")
+    func 새_목록_조회는_이전_요청을_취소하고_마지막_응답만_반영한다() async {
+        let firstTodo = makeTodoListTodo(id: "todo-first", number: 1)
+        let secondTodo = makeTodoListTodo(id: "todo-second", number: 2)
+        let fetchSpy = TodoListDelayedFirstFetchTodosUseCaseSpy(pages: [
+            TodoPage(items: [firstTodo], nextCursor: nil),
+            TodoPage(items: [secondTodo], nextCursor: nil)
+        ])
+        let adapter = TodoListStoreTestAdapter(fetchUseCase: fetchSpy)
+
+        await adapter.onAppear()
+        await adapter.setSortTarget(.updatedAt)
+
+        await waitUntil(timeout: .seconds(2)) {
+            adapter.todos == [TodoListItem(from: secondTodo)!]
+        }
+
+        let queries = await fetchSpy.calledQueries()
+        let cancelledCalls = await fetchSpy.cancelledCalls()
+
+        #expect(adapter.todos == [TodoListItem(from: secondTodo)!])
+        #expect(queries.map(\.sortTarget) == [.createdAt, .updatedAt])
+        #expect(cancelledCalls == [0])
+        #expect(!adapter.showAlert)
+    }
+
     @Test("필터와 정렬 액션은 query와 적용 필터 수를 갱신한다")
     func 필터와_정렬_액션은_query와_적용_필터_수를_갱신한다() async {
         let adapter = TodoListStoreTestAdapter()