refactor: 위젯 동기화 트리거를 앱이 백그라운드로 내려갔을때로 축소

Author: opficdev

DevLog/App/Assembler/DataAssembler.swift

@@ -29,8 +29,7 @@ final class DataAssembler: Assembler {
         container.register(TodoRepository.self) {
             TodoRepositoryImpl(
                 todoService: container.resolve(TodoService.self),
-                todoCategoryService: container.resolve(TodoCategoryService.self),
-                widgetSyncEventBus: container.resolve(WidgetSyncEventBus.self)
+                todoCategoryService: container.resolve(TodoCategoryService.self)
             )
         }
 
@@ -98,8 +97,7 @@ final class DataAssembler: Assembler {
             UserPreferencesRepositoryImpl(
                 store: container.resolve(UserDefaultsStore.self),
                 themeStore: container.resolve(ThemeStore.self),
-                widgetSnapshotPreferenceStore: container.resolve(WidgetSnapshotPreferenceStore.self),
-                widgetSyncEventBus: container.resolve(WidgetSyncEventBus.self)
+                widgetSnapshotPreferenceStore: container.resolve(WidgetSnapshotPreferenceStore.self)
             )
         }
     }

DevLog/App/DevLogApp.swift

@@ -11,6 +11,7 @@ import SwiftUI
 struct DevLogApp: App {
     @UIApplicationDelegateAdaptor(AppDelegate.self) var delegate
     @Environment(\.diContainer) var container: DIContainer
+    @Environment(\.scenePhase) var scenePhase
 
     init() {
         AppAssembler().assemble(AppDIContainer.shared)
@@ -24,6 +25,10 @@ struct DevLogApp: App {
                 systemThemeUseCase: container.resolve(ObserveSystemThemeUseCase.self)
             ))
             .autocorrectionDisabled()
+            .onChange(of: scenePhase) { _, phase in
+                guard phase == .background else { return }
+                container.resolve(WidgetSyncEventBus.self).publish(.syncRequested)
+            }
         }
     }
 }

DevLog/Data/Repository/TodoRepositoryImpl.swift

