Java SDK: Drop the unused switch in the condition test's decider

Author: jason810496

java-sdk/sdk/src/test/kotlin/org/apache/airflow/sdk/ClientTest.kt

@@ -46,7 +46,6 @@ private class FakeTransport(
   val stored: TaskStateStoreResult? = null,
 ) : org.apache.airflow.sdk.execution.Client {
   val calls = mutableListOf<StateStoreCall>()
-  val skipped = mutableListOf<String>()
 
   override fun getConnection(id: String): ConnectionResult = connection
 
@@ -106,9 +105,7 @@ private class FakeTransport(
     calls.add(StateStoreCall("clear", tiId))
   }
 
-  override fun skipDownstreamTasks(taskIds: List<String>) {
-    skipped += taskIds
-  }
+  override fun skipDownstreamTasks(taskIds: List<String>) = Unit
 }
 
 class ClientTest {

java-sdk/sdk/src/test/kotlin/org/apache/airflow/sdk/ConditionTest.kt

@@ -27,16 +27,12 @@ import org.junit.jupiter.api.Assertions.assertTrue
 import org.junit.jupiter.api.DisplayName
 import org.junit.jupiter.api.Test
 
-/** A condition the tests register; what it decides is driven from the runtime tests. */
+/** A condition the tests register; it always holds. */
 class HasRows : ConditionTask {
   override fun decide(
     context: Context,
     client: Client,
-  ): Boolean = decision
-
-  companion object {
-    var decision: Boolean = true
-  }
+  ): Boolean = true
 }
 
 internal class ConditionTest {