Update voice memo test dependencies directly on store. (#1261)

(cherry picked from commit 3b96255304f2717e672a99db0058bd601c6ec6e0)

# Conflicts:
#	Examples/VoiceMemos/VoiceMemosTests/VoiceMemosTests.swift

Author: mbrandonw

Examples/VoiceMemos/VoiceMemos/RecordingMemo.swift

@@ -1,4 +1,5 @@
 import ComposableArchitecture
+import ReactiveSwift
 import SwiftUI
 
 struct RecordingMemoState: Equatable {
@@ -30,7 +31,7 @@ enum RecordingMemoAction: Equatable {
 
 struct RecordingMemoEnvironment {
   var audioRecorder: AudioRecorderClient
-  var mainRunLoop: AnySchedulerOf<RunLoop>
+  var mainRunLoop: DateScheduler
 }
 
 let recordingMemoReducer = Reducer<

Examples/VoiceMemos/VoiceMemos/VoiceMemos.swift

@@ -78,7 +78,7 @@ let voiceMemosReducer = Reducer<VoiceMemosState, VoiceMemosAction, VoiceMemosEnv
 
       case .allowed:
         state.recordingMemo = RecordingMemoState(
-          date: environment.mainRunLoop.now.date,
+          date: environment.mainRunLoop.currentDate,
           url: environment.temporaryDirectory()
             .appendingPathComponent(environment.uuid().uuidString)
             .appendingPathExtension("m4a")
@@ -110,7 +110,7 @@ let voiceMemosReducer = Reducer<VoiceMemosState, VoiceMemosAction, VoiceMemosEnv
       state.audioRecorderPermission = permission ? .allowed : .denied
       if permission {
         state.recordingMemo = RecordingMemoState(
-          date: environment.mainRunLoop.now.date,
+          date: environment.mainRunLoop.currentDate,
           url: environment.temporaryDirectory()
             .appendingPathComponent(environment.uuid().uuidString)
             .appendingPathExtension("m4a")

Examples/VoiceMemos/VoiceMemosTests/VoiceMemosTests.swift

@@ -14,27 +14,26 @@ class VoiceMemosTests: XCTestCase {
     // NB: Combine's concatenation behavior is different in 13.3
     guard #available(iOS 13.4, *) else { return }
 
+    let store = TestStore(
+      initialState: VoiceMemosState(),
+      reducer: voiceMemosReducer,
+      environment: .unimplemented
+    )
+
     let didFinish = AsyncThrowingStream<Bool, Error>.streamWithContinuation()
 
-    var environment = VoiceMemosEnvironment.unimplemented
-    environment.audioRecorder.currentTime = { 2.5 }
-    environment.audioRecorder.requestRecordPermission = { true }
-    environment.audioRecorder.startRecording = { _ in
+    store.environment.audioRecorder.currentTime = { 2.5 }
+    store.environment.audioRecorder.requestRecordPermission = { true }
+    store.environment.audioRecorder.startRecording = { _ in
       try await didFinish.stream.first { _ in true }!
     }
-    environment.audioRecorder.stopRecording = {
+    store.environment.audioRecorder.stopRecording = {
       didFinish.continuation.yield(true)
       didFinish.continuation.finish()
     }
-    environment.mainRunLoop = mainRunLoop
-    environment.temporaryDirectory = { URL(fileURLWithPath: "/tmp") }
-    environment.uuid = { UUID(uuidString: "DEADBEEF-DEAD-BEEF-DEAD-BEEFDEADBEEF")! }
-
-    let store = TestStore(
-      initialState: VoiceMemosState(),
-      reducer: voiceMemosReducer,
-      environment: environment
-    )
+    store.environment.mainRunLoop = self.mainRunLoop
+    store.environment.temporaryDirectory = { URL(fileURLWithPath: "/tmp") }
+    store.environment.uuid = { UUID(uuidString: "DEADBEEF-DEAD-BEEF-DEAD-BEEFDEADBEEF")! }
 
     await store.send(.recordButtonTapped)
     await self.mainRunLoop.advance()
@@ -82,17 +81,16 @@ class VoiceMemosTests: XCTestCase {
   func testPermissionDenied() async {
     let didOpenSettings = ActorIsolated(false)
 
-    var environment = VoiceMemosEnvironment.unimplemented
-    environment.audioRecorder.requestRecordPermission = { false }
-    environment.mainRunLoop = mainRunLoop
-    environment.openSettings = { await didOpenSettings.setValue(true) }
-
     let store = TestStore(
       initialState: VoiceMemosState(),
       reducer: voiceMemosReducer,
-      environment: environment
+      environment: .unimplemented
     )
 
+    store.environment.audioRecorder.requestRecordPermission = { false }
+    store.environment.mainRunLoop = ImmediateScheduler()
+    store.environment.openSettings = { await didOpenSettings.setValue(true) }
+
     await store.send(.recordButtonTapped)
     await store.receive(.recordPermissionResponse(false)) {
       $0.alert = AlertState(title: TextState("Permission is required to record voice memos."))
@@ -107,24 +105,23 @@ class VoiceMemosTests: XCTestCase {
 
   func testRecordMemoFailure() async {
     struct SomeError: Error, Equatable {}
-    let didFinish = AsyncThrowingStream<Bool, Error>.streamWithContinuation()
-
-    var environment = VoiceMemosEnvironment.unimplemented
-    environment.audioRecorder.currentTime = { 2.5 }
-    environment.audioRecorder.requestRecordPermission = { true }
-    environment.audioRecorder.startRecording = { _ in
-      try await didFinish.stream.first { _ in true }!
-    }
-    environment.mainRunLoop = TestScheduler(startDate: Date(timeIntervalSince1970: 0))
-    environment.temporaryDirectory = { URL(fileURLWithPath: "/tmp") }
-    environment.uuid = { UUID(uuidString: "DEADBEEF-DEAD-BEEF-DEAD-BEEFDEADBEEF")! }
 
     let store = TestStore(
       initialState: VoiceMemosState(),
       reducer: voiceMemosReducer,
-      environment: environment
+      environment: .unimplemented
     )
 
+    let didFinish = AsyncThrowingStream<Bool, Error>.streamWithContinuation()
+
+    store.environment.audioRecorder.requestRecordPermission = { true }
+    store.environment.audioRecorder.startRecording = { _ in
+      try await didFinish.stream.first { _ in true }!
+    }
+    store.environment.mainRunLoop = TestScheduler(startDate: Date(timeIntervalSince1970: 0))
+    store.environment.temporaryDirectory = { URL(fileURLWithPath: "/tmp") }
+    store.environment.uuid = { UUID(uuidString: "DEADBEEF-DEAD-BEEF-DEAD-BEEFDEADBEEF")! }
+
     await store.send(.recordButtonTapped)
     await self.mainRunLoop.advance(by: 0.5)
     await store.receive(.recordPermissionResponse(true)) {
@@ -149,13 +146,6 @@ class VoiceMemosTests: XCTestCase {
   }
 
   func testPlayMemoHappyPath() async {
-    var environment = VoiceMemosEnvironment.unimplemented
-    environment.audioPlayer.play = { _ in
-      try await self.mainRunLoop.sleep(for: .milliseconds(1250))
-      return true
-    }
-    environment.mainRunLoop = mainRunLoop
-
     let url = URL(fileURLWithPath: "pointfreeco/functions.m4a")
     let store = TestStore(
       initialState: VoiceMemosState(
@@ -170,9 +160,15 @@ class VoiceMemosTests: XCTestCase {
         ]
       ),
       reducer: voiceMemosReducer,
-      environment: environment
+      environment: .unimplemented
     )
 
+    store.environment.audioPlayer.play = { _ in
+      try await self.mainRunLoop.sleep(for: .milliseconds(1250))
+      return true
+    }
+    store.environment.mainRunLoop = self.mainRunLoop
+
     let task = await store.send(.voiceMemo(id: url, action: .playButtonTapped)) {
       $0.voiceMemos[id: url]?.mode = .playing(progress: 0)
     }
@@ -194,10 +190,6 @@ class VoiceMemosTests: XCTestCase {
   func testPlayMemoFailure() async {
     struct SomeError: Error, Equatable {}
 
-    var environment = VoiceMemosEnvironment.unimplemented
-    environment.audioPlayer.play = { _ in throw SomeError() }
-    environment.mainRunLoop = mainRunLoop
-
     let url = URL(fileURLWithPath: "pointfreeco/functions.m4a")
     let store = TestStore(
       initialState: VoiceMemosState(
@@ -212,9 +204,12 @@ class VoiceMemosTests: XCTestCase {
         ]
       ),
       reducer: voiceMemosReducer,
-      environment: environment
+      environment: .unimplemented
     )
 
+    store.environment.audioPlayer.play = { _ in throw SomeError() }
+    store.environment.mainRunLoop = self.mainRunLoop
+
     let task = await store.send(.voiceMemo(id: url, action: .playButtonTapped)) {
       $0.voiceMemos[id: url]?.mode = .playing(progress: 0)
     }
@@ -273,9 +268,6 @@ class VoiceMemosTests: XCTestCase {
 
   func testDeleteMemoWhilePlaying() async {
     let url = URL(fileURLWithPath: "pointfreeco/functions.m4a")
-    var environment = VoiceMemosEnvironment.unimplemented
-    environment.audioPlayer.play = { _ in try await Task.never() }
-    environment.mainRunLoop = mainRunLoop
 
     let store = TestStore(
       initialState: VoiceMemosState(
@@ -290,9 +282,12 @@ class VoiceMemosTests: XCTestCase {
         ]
       ),
       reducer: voiceMemosReducer,
-      environment: environment
+      environment: .unimplemented
     )
 
+    store.environment.audioPlayer.play = { _ in try await Task.never() }
+    store.environment.mainRunLoop = self.mainRunLoop
+
     await store.send(.voiceMemo(id: url, action: .playButtonTapped)) {
       $0.voiceMemos[id: url]?.mode = .playing(progress: 0)
     }