🔀 :: [#97] 공지 Feature Refresh 기능 추가

Projects/Feature/NoticeFeature/Sources/Scene/NoticeStore.swift

@@ -35,10 +35,11 @@ final class NoticeStore: BaseStore {
         var currentUserRole: UserRoleType = .member
         var isEditingMode = false
         var selectedNotice: Set<Int> = []
+        var isRefreshing = false
     }
     enum Action {
         case viewDidLoad
-        case viewWillAppear
+        case fetchNoticeList
         case editButtonDidTap
         case noticeDidTap(Int)
     }
@@ -49,6 +50,7 @@ final class NoticeStore: BaseStore {
         case insertSelectedNotice(Int)
         case removeSelectedNotice(Int)
         case removeAllSelectedNotice
+        case updateIsRefreshing(Bool)
     }
 }
 
@@ -58,8 +60,8 @@ extension NoticeStore {
         case .viewDidLoad:
             return viewDidLoad()
 
-        case .viewWillAppear:
-            return viewWillAppear()
+        case .fetchNoticeList:
+            return fetchNoticeList()
 
         case .editButtonDidTap:
             return .merge(
@@ -97,20 +99,25 @@ extension NoticeStore {
 
         case .removeAllSelectedNotice:
             newState.selectedNotice.removeAll()
+
+        case let .updateIsRefreshing(isRefreshing):
+            newState.isRefreshing = isRefreshing
         }
         return newState
     }
 }
 
+// MARK: - Action
 private extension NoticeStore {
-    func viewWillAppear() -> SideEffect<Mutation, Never> {
-        return SideEffect<[NoticeModel], Error>
+    func fetchNoticeList() -> SideEffect<Mutation, Never> {
+        let noticeEffect = SideEffect<[NoticeModel], Error>
             .tryAsync { [fetchNoticeListUseCase] in
                 try await fetchNoticeListUseCase()
             }
             .map(Mutation.updateNoticeList)
             .catchToNever()
             .eraseToSideEffect()
+        return self.makeRefreshingSideEffect(noticeEffect)
     }
 
     func viewDidLoad() -> SideEffect<Mutation, Never> {
@@ -132,7 +139,10 @@ private extension NoticeStore {
         : Mutation.insertSelectedNotice(noticeID)
         return .just(mutation)
     }
+}
 
+// MARK: - Mutate
+private extension NoticeStore {
     func noticeListToSections(noticeList: [NoticeModel]) -> [SectiondNoticeTuple] {
         return noticeList.reduce(
             into: [SectiondNoticeTuple]()
@@ -150,3 +160,19 @@ private extension NoticeStore {
         }
     }
 }
+
+// MARK: - Reusable
+private extension NoticeStore {
+    func makeRefreshingSideEffect(
+        _ publisher: SideEffect<Mutation, Never>
+    ) -> SideEffect<Mutation, Never> {
+        let startLoadingPublisher = SideEffect<Mutation, Never>
+            .just(Mutation.updateIsRefreshing(true))
+        let endLoadingPublisher = SideEffect<Mutation, Never>
+            .just(Mutation.updateIsRefreshing(false))
+        return startLoadingPublisher
+            .append(publisher)
+            .append(endLoadingPublisher)
+            .eraseToSideEffect()
+    }
+}

Projects/Feature/NoticeFeature/Sources/Scene/NoticeViewController.swift

@@ -37,6 +37,7 @@ final class NoticeViewController: BaseViewController<NoticeStore> {
         .then {
             $0.register(cellType: NoticeCell.self)
         }
+    private let noticeRefreshControl = UIRefreshControl()
     private lazy var noticeTableAdapter = TableViewAdapter<GenericSectionModel<NoticeModel>>(
         tableView: noticeTableView
     ) { tableView, indexPath, notice in
@@ -59,6 +60,7 @@ final class NoticeViewController: BaseViewController<NoticeStore> {
             headerStackView
             noticeTableView
         }
+        noticeTableView.refreshControl = noticeRefreshControl
     }
 
     override func setLayout() {
@@ -86,7 +88,7 @@ final class NoticeViewController: BaseViewController<NoticeStore> {
             .store(in: &subscription)
 
         viewWillAppearPublisher
-            .map { Store.Action.viewWillAppear }
+            .map { Store.Action.fetchNoticeList }
             .sink(receiveValue: store.send(_:))
             .store(in: &subscription)
 
@@ -101,6 +103,11 @@ final class NoticeViewController: BaseViewController<NoticeStore> {
             .map(Store.Action.noticeDidTap)
             .sink(receiveValue: store.send(_:))
             .store(in: &subscription)
+
+        noticeRefreshControl.controlPublisher(for: .valueChanged)
+            .map { _ in Store.Action.fetchNoticeList }
+            .sink(receiveValue: store.send(_:))
+            .store(in: &subscription)
     }
 
     override func bindState() {
@@ -154,6 +161,13 @@ final class NoticeViewController: BaseViewController<NoticeStore> {
                 self?.transformWriteOrRemoveButton(isEditMode: isEditingMode)
             })
             .store(in: &subscription)
+
+        sharedState
+            .map(\.isRefreshing)
+            .sink(with: noticeRefreshControl) { refreshControl, isRefreshing in
+                isRefreshing ? refreshControl.beginRefreshing() : refreshControl.endRefreshing()
+            }
+            .store(in: &subscription)
     }
 }
 

Projects/Feature/NoticeFeature/Tests/NoticeFeatureTest.swift

@@ -66,13 +66,13 @@ final class NoticeFeatureTests: XCTestCase {
         ]
         fetchNoticeListUseCase.fetchNoticeListReturn = expected
 
-        sut.state.map(\.noticeList).sink { _ in
+        sut.state.map(\.noticeList).removeDuplicates().sink { _ in
             expectation.fulfill()
         }
         .store(in: &subscription)
 
         XCTAssertEqual(fetchNoticeListUseCase.fetchNoticeListCallCount, 0)
-        sut.send(.viewWillAppear)
+        sut.send(.fetchNoticeList)
 
         wait(for: [expectation], timeout: 1.0)
         XCTAssertEqual(expected, sut.currentState.noticeList)