Point the coroutines-test comments at the file that still uses it
Both the dependency declaration and its catalog entry named EscapedCoroutineErrors.kt as the sole reason kotlinx-coroutines-test is on the test classpath. That file is gone, and nothing in the gate -- not ktlint, not detekt, not lint -- fails on prose naming a deleted file, so this would have survived as a reference a reader could only resolve through git history. The dependency itself stays, and for a reason worth restating where it is declared: `runTest` is what registers the collector callback, so the one test that deliberately lets an error escape is the scope that receives it. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -333,9 +333,9 @@ dependencies {
|
|||||||
// the tests stay green.
|
// the tests stay green.
|
||||||
testImplementation(platform(libs.compose.bom))
|
testImplementation(platform(libs.compose.bom))
|
||||||
testImplementation(libs.compose.ui.test.junit4)
|
testImplementation(libs.compose.ui.test.junit4)
|
||||||
// For `runTest` alone, in EscapedCoroutineErrors.kt. It arrives transitively with the
|
// For `runTest` alone, in ConversionViewModelProbeFailureTest. It arrives transitively
|
||||||
// rule above anyway; declared because a test file imports it directly, and an import of
|
// with the rule above anyway; declared because a test file imports it directly, and an
|
||||||
// something nobody asked for breaks the day the library that pulled it in stops.
|
// import of something nobody asked for breaks the day the library that pulled it in stops.
|
||||||
testImplementation(libs.kotlinx.coroutines.test)
|
testImplementation(libs.kotlinx.coroutines.test)
|
||||||
|
|
||||||
androidTestImplementation(platform(libs.compose.bom))
|
androidTestImplementation(platform(libs.compose.bom))
|
||||||
|
|||||||
@@ -162,8 +162,9 @@ androidx-uiautomator = { group = "androidx.test.uiautomator", name = "uiautomato
|
|||||||
# a TDD loop anyone here can execute.
|
# a TDD loop anyone here can execute.
|
||||||
robolectric = { group = "org.robolectric", name = "robolectric", version.ref = "robolectric" }
|
robolectric = { group = "org.robolectric", name = "robolectric", version.ref = "robolectric" }
|
||||||
|
|
||||||
# Only for its `runTest`, and only to drain the collector kotlinx-coroutines-test installs
|
# Only for its `runTest`, and only so the one test that deliberately lets a coroutine error
|
||||||
# process-wide. See EscapedCoroutineErrors.kt in the JVM test source set.
|
# escape owns the collector callback while it does -- otherwise the error is kept process-wide
|
||||||
|
# and rethrown at whichever `runTest` starts next. See ConversionViewModelProbeFailureTest.
|
||||||
kotlinx-coroutines-test = { group = "org.jetbrains.kotlinx", name = "kotlinx-coroutines-test", version.ref = "coroutinesTest" }
|
kotlinx-coroutines-test = { group = "org.jetbrains.kotlinx", name = "kotlinx-coroutines-test", version.ref = "coroutinesTest" }
|
||||||
|
|
||||||
[plugins]
|
[plugins]
|
||||||
|
|||||||
Reference in New Issue
Block a user