Temp commit for broken compose test

Author: steve-the-edwards

samples/compose-samples/src/androidTest/java/com/squareup/sample/compose/launcher/SampleLauncherTest.kt

@@ -1,6 +1,5 @@
 package com.squareup.sample.compose.launcher
 
-import androidx.compose.ui.test.ExperimentalTestApi
 import androidx.compose.ui.test.assertIsDisplayed
 import androidx.compose.ui.test.hasScrollToIndexAction
 import androidx.compose.ui.test.junit4.createAndroidComposeRule
@@ -31,7 +30,6 @@ class SampleLauncherTest {
     .around(composeRule)
     .around(IdlingDispatcherRule)
 
-  @OptIn(ExperimentalTestApi::class)
   @Test
   fun allSamplesLaunch() {
     val appName =

samples/compose-samples/src/main/java/com/squareup/sample/compose/inlinerendering/InlineRenderingWorkflow.kt

@@ -12,6 +12,7 @@ import androidx.compose.animation.slideOutVertically
 import androidx.compose.animation.with
 import androidx.compose.foundation.layout.Box
 import androidx.compose.material.Button
+import androidx.compose.material.MaterialTheme
 import androidx.compose.material.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.getValue
@@ -53,9 +54,8 @@ object InlineRenderingWorkflow : StatefulWorkflow<Unit, Int, Nothing, AndroidScr
   override fun snapshotState(state: Int): Snapshot = Snapshot.of(state)
 }
 
-@Preview
 @Composable
-fun InlineRenderingWorkflowPreview() {
+fun InlineRenderingWorkflowRendering() {
   val rendering by InlineRenderingWorkflow.renderAsState(
     props = Unit,
     onOutput = {},
@@ -64,6 +64,12 @@ fun InlineRenderingWorkflowPreview() {
   WorkflowRendering(rendering, ViewEnvironment.EMPTY)
 }
 
+@Preview(showBackground = true)
+@Composable
+internal fun InlineRenderingWorkflowPreview() {
+  InlineRenderingWorkflowRendering()
+}
+
 @OptIn(ExperimentalAnimationApi::class)
 @Composable
 private fun AnimatedCounter(

workflow-ui/compose/src/main/java/com/squareup/workflow1/ui/compose/RenderAsState.kt

@@ -23,12 +23,14 @@ import com.squareup.workflow1.WorkflowInterceptor
 import com.squareup.workflow1.renderWorkflowIn
 import com.squareup.workflow1.ui.WorkflowUiExperimentalApi
 import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.CoroutineStart.UNDISPATCHED
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.Job
 import kotlinx.coroutines.cancel
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.launchIn
 import kotlinx.coroutines.flow.onEach
+import kotlinx.coroutines.launch
 import kotlinx.coroutines.plus
 import okio.ByteString
 
@@ -206,18 +208,23 @@ private class WorkflowRuntimeState<PropsT, OutputT : Any, RenderingT>(
       onOutput = onOutput
     )
 
-    renderings
-      .onEach {
-        renderingState.value = it.rendering
-        snapshotState.value = it.snapshot
-      }
+    workflowScope.launch(
+      start = UNDISPATCHED,
+      context = Dispatchers.Unconfined
+    ) {
       // We collect the renderings in the workflowScope to participate in structured concurrency,
       // however we don't need to use its dispatcher – this collector is simply setting snapshot
       // state values, which is thread safe.
       // Also, if the scope uses a non-immediate dispatcher, the initial states won't get set until
       // the dispatcher dispatches the collection coroutine, but our contract requires them to be
-      // set by the time this function returns and using the Unconfined dispatcher guarantees that.
-      .launchIn(workflowScope + Dispatchers.Unconfined)
+      // set by the time this function returns and using the Unconfined dispatcher along with
+      // launching this coroutine as CoroutineStart.UNDISPATCHED guarantees that.
+
+      renderings.collect {
+        renderingState.value = it.rendering
+        snapshotState.value = it.snapshot
+      }
+    }
   }
 
   fun setProps(props: PropsT) {