Reduce the number of iterations and time spent in [nowadays] less relevant tests

Author: qwwdfsad

kotlinx-coroutines-core/concurrent/test/channels/BroadcastChannelSubStressTest.kt

@@ -15,13 +15,13 @@ import kotlin.test.*
  */
 class BroadcastChannelSubStressTest: TestBase() {
 
-    private val nSeconds = 5 * stressTestMultiplier
+    private val nSeconds = maxOf(5, stressTestMultiplier)
     private val sentTotal = atomic(0L)
     private val receivedTotal = atomic(0L)
 
     @Test
     fun testStress() = runTest {
-        TestBroadcastChannelKind.values().forEach { kind ->
+        TestBroadcastChannelKind.entries.forEach { kind ->
             println("--- BroadcastChannelSubStressTest $kind")
             val broadcast = kind.create<Long>()
             val sender =

kotlinx-coroutines-core/concurrent/test/sync/MutexStressTest.kt

@@ -11,7 +11,7 @@ import kotlin.test.*
 
 class MutexStressTest : TestBase() {
 
-    private val n = (if (isNative) 1_000 else 10_000) * stressTestMultiplier
+    private val n = 1000 * stressTestMultiplier // It mostly stresses K/N as JVM Mutex is tested by lincheck
 
     @Test
     fun testDefaultDispatcher() = runTest { testBody(Dispatchers.Default) }

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

@@ -18,15 +18,17 @@ import java.util.concurrent.atomic.*
 class BroadcastChannelMultiReceiveStressTest(
     private val kind: TestBroadcastChannelKind
 ) : TestBase() {
+
+    // Stressed by lincheck
     companion object {
         @Parameterized.Parameters(name = "{0}")
         @JvmStatic
         fun params(): Collection<Array<Any>> =
-            TestBroadcastChannelKind.values().map { arrayOf<Any>(it) }
+            TestBroadcastChannelKind.entries.map { arrayOf<Any>(it) }
     }
 
     private val nReceivers = if (isStressTest) 10 else 5
-    private val nSeconds = 3 * stressTestMultiplier
+    private val nSeconds = 3 * stressTestMultiplierSqrt
 
     private val broadcast = kind.create<Long>()
     private val pool = newFixedThreadPoolContext(nReceivers + 1, "BroadcastChannelMultiReceiveStressTest")
@@ -65,13 +67,13 @@ class BroadcastChannelMultiReceiveStressTest(
             println("Launching $name")
             receivers += launch(pool + CoroutineName(name)) {
                 val channel = broadcast.openSubscription()
-                    when (receiverIndex % 5) {
-                        0 -> doReceive(channel, receiverIndex)
-                        1 -> doReceiveCatching(channel, receiverIndex)
-                        2 -> doIterator(channel, receiverIndex)
-                        3 -> doReceiveSelect(channel, receiverIndex)
-                        4 -> doReceiveCatchingSelect(channel, receiverIndex)
-                    }
+                when (receiverIndex % 5) {
+                    0 -> doReceive(channel, receiverIndex)
+                    1 -> doReceiveCatching(channel, receiverIndex)
+                    2 -> doIterator(channel, receiverIndex)
+                    3 -> doReceiveSelect(channel, receiverIndex)
+                    4 -> doReceiveCatchingSelect(channel, receiverIndex)
+                }
                 channel.cancel()
             }
             printProgress()
@@ -96,7 +98,7 @@ class BroadcastChannelMultiReceiveStressTest(
         } catch (e: Exception) {
             println("Failed: $e")
             pool.dumpThreads("Threads in pool")
-            receivers.indices.forEach { index  ->
+            receivers.indices.forEach { index ->
                 println("lastReceived[$index] = ${lastReceived[index].get()}")
             }
             throw e
@@ -119,8 +121,9 @@ class BroadcastChannelMultiReceiveStressTest(
             try {
                 val stop = doReceived(receiverIndex, channel.receive())
                 if (stop) break
+            } catch (ex: ClosedReceiveChannelException) {
+                break
             }
-            catch (ex: ClosedReceiveChannelException) { break }
         }
     }
 
@@ -144,7 +147,9 @@ class BroadcastChannelMultiReceiveStressTest(
                 val event = select<Long> { channel.onReceive { it } }
                 val stop = doReceived(receiverIndex, event)
                 if (stop) break
-            } catch (ex: ClosedReceiveChannelException) { break }
+            } catch (ex: ClosedReceiveChannelException) {
+                break
+            }
         }
     }
 
@@ -155,4 +160,10 @@ class BroadcastChannelMultiReceiveStressTest(
             if (stop) break
         }
     }
+
+    @Suppress("UNUSED_PARAMETER")
+    private fun println(debugMessage: String) {
+        // Uncomment for local debugging
+        // kotlin.io.println(debugMessage)
+    }
 }

kotlinx-coroutines-core/jvm/test/selects/SelectPhilosophersStressTest.kt

@@ -10,7 +10,7 @@ import org.junit.Test
 import kotlin.test.*
 
 class SelectPhilosophersStressTest : TestBase() {
-    private val TEST_DURATION = 3000L * stressTestMultiplier
+    private val TEST_DURATION = 3000L * stressTestMultiplierSqrt
 
     val n = 10 // number of philosophers
     private val forks = Array(n) { Mutex() }
@@ -62,4 +62,4 @@ class SelectPhilosophersStressTest : TestBase() {
             assertTrue(eats > 0, "$id shall not starve")
         }
     }
-}
+}