fix: error descriptions

wearemakery · Feb 2, 2021 · b29a437 · b29a437
1 parent c53e43b
commit b29a437
Showing 1 changed file with 25 additions and 7 deletions.
diff --git a/composable-architecture-test/src/main/kotlin/composablearchitecture/test/TestStore.kt b/composable-architecture-test/src/main/kotlin/composablearchitecture/test/TestStore.kt
@@ -70,6 +70,20 @@ private constructor(
                 Prism.id(),
                 testDispatcher
             )
+        fun <State, Action : Comparable<Action>, Environment> create (
+            state: State,
+            reducer: Reducer<State, Action, Environment>,
+            environment: Environment,
+            testDispatcher: TestCoroutineDispatcher = TestCoroutineDispatcher()
+        ) =
+            TestStore(
+                state,
+                reducer,
+                environment,
+                Lens.id(),
+                Prism.id(),
+                testDispatcher
+            )
     }
 
     fun <S, A> scope(
@@ -119,28 +133,32 @@ private constructor(
                     expectedState = step.block(expectedState)
                 }
                 is Step.Receive<LocalAction, LocalState, Environment> -> {
-                    require(receivedActions.isNotEmpty()) { "Expected to receive" + receivedActions.joinToString(separator = "\n") + "but received none" }
+                    require(receivedActions.isNotEmpty()) {
+                        "Expected to receive" + receivedActions.joinToString(separator = "\n") + "but received none"
+                    }
                     val receivedAction = receivedActions.removeFirst()
-                    require(step.action == receivedAction) { "Actual and expected actions do not match" }
+                    require(step.action == receivedAction) {
+                        "Actual and expected states do not match.\n\n${step.action}\n---vs---\n$receivedAction"
+                    }
                     runReducer(fromLocalAction.reverseGet(step.action))
                     expectedState = step.block(expectedState)
                 }
                 is Step.Environment<LocalAction, LocalState, Environment> -> {
-                    require(receivedActions.isEmpty()) { "Must handle all received actions before performing this work." + receivedActions.joinToString(separator = "\n") + "are not handled" }
+                    require(receivedActions.isEmpty()) {
+                        "Must handle all received actions before performing this work." + receivedActions.joinToString(separator = "\n") + "are not handled"
+                    }
                     step.block(environment)
                 }
                 is Step.Do -> step.block()
             }
 
             val actualState = toLocalState.get(state)
             require(actualState == expectedState) {
-                println(actualState)
-                println("---vs---")
-                println(expectedState)
-                "Actual and expected states do not match"
+                "Actual and expected states do not match\n\n$actualState\n---vs---\n$expectedState"
             }
         }
 
         require(receivedActions.isEmpty()) { "Must handle all actions. Unhandled actions:\n" + receivedActions.joinToString(separator = "\n") }
     }
 }
+