Revert select implementation via selectOld and turn on all the tests

Author: Nikita Koval

kotlinx-coroutines-core/api/kotlinx-coroutines-core.api

@@ -1271,10 +1271,6 @@ public final class kotlinx/coroutines/selects/SelectKt {
 	public static final fun select (Lkotlin/jvm/functions/Function1;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
 }
 
-public final class kotlinx/coroutines/selects/SelectOldKt {
-	public static final fun selectOld (Lkotlin/jvm/functions/Function1;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
-}
-
 public final class kotlinx/coroutines/selects/SelectUnbiasedKt {
 	public static final fun selectUnbiased (Lkotlin/jvm/functions/Function1;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
 }

kotlinx-coroutines-core/common/src/selects/Select.kt

@@ -56,7 +56,6 @@ public suspend inline fun <R> select(crossinline builder: SelectBuilder<R>.() ->
     contract {
         callsInPlace(builder, InvocationKind.EXACTLY_ONCE)
     }
-    return selectOld(builder)
     return SelectImplementation<R>(coroutineContext).run {
         builder(this)
         // TAIL-CALL OPTIMIZATION: the only

kotlinx-coroutines-core/common/src/selects/SelectOld.kt

@@ -85,7 +85,7 @@ internal class UnbiasedSelectBuilderImpl<R>(
 }
 
 // This is the old version of `select`. It should work to guarantee binary compatibility.
-public suspend inline fun <R> selectOld(crossinline builder: SelectBuilder<R>.() -> Unit): R {
+internal suspend inline fun <R> selectOld(crossinline builder: SelectBuilder<R>.() -> Unit): R {
     return suspendCoroutineUninterceptedOrReturn { uCont ->
         val scope = SelectBuilderImpl(uCont)
         try {

kotlinx-coroutines-core/jvm/test/channels/ChannelUndeliveredElementStressTest.kt

@@ -18,7 +18,6 @@ import kotlin.test.*
  * Tests resource transfer via channel send & receive operations, including their select versions,
  * using `onUndeliveredElement` to detect lost resources and close them properly.
  */
-@Ignore
 @RunWith(Parameterized::class)
 class ChannelUndeliveredElementStressTest(private val kind: TestChannelKind) : TestBase() {
     companion object {

kotlinx-coroutines-core/jvm/test/lincheck/ChannelsLincheckTest.kt

@@ -70,7 +70,7 @@ abstract class ChannelLincheckTestBase(
                 else false
             }
 
-//    @Operation(promptCancellation = true)
+    @Operation(promptCancellation = true)
     suspend fun sendViaSelect(@Param(name = "value") value: Int): Any = try {
         select<Unit> { c.onSend(value) {} }
     } catch (e: NumberedCancellationException) {
@@ -90,7 +90,7 @@ abstract class ChannelLincheckTestBase(
             .onSuccess { return it }
             .onFailure { return if (it is NumberedCancellationException) it.testResult else null }
 
-//    @Operation(promptCancellation = true)
+    @Operation(promptCancellation = true)
     suspend fun receiveViaSelect(): Any = try {
         select<Int> { c.onReceive { it } }
     } catch (e: NumberedCancellationException) {

kotlinx-coroutines-core/jvm/test/lincheck/MutexLincheckTest.kt

@@ -22,7 +22,7 @@ class MutexLincheckTest : AbstractLincheckTest() {
     @Operation(promptCancellation = true)
     suspend fun lock(@Param(name = "owner") owner: Int) = mutex.lock(owner.asOwnerOrNull)
 
-//    @Operation(promptCancellation = true)
+    @Operation(promptCancellation = true)
     suspend fun onLock(@Param(name = "owner") owner: Int) = select<Unit> { mutex.onLock(owner.asOwnerOrNull) {} }
 
     @Operation(handleExceptionsAsResult = [IllegalStateException::class])

kotlinx-coroutines-debug/test/SanitizedProbesTest.kt

@@ -9,7 +9,6 @@ import kotlinx.coroutines.*
 import kotlinx.coroutines.debug.*
 import kotlinx.coroutines.selects.*
 import org.junit.*
-import org.junit.Ignore
 import org.junit.Test
 import java.util.concurrent.*
 import kotlin.test.*
@@ -75,7 +74,6 @@ class SanitizedProbesTest : DebugTestBase() {
         deferred.cancelAndJoin()
     }
 
-    @Ignore
     @Test
     fun testSelectBuilder() = runTest {
         val selector = launchSelector()