Remove the public from test source sets

Command was:
git grep -l '^ *public ' | grep /test/ | xargs sed -i 's/^\( *\)public /\1/g'

Author: dkhalanskyjb

kotlinx-coroutines-core/common/test/flow/NamedDispatchers.kt

@@ -5,15 +5,15 @@ import kotlin.coroutines.*
 /**
  * Test dispatchers that emulate multiplatform context tracking.
  */
-public object NamedDispatchers {
+object NamedDispatchers {
 
     private val stack = ArrayStack()
 
-    public fun name(): String = stack.peek() ?: error("No names on stack")
+    fun name(): String = stack.peek() ?: error("No names on stack")
 
-    public fun nameOr(defaultValue: String): String = stack.peek() ?: defaultValue
+    fun nameOr(defaultValue: String): String = stack.peek() ?: defaultValue
 
-    public operator fun invoke(name: String) = named(name)
+    operator fun invoke(name: String) = named(name)
 
     private fun named(name: String): CoroutineDispatcher = object : CoroutineDispatcher() {
         override fun dispatch(context: CoroutineContext, block: Runnable) {
@@ -32,14 +32,14 @@ private class ArrayStack {
     private var elements = arrayOfNulls<String>(16)
     private var head = 0
 
-    public fun push(value: String) {
+    fun push(value: String) {
         if (elements.size == head - 1) ensureCapacity()
         elements[head++] = value
     }
 
-    public fun peek(): String? = elements.getOrNull(head - 1)
+    fun peek(): String? = elements.getOrNull(head - 1)
 
-    public fun pop(): String? {
+    fun pop(): String? {
         if (head == 0) return null
         return elements[--head]
     }

kotlinx-coroutines-core/common/test/flow/VirtualTime.kt

@@ -89,7 +89,7 @@ internal class VirtualTimeDispatcher(enclosingScope: CoroutineScope) : Coroutine
  *    return from [withVirtualTime] before the test is executed completely.
  *    To decrease the probability of such error, additional `finish` constraint is added.
  */
-public fun TestBase.withVirtualTime(block: suspend CoroutineScope.() -> Unit) = runTest {
+fun TestBase.withVirtualTime(block: suspend CoroutineScope.() -> Unit) = runTest {
     withContext(Dispatchers.Unconfined) {
         // Create a platform-independent event loop
         val dispatcher = VirtualTimeDispatcher(this)

kotlinx-coroutines-core/common/test/flow/operators/FlowOnTest.kt

@@ -55,7 +55,7 @@ class FlowOnTest : TestBase() {
     }
 
     @Test
-    public fun testFlowOnThrowingSource() = runTest {
+    fun testFlowOnThrowingSource() = runTest {
         val flow = flow {
             expect(1)
             emit(NamedDispatchers.name())
@@ -73,7 +73,7 @@ class FlowOnTest : TestBase() {
     }
 
     @Test
-    public fun testFlowOnThrowingOperator() = runTest {
+    fun testFlowOnThrowingOperator() = runTest {
         val flow = flow {
             expect(1)
             emit(NamedDispatchers.name())
@@ -90,7 +90,7 @@ class FlowOnTest : TestBase() {
     }
 
     @Test
-    public fun testFlowOnDownstreamOperator() = runTest() {
+    fun testFlowOnDownstreamOperator() = runTest() {
         val flow = flow {
             expect(2)
             emit(NamedDispatchers.name())
@@ -112,7 +112,7 @@ class FlowOnTest : TestBase() {
     }
 
     @Test
-    public fun testFlowOnThrowingConsumer() = runTest {
+    fun testFlowOnThrowingConsumer() = runTest {
         val flow = flow {
             expect(2)
             emit(NamedDispatchers.name())
@@ -323,7 +323,7 @@ class FlowOnTest : TestBase() {
     }
 
     private inner class Source(private val value: Int) {
-        public var contextName: String = "unknown"
+        var contextName: String = "unknown"
 
         fun produce(): Int {
             contextName = NamedDispatchers.nameOr("main")
@@ -332,7 +332,7 @@ class FlowOnTest : TestBase() {
     }
 
     private inner class Consumer(private val expected: Int) {
-        public var contextName: String = "unknown"
+        var contextName: String = "unknown"
 
         fun consume(value: Int) {
             contextName = NamedDispatchers.nameOr("main")

kotlinx-coroutines-core/common/test/flow/operators/SampleTest.kt

@@ -9,7 +9,7 @@ import kotlin.time.Duration.Companion.milliseconds
 
 class SampleTest : TestBase() {
     @Test
-    public fun testBasic() = withVirtualTime {
+    fun testBasic() = withVirtualTime {
         expect(1)
         val flow = flow {
             expect(3)

kotlinx-coroutines-core/common/test/flow/sharing/SharedFlowTest.kt

@@ -558,7 +558,7 @@ class SharedFlowTest : TestBase() {
     }
 
     @Test
-    public fun testOnSubscription() = runTest {
+    fun testOnSubscription() = runTest {
         expect(1)
         val sh = MutableSharedFlow<String>()
         fun share(s: String) { launch(start = CoroutineStart.UNDISPATCHED) { sh.emit(s) } }
@@ -762,10 +762,10 @@ class SharedFlowTest : TestBase() {
     }
 
     @Test
-    public fun testReplayCancellability() = testCancellability(fromReplay = true)
+    fun testReplayCancellability() = testCancellability(fromReplay = true)
 
     @Test
-    public fun testEmitCancellability() = testCancellability(fromReplay = false)
+    fun testEmitCancellability() = testCancellability(fromReplay = false)
 
     private fun testCancellability(fromReplay: Boolean) = runTest {
         expect(1)

kotlinx-coroutines-core/common/test/flow/sharing/StateFlowTest.kt

@@ -172,7 +172,7 @@ class StateFlowTest : TestBase() {
     }
 
     @Test
-    public fun testOnSubscriptionWithException() = runTest {
+    fun testOnSubscriptionWithException() = runTest {
         expect(1)
         val state = MutableStateFlow("A")
         state

kotlinx-coroutines-core/concurrent/test/AbstractDispatcherConcurrencyTest.kt

@@ -7,7 +7,7 @@ import kotlin.test.*
 
 abstract class AbstractDispatcherConcurrencyTest : TestBase() {
 
-   public abstract val dispatcher: CoroutineDispatcher
+   abstract val dispatcher: CoroutineDispatcher
 
     @Test
     fun testLaunchAndJoin() = runTest {

kotlinx-coroutines-core/jvm/test/AwaitJvmTest.kt

@@ -5,7 +5,7 @@ import org.junit.*
 
 class AwaitJvmTest : TestBase() {
     @Test
-    public fun testSecondLeak() = runTest {
+    fun testSecondLeak() = runTest {
         // This test is to make sure that handlers installed on the second deferred do not leak
         val d1 = CompletableDeferred<Int>()
         val d2 = CompletableDeferred<Int>()

kotlinx-coroutines-core/jvm/test/AwaitStressTest.kt

@@ -9,7 +9,7 @@ class AwaitStressTest : TestBase() {
 
     private val iterations = 50_000 * stressTestMultiplier
     @get:Rule
-    public val pool =  ExecutorRule(4)
+    val pool =  ExecutorRule(4)
 
     @Test
     fun testMultipleExceptions() = runTest {

kotlinx-coroutines-core/jvm/test/CancellableContinuationJvmTest.kt

@@ -63,16 +63,16 @@ class CancellableContinuationJvmTest : TestBase() {
         private var isCancelled = false
 
         @Volatile
-        public var hasSubscriber = false
+        var hasSubscriber = false
 
-        public fun subscribe() {
+        fun subscribe() {
             hasSubscriber = true
             while (!isCancelled) {
                 Thread.sleep(10)
             }
         }
 
-        public fun cancel() {
+        fun cancel() {
             isCancelled = true
         }
     }