@@ -10,16 +10,13 @@ import Foundation
 final class TodoRepositoryImpl: TodoRepository {
     private let todoService: TodoService
     private let todoCategoryService: TodoCategoryService
-    private let widgetSyncEventBus: WidgetSyncEventBus
 
     init(
         todoService: TodoService,
-        todoCategoryService: TodoCategoryService,
-        widgetSyncEventBus: WidgetSyncEventBus
+        todoCategoryService: TodoCategoryService
     ) {
         self.todoService = todoService
         self.todoCategoryService = todoCategoryService
-        self.widgetSyncEventBus = widgetSyncEventBus
     }
 
     func fetchTodos(_ query: TodoQuery, cursor: TodoCursor?) async throws -> TodoPage {
@@ -92,17 +89,14 @@ final class TodoRepositoryImpl: TodoRepository {
     func upsertTodo(_ todo: Todo) async throws {
         let request = TodoRequest.fromDomain(todo)
         try await todoService.upsertTodo(request: request)
-        widgetSyncEventBus.publish(.todoDataChanged)
     }
 
     func deleteTodo(_ todoId: String) async throws {
         try await todoService.deleteTodo(todoId: todoId)
-        widgetSyncEventBus.publish(.todoDataChanged)
     }
 
     func undoDeleteTodo(_ todoId: String) async throws {
         try await todoService.undoDeleteTodo(todoId: todoId)
-        widgetSyncEventBus.publish(.todoDataChanged)
     }
 }
 

DevLog/Data/Repository/UserPreferencesRepositoryImpl.swift

@@ -20,18 +20,15 @@ final class UserPreferencesRepositoryImpl: UserPreferencesRepository {
     private let store: UserDefaultsStore
     private let themeStore: ThemeStore
     private let widgetSnapshotPreferenceStore: WidgetSnapshotPreferenceStore
-    private let widgetSyncEventBus: WidgetSyncEventBus
 
     init(
         store: UserDefaultsStore,
         themeStore: ThemeStore,
-        widgetSnapshotPreferenceStore: WidgetSnapshotPreferenceStore,
-        widgetSyncEventBus: WidgetSyncEventBus
+        widgetSnapshotPreferenceStore: WidgetSnapshotPreferenceStore
     ) {
         self.store = store
         self.themeStore = themeStore
         self.widgetSnapshotPreferenceStore = widgetSnapshotPreferenceStore
-        self.widgetSyncEventBus = widgetSyncEventBus
         themeStore.send(systemTheme())
     }
 
@@ -93,7 +90,6 @@ final class UserPreferencesRepositoryImpl: UserPreferencesRepository {
 
     func setHeatmapActivityTypes(_ activityTypes: [String]) {
         widgetSnapshotPreferenceStore.setHeatmapActivityTypes(activityTypes)
-        widgetSyncEventBus.publish(.heatmapActivityKindsChanged)
     }
 
     func todayDisplayOptions() -> TodayDisplayOptions {
@@ -102,6 +98,5 @@ final class UserPreferencesRepositoryImpl: UserPreferencesRepository {
 
     func setTodayDisplayOptions(_ options: TodayDisplayOptions) {
         widgetSnapshotPreferenceStore.setTodayDisplayOptions(options)
-        widgetSyncEventBus.publish(.todayDisplayOptionsChanged)
     }
 }

DevLog/Widget/Common/WidgetSyncEvent.swift

@@ -6,7 +6,5 @@
 //
 
 enum WidgetSyncEvent: Equatable {
-    case todoDataChanged
-    case todayDisplayOptionsChanged
-    case heatmapActivityKindsChanged
+    case syncRequested
 }

DevLog/Widget/Sync/WidgetSyncEventHandler.swift

@@ -34,21 +34,13 @@ final class WidgetSyncEventHandler {
 private extension WidgetSyncEventHandler {
     func handle(_ event: WidgetSyncEvent) {
         switch event {
-        case .todoDataChanged:
+        case .syncRequested:
             Task { [weak self] in
                 guard let self else { return }
                 async let todaySnapshot: Void = updateTodayWidgetSnapshot()
                 async let heatmapSnapshot: Void = updateHeatmapWidgetSnapshot()
                 _ = await (todaySnapshot, heatmapSnapshot)
             }
-        case .todayDisplayOptionsChanged:
-            Task { [weak self] in
-                await self?.updateTodayWidgetSnapshot()
-            }
-        case .heatmapActivityKindsChanged:
-            Task { [weak self] in
-                await self?.updateHeatmapWidgetSnapshot()
-            }
         }
     }
 

DevLog_Unit/Widget/WidgetSyncEventBusTests.swift

@@ -19,13 +19,9 @@ struct WidgetSyncEventBusTests {
                 receivedEvents.append(event)
             }
 
-        bus.publish(.todoDataChanged)
-        bus.publish(.todayDisplayOptionsChanged)
+        bus.publish(.syncRequested)
 
-        #expect(receivedEvents == [
-            .todoDataChanged,
-            .todayDisplayOptionsChanged
-        ])
+        #expect(receivedEvents == [.syncRequested])
         _ = cancellable
     }
 }

DevLog_Unit/Widget/WidgetSyncEventHandlerTests.swift

@@ -10,8 +10,8 @@ import Testing
 @testable import DevLog
 
 struct WidgetSyncEventHandlerTests {
-    @Test("Todo 데이터 변경 이벤트는 Today와 Heatmap 스냅샷을 갱신한다")
-    func todo_데이터_변경_이벤트는_today와_heatmap_스냅샷을_갱신한다() async throws {
+    @Test("위젯 동기화 요청 이벤트는 Today와 Heatmap 스냅샷을 갱신한다")
+    func 위젯_동기화_요청_이벤트는_today와_heatmap_스냅샷을_갱신한다() async throws {
         let calendar = Calendar.current
         let now = Date()
         let quarterStart = calendar.startOfQuarter(for: now)
@@ -32,7 +32,7 @@ struct WidgetSyncEventHandlerTests {
             ]
         )
 
-        fixture.bus.publish(.todoDataChanged)
+        fixture.bus.publish(.syncRequested)
 
         let todaySnapshot = try await loadTodaySnapshot(from: fixture.snapshotStore)
         let heatmapSnapshot = try await loadHeatmapSnapshot(from: fixture.snapshotStore)

DevLog_Unit/Widget/WidgetSyncEventTests.swift

@@ -10,10 +10,8 @@ import Testing
 @testable import DevLog
 
 struct WidgetSyncEventTests {
-    @Test("위젯 동기화 이벤트는 변경 원인만 표현한다")
-    func 위젯_동기화_이벤트는_변경_원인만_표현한다() {
-        #expect(WidgetSyncEvent.todoDataChanged == .todoDataChanged)
-        #expect(WidgetSyncEvent.todayDisplayOptionsChanged == .todayDisplayOptionsChanged)
-        #expect(WidgetSyncEvent.heatmapActivityKindsChanged == .heatmapActivityKindsChanged)
+    @Test("위젯 동기화 이벤트는 동기화 요청만 표현한다")
+    func 위젯_동기화_이벤트는_동기화_요청만_표현한다() {
+        #expect(WidgetSyncEvent.syncRequested == .syncRequested)
     }
 }