Get rid of unnecessary println in tests

Author: qwwdfsad

benchmarks/src/jmh/kotlin/benchmarks/ForkJoinBenchmark.kt

@@ -154,13 +154,4 @@ private fun compute(coefficients: LongArray, start: Int, end: Int): Double {
     }
 
     return result
-}
-
-
-fun main(args: Array<String>) {
-    // Hand-rollled test
-    val arr = LongArray(BATCH_SIZE * 8) { ThreadLocalRandom.current().nextLong(0, 100) }
-
-    println(ForkJoinPool.commonPool().submit(ForkJoinBenchmark.Task(arr, 0, arr.size)).join())
-    println(ForkJoinBenchmark.RecursiveAction(arr, 0, arr.size).invoke())
-}
+}

binary-compatibility-validator/test/PublicApiTest.kt

@@ -99,14 +99,12 @@ class PublicApiTest {
         val jarFile = getJarPath(base, jarPattern)
         val kotlinJvmMappingsFiles = listOf(base.resolve("../visibilities.json"))
 
-        println("Reading kotlin visibilities from $kotlinJvmMappingsFiles")
         val publicPackagePrefixes = publicPackages.map { it.replace('.', '/') + '/' }
         val visibilities =
                 kotlinJvmMappingsFiles
                         .map { readKotlinVisibilities(it).filterKeys { name -> publicPackagePrefixes.none { name.startsWith(it) } } }
                         .reduce { m1, m2 -> m1 + m2 }
 
-        println("Reading binary API from $jarFile")
         val api = getBinaryAPI(JarFile(jarFile), visibilities).filterOutNonPublic(nonPublicPackages)
 
         val target = File("reference-public-api")

common/kotlinx-coroutines-core-common/test/WithTimeoutOrNullTest.kt

@@ -111,9 +111,7 @@ class WithTimeoutOrNullTest : TestBase() {
             }
         }
         assertEquals(null, result)
-        // under load counter may be equal to 1, so the check is lenient here
-        println("Executed: $counter times")
-        check(counter in 1..2)
+        check(counter in 1..2) {"Executed: $counter times"}
     }
 
     @Test

core/kotlinx-coroutines-core/test/scheduling/CoroutineSchedulerStressTest.kt

@@ -94,9 +94,8 @@ class CoroutineSchedulerStressTest : TestBase() {
         })
 
         finishLatch.await()
-        val n = observedThreads.size
-        println("Observed $n threads with $AVAILABLE_PROCESSORS available processors")
-        assertTrue(AVAILABLE_PROCESSORS in (n - 1)..n)
+        val observed = observedThreads.size
+        assertTrue(AVAILABLE_PROCESSORS in (observed - 1)..observed, "Observed $observed threads with $AVAILABLE_PROCESSORS available processors")
         validateResults()
     }
 

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

@@ -38,7 +38,6 @@ class SelectPhilosophersStressTest : TestBase() {
 
     @Test
     fun testPhilosophers() = runBlocking<Unit> {
-        println("--- SelectPhilosophersStressTest")
         val timeLimit = System.currentTimeMillis() + TEST_DURATION
         val philosophers = List<Deferred<Int>>(n) { id ->
             async {

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

@@ -24,7 +24,6 @@ class MutexStressTest : TestBase() {
             }
         }
         jobs.forEach { it.join() }
-        println("Shared value = $shared")
         assertEquals(n * k, shared)
     }
 }

native/kotlinx-coroutines-core-native/src/internal/CopyOnWriteList.kt

@@ -11,7 +11,6 @@ internal class CopyOnWriteList<E>(private var array: Array<Any?> = arrayOfNulls(
     override val size: Int get() = _size
 
     override fun add(element: E): Boolean {
-        println("Add")
         val newSize = if (_size == array.size) array.size * 2 else  array.size
         val update = array.copyOf(newSize)
         update[_size++] = element

reactive/kotlinx-coroutines-reactor/test/SchedulerTest.kt

@@ -24,12 +24,10 @@ class SchedulerTest : TestBase() {
         val mainThread = Thread.currentThread()
         withContext(Schedulers.single().asCoroutineDispatcher()) {
             val t1 = Thread.currentThread()
-            println(t1)
             assertThat(t1, IsNot(IsEqual(mainThread)))
             expect(2)
             delay(100)
             val t2 = Thread.currentThread()
-            println(t2)
             assertThat(t2, IsNot(IsEqual(mainThread)))
             expect(3)
         }

reactive/kotlinx-coroutines-rx1/test/SchedulerTest.kt

@@ -24,12 +24,10 @@ class SchedulerTest : TestBase() {
         val mainThread = Thread.currentThread()
         withContext(Schedulers.io().asCoroutineDispatcher()) {
             val t1 = Thread.currentThread()
-            println(t1)
             assertThat(t1, IsNot(IsEqual(mainThread)))
             expect(2)
             delay(100)
             val t2 = Thread.currentThread()
-            println(t2)
             assertThat(t2, IsNot(IsEqual(mainThread)))
             expect(3)
         }

reactive/kotlinx-coroutines-rx2/test/SchedulerTest.kt

@@ -24,12 +24,10 @@ class SchedulerTest : TestBase() {
         val mainThread = Thread.currentThread()
         withContext(Schedulers.io().asCoroutineDispatcher()) {
             val t1 = Thread.currentThread()
-            println(t1)
             assertThat(t1, IsNot(IsEqual(mainThread)))
             expect(2)
             delay(100)
             val t2 = Thread.currentThread()
-            println(t2)
             assertThat(t2, IsNot(IsEqual(mainThread)))
             expect(3)
         }