Skip to content

Commit 6c9d358

Browse files
authored
Remove some broken tests (#3513)
These tests haven't been reliable for a very long time for some reason. We could ensure that they pass by a manual check, but it feels like nobody actually relies on this behavior, and it's not clear who would even want to.
1 parent 06b3b3b commit 6c9d358

File tree

2 files changed

+0
-27
lines changed

2 files changed

+0
-27
lines changed

kotlinx-coroutines-test/common/test/RunTestTest.kt

-14
Original file line numberDiff line numberDiff line change
@@ -69,20 +69,6 @@ class RunTestTest {
6969
deferred.await()
7070
}
7171

72-
/** Tests that a dispatch timeout of `0` will fail the test if there are some dispatches outside the scheduler. */
73-
@Test
74-
fun testRunTestWithZeroTimeoutWithUncontrolledDispatches() = testResultMap({ fn ->
75-
assertFailsWith<UncompletedCoroutinesError> { fn() }
76-
}) {
77-
runTest(dispatchTimeoutMs = 0) {
78-
withContext(Dispatchers.Default) {
79-
delay(10)
80-
3
81-
}
82-
fail("shouldn't be reached")
83-
}
84-
}
85-
8672
/** Tests that too low of a dispatch timeout causes crashes. */
8773
@Test
8874
fun testRunTestWithSmallTimeout() = testResultMap({ fn ->

kotlinx-coroutines-test/jvm/test/migration/RunTestLegacyScopeTest.kt

-13
Original file line numberDiff line numberDiff line change
@@ -66,19 +66,6 @@ class RunTestLegacyScopeTest {
6666
deferred.await()
6767
}
6868

69-
@Test
70-
fun testRunTestWithZeroTimeoutWithUncontrolledDispatches() = testResultMap({ fn ->
71-
assertFailsWith<UncompletedCoroutinesError> { fn() }
72-
}) {
73-
runTestWithLegacyScope(dispatchTimeoutMs = 0) {
74-
withContext(Dispatchers.Default) {
75-
delay(10)
76-
3
77-
}
78-
fail("shouldn't be reached")
79-
}
80-
}
81-
8269
@Test
8370
fun testRunTestWithSmallTimeout() = testResultMap({ fn ->
8471
assertFailsWith<UncompletedCoroutinesError> { fn() }

0 commit comments

Comments
 (0)