search

michaelbull / kotlin-result

A multiplatform Result monad for modelling success or failure operations.
ISC License
1.05k stars 63 forks source link

Move coroutine tests to use kotlinx-coroutines-test instead of runBlockingTest #50

Closed Munzey closed 3 years ago

Munzey commented 3 years ago

See Roman's latest comment: https://youtrack.jetbrains.com/issue/KT-22228#focus=Comments-27-4791713.0-0

michaelbull commented 3 years ago

Also may be relevant: https://blog.jetbrains.com/kotlin/2021/04/kotlin-1-5-0-rc-released/#test-library-changes

michaelbull commented 3 years ago

For some reason, the kotlinx-coroutines-test library depends on kotlinx-coroutines-core-jvm, so it isn't cross-platform and thus can't replace the runBlockingTest in this library.

Looks like I'm not the only one to find this strange...

michaelbull commented 3 years ago

Going to close this issue until https://github.com/Kotlin/kotlinx.coroutines/issues/1996 is resolved.