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 =
@@ -46,6 +44,7 @@ class SampleLauncherTest {
         composeRule.onNodeWithText(sample.description)
           .performClick()
         pressBack()
+        Thread.sleep(2000)
       } catch (e: Throwable) {
         throw AssertionError("Failed to launch sample ${sample.name}", e)
       }

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

@@ -206,8 +206,13 @@ private class WorkflowRuntimeState<PropsT, OutputT : Any, RenderingT>(
       onOutput = onOutput
     )
 
+    // TODO: Figure out why one test (scrolling a LazyColumn) these aren't getting set synchronously
+    renderingState.value = renderings.value.rendering
+    snapshotState.value = renderings.value.snapshot
+
     renderings
       .onEach {
+        // We have the rendering from above but its not passed to renderingState correctly? (it's still null)
         renderingState.value = it.rendering
         snapshotState.value = it.snapshot
       }