Merge pull request #884 from square/zachklipp/worker-finished-type

Paramaterize the return type of Worker.finished() to make it more convenient to use in tests.

Author: zach-klippenstein

kotlin/workflow-core/src/main/java/com/squareup/workflow/Worker.kt

@@ -213,7 +213,7 @@ interface Worker<out OutputT> {
     /**
      * Returns a [Worker] that finishes immediately without emitting anything.
      */
-    fun finished(): Worker<Nothing> = FinishedWorker
+    fun <T> finished(): Worker<T> = FinishedWorker
 
     /**
      * Creates a [Worker] from a function that returns a single value.

kotlin/workflow-testing/src/test/java/com/squareup/workflow/WorkerTest.kt

@@ -234,7 +234,7 @@ class WorkerTest {
   }
 
   @Test fun `finished worker is equivalent to self`() {
-    assertTrue(Worker.finished().doesSameWorkAs(Worker.finished()))
+    assertTrue(Worker.finished<Nothing>().doesSameWorkAs(Worker.finished<Nothing>()))
   }
 
   @Test fun `transformed workers are equivalent with equivalent source`() {

kotlin/workflow-testing/src/test/java/com/squareup/workflow/testing/RealRenderTesterTest.kt

@@ -189,7 +189,7 @@ class RealRenderTesterTest {
         initialState = Unit,
         render = {
           // Need to satisfy the expectation.
-          runningWorker(Worker.finished() as Worker<Unit>) { noAction() }
+          runningWorker(Worker.finished<Unit>()) { noAction() }
           return@stateful actionSink.contraMap { it }
         }
     )
@@ -571,7 +571,7 @@ class RealRenderTesterTest {
   }
 
   @Test fun `verifyAction verifies worker output`() {
-    val worker: Worker<String> = Worker.finished()
+    val worker = Worker.finished<String>()
     val workflow = Workflow.stateless<Unit, Nothing, Unit> {
       runningWorker(worker) { TestAction(it) }
     }