refactor: 옵셔널 주입 제거

Author: opficdev

Application/DevLogPresentation/Sources/Home/Editor/TodoEditorFeature.swift

@@ -230,7 +230,7 @@ extension DependencyValues {
         set { self[UpsertTodoUseCaseKey.self] = newValue }
     }
 
-    var trackAnalyticsEventUseCase: TrackAnalyticsEventUseCase? {
+    var trackAnalyticsEventUseCase: TrackAnalyticsEventUseCase {
         get { self[TrackAnalyticsEventUseCaseKey.self] }
         set { self[TrackAnalyticsEventUseCaseKey.self] = newValue }
     }
@@ -257,7 +257,17 @@ private enum UpsertTodoUseCaseKey: DependencyKey {
 }
 
 private enum TrackAnalyticsEventUseCaseKey: DependencyKey {
-    static let liveValue: TrackAnalyticsEventUseCase? = nil
+    static var liveValue: TrackAnalyticsEventUseCase {
+        preconditionFailure("TrackAnalyticsEventUseCase must be provided.")
+    }
+
+    static var testValue: TrackAnalyticsEventUseCase {
+        NoOpTrackAnalyticsEventUseCase()
+    }
+}
+
+private struct NoOpTrackAnalyticsEventUseCase: TrackAnalyticsEventUseCase {
+    func execute(_ event: AnalyticsEvent) { }
 }
 
 private extension TodoEditorFeature {
@@ -294,7 +304,7 @@ private extension TodoEditorFeature {
             await send(.loading(.begin(target: .default, mode: .immediate)))
             do {
                 try await upsertTodoUseCase.execute(draft)
-                trackAnalyticsEventUseCase?.execute(.todoCreate)
+                trackAnalyticsEventUseCase.execute(.todoCreate)
                 await send(.createSucceeded)
             } catch {
                 await send(.saveFailed)

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

@@ -73,7 +73,7 @@ extension HomeFeature {
             await send(.loading(.begin(target: LoadingTarget.overlay.target, mode: .delayed)))
             do {
                 try await addWebPageUseCase.execute(urlString)
-                trackAnalyticsEventUseCase?.execute(.webPageCreate)
+                trackAnalyticsEventUseCase.execute(.webPageCreate)
                 let pages = try await fetchWebPagesUseCase.execute("")
                 await send(.store(.updateWebPages(pages.map(WebPageItem.init(from:)))))
             } catch {

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

@@ -44,7 +44,7 @@ extension TodoListFeature {
                     todo.updatedAt = now
                     try await upsertTodoUseCase.execute(todo)
                     if todo.isCompleted {
-                        trackAnalyticsEventUseCase?.execute(.todoComplete)
+                        trackAnalyticsEventUseCase.execute(.todoComplete)
                     }
                     guard let todoListItem = TodoListItem(from: todo) else {
                         await send(.store(.setAlert(true)))

Application/DevLogPresentation/Sources/Main/MainFeature.swift

@@ -119,7 +119,7 @@ private extension MainFeature {
 
     func trackScreenViewEffect(_ screenName: String) -> Effect<Action> {
         .run { [trackAnalyticsEventUseCase] _ in
-            trackAnalyticsEventUseCase?.execute(.screenView(screenName))
+            trackAnalyticsEventUseCase.execute(.screenView(screenName))
         }
     }
 

Application/DevLogPresentation/Sources/Root/RootFeature.swift

@@ -183,7 +183,7 @@ private extension RootFeature {
 
     func trackLoginScreenEffect() -> Effect<Action> {
         .run { [trackAnalyticsEventUseCase] _ in
-            trackAnalyticsEventUseCase?.execute(.screenView("login"))
+            trackAnalyticsEventUseCase.execute(.screenView("login"))
         }
     }
 

Application/DevLogPresentation/Sources/Today/TodayFeature.swift

@@ -308,7 +308,7 @@ private extension TodayFeature {
                 todo.completedAt = now
                 todo.updatedAt = now
                 try await upsertTodoUseCase.execute(todo)
-                trackAnalyticsEventUseCase?.execute(.todoComplete)
+                trackAnalyticsEventUseCase.execute(.todoComplete)
                 await send(.store(.removeTodo(todo.id)))
                 await send(.loading(.end(target: .default, mode: .delayed)))
             } catch {

Application/DevLogPresentation/Tests/Home/HomeFeatureTestSupport.swift

@@ -55,7 +55,7 @@ struct HomeStoreTestAdapter {
         fetchTodosUseCase: FetchTodosUseCase = FetchTodosUseCaseSpy(),
         fetchWebPagesUseCase: FetchWebPagesUseCase = FetchWebPagesUseCaseSpy(webPages: []),
         networkConnectivityUseCase: ObserveNetworkConnectivityUseCase = ObserveNetworkConnectivityUseCaseSpy(),
-        trackAnalyticsEventUseCase: TrackAnalyticsEventUseCase? = HomeTrackAnalyticsEventUseCaseSpy(),
+        trackAnalyticsEventUseCase: TrackAnalyticsEventUseCase = HomeTrackAnalyticsEventUseCaseSpy(),
         configureDependencies: ((inout DependencyValues) -> Void)? = nil
     ) {
         let clock = TestClock()

Application/DevLogPresentation/Tests/Home/TodoEditorFeatureTestDoubles.swift

@@ -41,7 +41,7 @@ final class TodoEditorStoreTestAdapter {
         fetchPreferencesUseCase: FetchTodoCategoryPreferencesUseCase = TodoEditorFetchPreferencesUseCaseSpy(),
         fetchReferenceItemsUseCase: FetchReferenceItemsUseCase = TodoEditorFetchReferenceItemsUseCaseSpy(),
         upsertTodoUseCase: UpsertTodoUseCase = TodoEditorUpsertTodoUseCaseSpy(),
-        trackAnalyticsEventUseCase: TrackAnalyticsEventUseCase? = nil
+        trackAnalyticsEventUseCase: TrackAnalyticsEventUseCase = TodoEditorTrackAnalyticsEventUseCaseSpy()
     ) {
         self.now = now
         store = TestStore(