Adds STABLE_EVENT_HANDLERS to runtime config tools

Overhauls the config calls. Doesn't actually use the new config option yet, incremental change due to CI problems.

Author: rjrjr

workflow-config/config-android/src/main/java/com/squareup/workflow1/config/AndroidRuntimeConfigTools.kt

@@ -5,52 +5,54 @@ import com.squareup.workflow1.RuntimeConfigOptions
 import com.squareup.workflow1.RuntimeConfigOptions.CONFLATE_STALE_RENDERINGS
 import com.squareup.workflow1.RuntimeConfigOptions.PARTIAL_TREE_RENDERING
 import com.squareup.workflow1.RuntimeConfigOptions.RENDER_ONLY_WHEN_STATE_CHANGES
+import com.squareup.workflow1.RuntimeConfigOptions.STABLE_EVENT_HANDLERS
 import com.squareup.workflow1.WorkflowExperimentalRuntime
 
 public class AndroidRuntimeConfigTools {
-
   public companion object {
     /**
      * Helper for Configuration for the workflow runtime in an application.
      * This allows one to specify a project property from the gradle build to choose a runtime.
-     * e.g. add "-Pworkflow.runtime=conflate" in your gradle build to build the conflate runtime
+     * e.g. add `-Pworkflow.runtime=conflate` in your gradle build to build the conflate runtime
      * into the application.
      *
      * Note that this must be specified in the application built for any ui/integration tests. Call
-     * this function, and then pass that to the call to [renderWorkflowIn] as the [RuntimeConfig].
+     * this function and pass the result to [renderWorkflowIn][com.squareup.workflow1.renderWorkflowIn]
+     * as the [RuntimeConfig] parameter.
+     *
+     * Current options (can be combined with `-` characters, e.g. `conflate-partial`):
      *
-     * Current options are:
-     * "conflate" : Process all queued actions before passing rendering
-     *      to the UI layer.
-     * "baseline" : Original Workflow Runtime. Note that this doesn't need to
-     *      be specified as it is the current default and is assumed by this utility.
+     * - `conflate` Process all queued actions before passing rendering to the UI layer.
      *
-     * Then, these can be combined (via '-') with:
-     * "stateChange" : Only re-render when the state of some WorkflowNode has been changed by an
+     * - `stateChange` Only re-render when the state of some WorkflowNode has been changed by an
      *   action cascade.
-     * "partial" : Which includes "stateChange" as well as partial tree rendering, which only
-     *   re-renders each Workflow node if: 1) its state changed; or 2) one of its descendant's state
-     *   changed.
      *
-     * E.g., "baseline-stateChange" to turn on the stateChange option with the baseline runtime.
+     * - `partial` Partial tree rendering, which only re-renders each Workflow node if: 1) its
+     *    state changed; or 2) one of its descendant's state changed. (This option requires
+     *    `stateChange`, and enables it as well.)
      *
+     * - `stable` Enables stable event handlers (changes the default value of the `remember`
+     *    parameter of `RenderContext.eventHandler` functions from `false` to `true`)
      */
     @WorkflowExperimentalRuntime
     public fun getAppWorkflowRuntimeConfig(): RuntimeConfig {
-      return when (BuildConfig.WORKFLOW_RUNTIME) {
-        "conflate" -> setOf(CONFLATE_STALE_RENDERINGS)
-        "conflate-stateChange" -> setOf(CONFLATE_STALE_RENDERINGS, RENDER_ONLY_WHEN_STATE_CHANGES)
-        "baseline-stateChange" -> setOf(RENDER_ONLY_WHEN_STATE_CHANGES)
-        "conflate-partial" -> setOf(
-          CONFLATE_STALE_RENDERINGS,
-          RENDER_ONLY_WHEN_STATE_CHANGES,
-          PARTIAL_TREE_RENDERING
-        )
-        "baseline-partial" -> setOf(RENDER_ONLY_WHEN_STATE_CHANGES, PARTIAL_TREE_RENDERING)
-        "", "baseline" -> RuntimeConfigOptions.RENDER_PER_ACTION
-        else ->
-          throw IllegalArgumentException("Unrecognized config \"${BuildConfig.WORKFLOW_RUNTIME}\"")
+      val selection = BuildConfig.WORKFLOW_RUNTIME.split("-")
+        // We used to have a no-op `baseline` option, let's not choke on it.
+        .filterNot { it == "baseline" }
+        .toSet()
+
+      val config = mutableSetOf<RuntimeConfigOptions>()
+      selection.forEach {
+        when (it) {
+          "conflate" -> config.add(CONFLATE_STALE_RENDERINGS)
+          "stateChange" -> config.add(RENDER_ONLY_WHEN_STATE_CHANGES)
+          "partial" -> config.addAll(setOf(RENDER_ONLY_WHEN_STATE_CHANGES, PARTIAL_TREE_RENDERING))
+          "stable" -> config.add(STABLE_EVENT_HANDLERS)
+          else -> throw IllegalArgumentException("Unrecognized runtime config option \"$it\"")
+        }
       }
+
+      return config
     }
   }
 }

workflow-config/config-jvm/src/main/java/com/squareup/workflow1/config/JvmTestRuntimeConfigTools.kt

@@ -5,53 +5,56 @@ import com.squareup.workflow1.RuntimeConfigOptions
 import com.squareup.workflow1.RuntimeConfigOptions.CONFLATE_STALE_RENDERINGS
 import com.squareup.workflow1.RuntimeConfigOptions.PARTIAL_TREE_RENDERING
 import com.squareup.workflow1.RuntimeConfigOptions.RENDER_ONLY_WHEN_STATE_CHANGES
+import com.squareup.workflow1.RuntimeConfigOptions.STABLE_EVENT_HANDLERS
 import com.squareup.workflow1.WorkflowExperimentalRuntime
 
 public class JvmTestRuntimeConfigTools {
   public companion object {
     /**
      * Helper for Configuration for the workflow runtime in an application.
      * This allows one to specify a project property from the gradle build to choose a runtime.
-     * e.g. add "-Pworkflow.runtime=conflate" in your gradle build to build the conflate runtime
+     * e.g. add `-Pworkflow.runtime=conflate` in your gradle build to build the conflate runtime
      * into the application.
      *
-     * The [WorkflowTestRuntime] already calls this utility, but if starting your own runtime, then
-     * call this function and pass the result to the call to [renderWorkflowIn] as the
-     * [RuntimeConfig].
+     * The [WorkflowTestRuntime][com.squareup.workflow1.testing.WorkflowTestRuntime]
+     * and [RenderTester][com.squareup.workflow1.testing.RenderTester] runtimes
+     * already call this utility. To honor this property from your own runtime call this
+     * function and pass the result to the call to
+     * [renderWorkflowIn][com.squareup.workflow1.renderWorkflowIn] as the [RuntimeConfig] parameter.
      *
-     * Current options are:
-     * "conflate" : Process all queued actions before passing rendering
-     *      to the UI layer.
-     * "baseline" : Original Workflow Runtime. Note that this doesn't need to
-     *      be specified as it is the current default and is assumed by this utility.
+     * Current options (can be combined with `-` characters, e.g. `conflate-partial`):
      *
-     * Then, these can be combined (via '-') with:
-     * "stateChange" : Only re-render when the state of some WorkflowNode has been changed by an
+     * - `conflate` Process all queued actions before passing rendering to the UI layer.
+     *
+     * - `stateChange` Only re-render when the state of some WorkflowNode has been changed by an
      *   action cascade.
-     * "partial" : Which includes "stateChange" as well as partial tree rendering, which only
-     *   re-renders each Workflow node if: 1) its state changed; or 2) one of its descendant's state
-     *   changed.
      *
-     * E.g., "baseline-stateChange" to turn on the stateChange option with the baseline runtime.
+     * - `partial` Partial tree rendering, which only re-renders each Workflow node if: 1) its
+     *    state changed; or 2) one of its descendant's state changed. (This option requires
+     *    `stateChange`, and enables it as well.)
      *
+     * - `stable` Enables stable event handlers (changes the default value of the `remember`
+     *    parameter of `RenderContext.eventHandler` functions from `false` to `true`)
      */
     @OptIn(WorkflowExperimentalRuntime::class)
     public fun getTestRuntimeConfig(): RuntimeConfig {
-      return when
-        (val runtimeConfig = System.getProperty("workflow.runtime", "baseline")) {
-        "conflate" -> setOf(CONFLATE_STALE_RENDERINGS)
-        "conflate-stateChange" -> setOf(CONFLATE_STALE_RENDERINGS, RENDER_ONLY_WHEN_STATE_CHANGES)
-        "baseline-stateChange" -> setOf(RENDER_ONLY_WHEN_STATE_CHANGES)
-        "conflate-partial" -> setOf(
-          CONFLATE_STALE_RENDERINGS,
-          RENDER_ONLY_WHEN_STATE_CHANGES,
-          PARTIAL_TREE_RENDERING
-        )
-        "baseline-partial" -> setOf(RENDER_ONLY_WHEN_STATE_CHANGES, PARTIAL_TREE_RENDERING)
-        "", "baseline" -> RuntimeConfigOptions.RENDER_PER_ACTION
-        else ->
-          throw IllegalArgumentException("Unrecognized config \"$runtimeConfig\"")
+      val selection = System.getProperty("workflow.runtime", "baseline").split("-")
+        // We used to have a no-op `baseline` option, let's not choke on it.
+        .filterNot { it == "baseline" }
+        .toSet()
+
+      val config = mutableSetOf<RuntimeConfigOptions>()
+      selection.forEach {
+        when (it) {
+          "conflate" -> config.add(CONFLATE_STALE_RENDERINGS)
+          "stateChange" -> config.add(RENDER_ONLY_WHEN_STATE_CHANGES)
+          "partial" -> config.addAll(setOf(RENDER_ONLY_WHEN_STATE_CHANGES, PARTIAL_TREE_RENDERING))
+          "stable" -> config.add(STABLE_EVENT_HANDLERS)
+          else -> throw IllegalArgumentException("Unrecognized runtime config option \"$it\"")
+        }
       }
+
+      return config
     }
   }
 }