-
Notifications
You must be signed in to change notification settings - Fork 101
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
1093: Yield in runtime to run side effects that were launched lazily
Add headlessIntegrationTest to renderWorkflowIn with a nice Turbine attached. Closes #1093
- Loading branch information
1 parent
551ee52
commit f896709
Showing
7 changed files
with
413 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
189 changes: 189 additions & 0 deletions
189
workflow-testing/src/main/java/com/squareup/workflow1/testing/HeadlessIntegrationTest.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,189 @@ | ||
| package com.squareup.workflow1.testing | ||
|
|
||
| import app.cash.turbine.ReceiveTurbine | ||
| import app.cash.turbine.test | ||
| import com.squareup.workflow1.RuntimeConfig | ||
| import com.squareup.workflow1.RuntimeConfigOptions | ||
| import com.squareup.workflow1.Workflow | ||
| import com.squareup.workflow1.WorkflowInterceptor | ||
| import com.squareup.workflow1.renderWorkflowIn | ||
| import com.squareup.workflow1.testing.WorkflowTurbine.Companion.WORKFLOW_TEST_DEFAULT_TIMEOUT_MS | ||
| import kotlinx.coroutines.CoroutineScope | ||
| import kotlinx.coroutines.ExperimentalCoroutinesApi | ||
| import kotlinx.coroutines.cancel | ||
| import kotlinx.coroutines.flow.MutableStateFlow | ||
| import kotlinx.coroutines.flow.StateFlow | ||
| import kotlinx.coroutines.flow.asStateFlow | ||
| import kotlinx.coroutines.flow.drop | ||
| import kotlinx.coroutines.flow.map | ||
| import kotlinx.coroutines.test.UnconfinedTestDispatcher | ||
| import kotlinx.coroutines.test.runTest | ||
| import kotlin.coroutines.CoroutineContext | ||
| import kotlin.time.Duration.Companion.milliseconds | ||
|
|
||
| /** | ||
| * This is a test harness to run integration tests for a Workflow tree. The parameters passed here are | ||
| * the same as those to start a Workflow runtime with [renderWorkflowIn] except for ignoring | ||
| * state persistence as that is not needed for this style of test. | ||
| * | ||
| * The [coroutineContext] rather than a [CoroutineScope] is passed so that this harness handles the | ||
| * scope for the Workflow runtime for you but you can still specify context for it. | ||
| * | ||
| * A [testTimeout] may be specified to override the default [WORKFLOW_TEST_DEFAULT_TIMEOUT_MS] for | ||
| * any particular test. This is the max amount of time the test could spend waiting on a rendering. | ||
| * | ||
| * This will start the Workflow runtime (with params as passed) rooted at whatever Workflow | ||
| * it is called on and then create a [WorkflowTurbine] for its renderings and run [testCase] on that. | ||
| * [testCase] can thus drive the test scenario and assert against renderings. | ||
| */ | ||
| @OptIn(ExperimentalCoroutinesApi::class) | ||
| public fun <PropsT, OutputT, RenderingT> Workflow<PropsT, OutputT, RenderingT>.headlessIntegrationTest( | ||
| props: StateFlow<PropsT>, | ||
| coroutineContext: CoroutineContext = UnconfinedTestDispatcher(), | ||
| interceptors: List<WorkflowInterceptor> = emptyList(), | ||
| runtimeConfig: RuntimeConfig = RuntimeConfigOptions.DEFAULT_CONFIG, | ||
| onOutput: suspend (OutputT) -> Unit = {}, | ||
| testTimeout: Long = WORKFLOW_TEST_DEFAULT_TIMEOUT_MS, | ||
| testCase: suspend WorkflowTurbine<RenderingT>.() -> Unit | ||
| ) { | ||
| val workflow = this | ||
|
|
||
| runTest( | ||
| context = coroutineContext, | ||
| timeout = testTimeout.milliseconds | ||
| ) { | ||
| // We use a sub-scope so that we can cancel the Workflow runtime when we are done with it so that | ||
| // tests don't all have to do that themselves. | ||
| val workflowRuntimeScope = CoroutineScope(coroutineContext) | ||
| val renderings = renderWorkflowIn( | ||
| workflow = workflow, | ||
| props = props, | ||
| scope = workflowRuntimeScope, | ||
| interceptors = interceptors, | ||
| runtimeConfig = runtimeConfig, | ||
| onOutput = onOutput | ||
| ) | ||
|
|
||
| val firstRendering = renderings.value.rendering | ||
|
|
||
| // Drop one as its provided separately via `firstRendering`. | ||
| renderings.drop(1).map { | ||
| it.rendering | ||
| }.test { | ||
| val workflowTurbine = WorkflowTurbine( | ||
| firstRendering, | ||
| this | ||
| ) | ||
| workflowTurbine.testCase() | ||
| cancelAndIgnoreRemainingEvents() | ||
| } | ||
| workflowRuntimeScope.cancel() | ||
| } | ||
| } | ||
|
|
||
| /** | ||
| * Version of [headlessIntegrationTest] that does not require props. For Workflows that have [Unit] | ||
| * props type. | ||
| */ | ||
| @OptIn(ExperimentalCoroutinesApi::class) | ||
| public fun <OutputT, RenderingT> Workflow<Unit, OutputT, RenderingT>.headlessIntegrationTest( | ||
| coroutineContext: CoroutineContext = UnconfinedTestDispatcher(), | ||
| interceptors: List<WorkflowInterceptor> = emptyList(), | ||
| runtimeConfig: RuntimeConfig = RuntimeConfigOptions.DEFAULT_CONFIG, | ||
| onOutput: suspend (OutputT) -> Unit = {}, | ||
| testTimeout: Long = WORKFLOW_TEST_DEFAULT_TIMEOUT_MS, | ||
| testCase: suspend WorkflowTurbine<RenderingT>.() -> Unit | ||
| ): Unit = headlessIntegrationTest( | ||
| props = MutableStateFlow(Unit).asStateFlow(), | ||
| coroutineContext = coroutineContext, | ||
| interceptors = interceptors, | ||
| runtimeConfig = runtimeConfig, | ||
| onOutput = onOutput, | ||
| testTimeout = testTimeout, | ||
| testCase = testCase | ||
| ) | ||
|
|
||
| /** | ||
| * Simple wrapper around a [ReceiveTurbine] of [RenderingT] to provide convenience helper methods specific | ||
| * to Workflow renderings. | ||
| * | ||
| * @property firstRendering The first rendering of the Workflow runtime is made synchronously. This is | ||
| * provided separately if any assertions or operations are needed from it. | ||
| */ | ||
| public class WorkflowTurbine<RenderingT>( | ||
| public val firstRendering: RenderingT, | ||
| private val receiveTurbine: ReceiveTurbine<RenderingT> | ||
| ) { | ||
| private var usedFirst = false | ||
|
|
||
| /** | ||
| * Suspend waiting for the next rendering to be produced by the Workflow runtime. Note this includes | ||
| * the first (synchronously made) rendering. | ||
| * | ||
| * @return the rendering. | ||
| */ | ||
| public suspend fun awaitNextRendering(): RenderingT { | ||
| if (!usedFirst) { | ||
| usedFirst = true | ||
| return firstRendering | ||
| } | ||
| return receiveTurbine.awaitItem() | ||
| } | ||
|
|
||
| public suspend fun skipRenderings(count: Int) { | ||
| val skippedCount = if (!usedFirst) { | ||
| usedFirst = true | ||
| count - 1 | ||
| } else { | ||
| count | ||
| } | ||
|
|
||
| if (skippedCount > 0) { | ||
| receiveTurbine.skipItems(skippedCount) | ||
| } | ||
| } | ||
|
|
||
| /** | ||
| * Suspend waiting for the next rendering to be produced by the Workflow runtime that satisfies the | ||
| * [predicate]. | ||
| * | ||
| * @return the rendering. | ||
| */ | ||
| public suspend fun awaitNextRenderingSatisfying( | ||
| predicate: (RenderingT) -> Boolean | ||
| ): RenderingT { | ||
| var rendering = awaitNextRendering() | ||
| while (!predicate(rendering)) { | ||
| rendering = awaitNextRendering() | ||
| } | ||
| return rendering | ||
| } | ||
|
|
||
| /** | ||
| * Suspend waiting for the next rendering which satisfies [precondition], can successfully be mapped | ||
| * using [map] and satisfies the [satisfying] predicate when called on the [T] rendering after it | ||
| * has been mapped. | ||
| * | ||
| * @return the mapped rendering as [T] | ||
| */ | ||
| public suspend fun <T> awaitNext( | ||
| precondition: (RenderingT) -> Boolean = { true }, | ||
| map: (RenderingT) -> T, | ||
| satisfying: T.() -> Boolean = { true } | ||
| ): T = | ||
| map( | ||
| awaitNextRenderingSatisfying { | ||
| precondition(it) && | ||
| with(map(it)) { | ||
| this.satisfying() | ||
| } | ||
| } | ||
| ) | ||
|
|
||
| public companion object { | ||
| /** | ||
| * Default timeout to use while waiting for renderings. | ||
| */ | ||
| public const val WORKFLOW_TEST_DEFAULT_TIMEOUT_MS: Long = 60_000L | ||
| } | ||
| } |
69 changes: 69 additions & 0 deletions
69
workflow-testing/src/test/java/com/squareup/workflow1/ParameterizedTestRunner.kt
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,69 @@ | ||
| package com.squareup.workflow1 | ||
|
|
||
| import kotlin.test.assertEquals | ||
| import kotlin.test.assertFailsWith | ||
| import kotlin.test.assertFalse | ||
| import kotlin.test.assertNotNull | ||
| import kotlin.test.assertNotSame | ||
| import kotlin.test.assertNull | ||
| import kotlin.test.assertTrue | ||
|
|
||
| /** | ||
| * This file is copied from workflow-runtime:commonTest so our tests that test across the runtime | ||
| * look consistent. We could have used a JUnit library like Jupiter, but didn't. | ||
| * | ||
| * This file is copied so as to avoid creating a workflow-core-testing module (for now). | ||
| * | ||
| * We do our best to tell you what the parameter was when the failure occured by wrapping | ||
| * assertions from kotlin.test and injecting our own message. | ||
| */ | ||
| class ParameterizedTestRunner<P : Any> { | ||
|
|
||
| var currentParam: P? = null | ||
|
|
||
| fun runParametrizedTest( | ||
| paramSource: Sequence<P>, | ||
| before: () -> Unit = {}, | ||
| after: () -> Unit = {}, | ||
| test: ParameterizedTestRunner<P>.(param: P) -> Unit | ||
| ) { | ||
| paramSource.forEach { | ||
| before() | ||
| currentParam = it | ||
| test(it) | ||
| after() | ||
| } | ||
| } | ||
|
|
||
| fun <T> assertEquals(expected: T, actual: T) { | ||
| assertEquals(expected, actual, message = "Using: ${currentParam?.toString()}") | ||
| } | ||
|
|
||
| fun <T> assertEquals(expected: T, actual: T, originalMessage: String) { | ||
| assertEquals(expected, actual, message = "$originalMessage; Using: ${currentParam?.toString()}") | ||
| } | ||
|
|
||
| fun assertTrue(statement: Boolean) { | ||
| assertTrue(statement, message = "Using: ${currentParam?.toString()}") | ||
| } | ||
|
|
||
| fun assertFalse(statement: Boolean) { | ||
| assertFalse(statement, message = "Using: ${currentParam?.toString()}") | ||
| } | ||
|
|
||
| inline fun <reified T : Throwable> assertFailsWith(block: () -> Unit) { | ||
| assertFailsWith<T>(message = "Using: ${currentParam?.toString()}", block) | ||
| } | ||
|
|
||
| fun <T : Any?> assertNotSame(illegal: T, actual: T) { | ||
| assertNotSame(illegal, actual, message = "Using: ${currentParam?.toString()}") | ||
| } | ||
|
|
||
| fun <T : Any> assertNotNull(actual: T?) { | ||
| assertNotNull(actual, message = "Using: ${currentParam?.toString()}") | ||
| } | ||
|
|
||
| fun assertNull(actual: Any?) { | ||
| assertNull(actual, message = "Using: ${currentParam?.toString()}") | ||
| } | ||
| } |
Oops, something went wrong.