fix action tester

Author: square-tomb

Samples/Tutorial/Frameworks/Tutorial5Complete/Tests/TodoEditWorkflowTests.swift

@@ -22,7 +22,7 @@ class TodoEditWorkflowTests: XCTestCase {
     func testAction() throws {
         TodoEditWorkflow.Action
             // Start with a todo of "Title" "Note"
-            .tester(withState: TodoEditWorkflow.State(todo: TodoModel(title: "Title", note: "Note")))
+            .tester(workflow: TodoEditWorkflow(initialTodo: TodoModel(title: "Title", note: "Note")))
             .verifyState { state in
                 XCTAssertEqual("Title", state.todo.title)
                 XCTAssertEqual("Note", state.todo.note)

Samples/Tutorial/Frameworks/Tutorial5Complete/Tests/TodoListWorkflowTests.swift

@@ -21,7 +21,7 @@ import XCTest
 class TodoListWorkflowTests: XCTestCase {
     func testActions() throws {
         TodoListWorkflow.Action
-            .tester(withState: TodoListWorkflow.State())
+            .tester(workflow: TodoListWorkflow(name: "", todos: []))
             .send(action: .onBack)
             .verifyOutput { output in
                 // The `.onBack` action should emit an output of `.back`.

Samples/Tutorial/Frameworks/Tutorial5Complete/Tests/WelcomeWorkflowTests.swift

@@ -21,7 +21,7 @@ import XCTest
 class WelcomeWorkflowTests: XCTestCase {
     func testNameUpdates() throws {
         WelcomeWorkflow.Action
-            .tester(withState: WelcomeWorkflow.State(name: ""))
+            .tester(workflow: WelcomeWorkflow())
             .send(action: .nameChanged(name: "myName"))
             // No output is expected when the name changes.
             .assertNoOutput()
@@ -33,7 +33,7 @@ class WelcomeWorkflowTests: XCTestCase {
 
     func testLogIn() throws {
         WelcomeWorkflow.Action
-            .tester(withState: WelcomeWorkflow.State(name: ""))
+            .tester(workflow: WelcomeWorkflow())
             .send(action: .didLogIn)
             // Since the name is empty, `.didLogIn` will not emit an output.
             .assertNoOutput()

Workflow/Tests/AnyWorkflowActionTests.swift

@@ -83,7 +83,7 @@ final class AnyWorkflowActionTests: XCTestCase {
         XCTAssertEqual(log, [])
 
         var state: Void = ()
-        _ = erased.apply(toState: &state)
+        _ = erased.apply(toState: &state, workflow: ExampleWorkflow())
 
         XCTAssertEqual(log, ["action invoked"])
     }

WorkflowTesting/Sources/WorkflowActionTester.swift

@@ -18,9 +18,17 @@ import Workflow
 import XCTest
 
 extension WorkflowAction {
-    /// Returns a state tester containing `self`.
-    public static func tester(withState state: WorkflowType.State) -> WorkflowActionTester<WorkflowType, Self> {
-        WorkflowActionTester(state: state)
+//    /// Returns a state tester containing `self`.
+//    public static func tester(withState state: WorkflowType.State) -> WorkflowActionTester<WorkflowType, Self> {
+//        WorkflowActionTester(state: state)
+//    }
+
+    public static func tester(workflow: WorkflowType, state: WorkflowType.State) -> WorkflowActionTester<WorkflowType, Self> {
+        WorkflowActionTester(workflow: workflow, state: state)
+    }
+
+    public static func tester(workflow: WorkflowType) -> WorkflowActionTester<WorkflowType, Self> {
+        tester(workflow: workflow, state: workflow.makeInitialState())
     }
 }
 
@@ -61,11 +69,13 @@ extension WorkflowAction {
 /// ```
 public struct WorkflowActionTester<WorkflowType, Action> where Action: WorkflowAction, Action.WorkflowType == WorkflowType {
     /// The current state
+    let workflow: WorkflowType
     let state: WorkflowType.State
     let output: WorkflowType.Output?
 
     /// Initializes a new state tester
-    fileprivate init(state: WorkflowType.State, output: WorkflowType.Output? = nil) {
+    fileprivate init(workflow: WorkflowType, state: WorkflowType.State, output: WorkflowType.Output? = nil) {
+        self.workflow = workflow
         self.state = state
         self.output = output
     }
@@ -78,8 +88,8 @@ public struct WorkflowActionTester<WorkflowType, Action> where Action: WorkflowA
     @discardableResult
     public func send(action: Action) -> WorkflowActionTester<WorkflowType, Action> {
         var newState = state
-        let output = action.apply(toState: &newState)
-        return WorkflowActionTester(state: newState, output: output)
+        let output = action.apply(toState: &newState, workflow: workflow)
+        return WorkflowActionTester(workflow: workflow, state: newState, output: output)
     }
 
     /// Asserts that the action produced no output

WorkflowTesting/Tests/WorkflowActionTesterTests.swift

@@ -21,14 +21,14 @@ import XCTest
 final class WorkflowActionTesterTests: XCTestCase {
     func test_stateTransitions() {
         TestAction
-            .tester(withState: false)
+            .tester(withState: false, workflow: TestWorkflow())
             .send(action: .toggleTapped)
             .verifyState { XCTAssertTrue($0) }
     }
 
     func test_stateTransitions_throw() throws {
         try TestAction
-            .tester(withState: false)
+            .tester(withState: false, workflow: TestWorkflow())
             .send(action: .toggleTapped)
             .verifyState {
                 try throwingNoop()
@@ -38,21 +38,21 @@ final class WorkflowActionTesterTests: XCTestCase {
 
     func test_stateTransitions_equatable() {
         TestAction
-            .tester(withState: false)
+            .tester(withState: false, workflow: TestWorkflow())
             .send(action: .toggleTapped)
             .assert(state: true)
     }
 
     func test_noOutputs() {
         TestAction
-            .tester(withState: false)
+            .tester(withState: false, workflow: TestWorkflow())
             .send(action: .toggleTapped)
             .assertNoOutput()
     }
 
     func test_outputs() {
         TestAction
-            .tester(withState: false)
+            .tester(withState: false, workflow: TestWorkflow())
             .send(action: .exitTapped)
             .verifyOutput { output in
                 XCTAssertEqual(output, .finished)
@@ -61,7 +61,7 @@ final class WorkflowActionTesterTests: XCTestCase {
 
     func test_outputs_throw() throws {
         try TestAction
-            .tester(withState: false)
+            .tester(withState: false, workflow: TestWorkflow())
             .send(action: .exitTapped)
             .verifyOutput { output in
                 try throwingNoop()
@@ -71,14 +71,14 @@ final class WorkflowActionTesterTests: XCTestCase {
 
     func test_outputs_equatable() {
         TestAction
-            .tester(withState: false)
+            .tester(withState: false, workflow: TestWorkflow())
             .send(action: .exitTapped)
             .assert(output: .finished)
     }
 
     func test_deprecated_methods() {
         TestAction
-            .tester(withState: false)
+            .tester(withState: false, workflow: TestWorkflow())
             .send(action: .exitTapped)
             .assert(output: .finished)
             .verifyState { state in
@@ -89,7 +89,7 @@ final class WorkflowActionTesterTests: XCTestCase {
     func test_testerExtension() {
         let state = true
         let tester = TestAction
-            .tester(withState: true)
+            .tester(withState: true, workflow: TestWorkflow())
         XCTAssertEqual(state, tester.state)
         XCTAssertNil(tester.output)
     }