Review fixes

Signed-off-by: Nikita Koval <[email protected]>

Author: ndkoval

kotlinx-coroutines-core/common/test/channels/ChannelsTest.kt

@@ -20,7 +20,7 @@ class ChannelsTest: TestBase() {
     }
 
     @Test
-    fun testCloseWithMultipleWaiters() = runTest {
+    fun testCloseWithMultipleSuspendedReceivers() = runTest {
         // Once the channel is closed, the waiting
         // requests should be cancelled in the order
         // they were suspended in the channel.
@@ -53,6 +53,40 @@ class ChannelsTest: TestBase() {
         finish(7)
     }
 
+    @Test
+    fun testCloseWithMultipleSuspendedSenders() = runTest {
+        // Once the channel is closed, the waiting
+        // requests should be cancelled in the order
+        // they were suspended in the channel.
+        val channel = Channel<Int>()
+        launch {
+            try {
+                expect(2)
+                channel.send(42)
+                expectUnreached()
+            } catch (e: CancellationException) {
+                expect(5)
+            }
+        }
+
+        launch {
+            try {
+                expect(3)
+                channel.send(42)
+                expectUnreached()
+            } catch (e: CancellationException) {
+                expect(6)
+            }
+        }
+
+        expect(1)
+        yield()
+        expect(4)
+        channel.cancel()
+        yield()
+        finish(7)
+    }
+
     @Test
     fun testEmptyList() = runTest {
         assertTrue(emptyList<Nothing>().asReceiveChannel().toList().isEmpty())