I'm seeing a behavior change between kotlinx-corou...

# coroutines

[Sam]

I'm seeing a behavior change between kotlinx-coroutines-core:1.1.0 and kotlinx-coroutines-core:1.2.0 when testing a cancel of launched coroutine within runBlocking (runBlocking is associated with Main dispatcher and launched coroutine with Default dispatcher)

1.1.0 -> runBlocking waits for the CancellationException handling
1.2.0 -> runBlocking finishes before handling of CancellationException

Is this on purpose?

@RunWith(JUnit4::class)
class CoroutineTest {

    @Test
    fun `test coroutine cancel`() {

        var backgroundWorkCanceled = false

        runBlocking {
            val scope = CoroutineScope( Dispatchers.Default + Job() )
            scope.launch {
                try {
                    backgroundWorkCanceled = false
                    delay(3000)
                } catch( e : CancellationException ) {
                    backgroundWorkCanceled = true
                }
            }
            delay( 2000 )
            scope.cancel()
        }

        assert( backgroundWorkCanceled )
    }
}

[Sam]

It works as expected within a fun main() with both 1.1.0 and 1.2.0. Just that unit test started failing with 1.2.0

[louiscad]

runBlocking doesn't use Dispatchers.Main by default, but creates its own event loop as a dispatcher from the thread it is blocking.

[louiscad]

Also, your test breaks structured concurrency as you are creating a custom job instead of just using the job brought by runBlocking lambda receiver. Consequently, you face a race condition as `runBlocking returns immediately since your prevent the coroutine you launch from being its child that it would wait for.

[louiscad]

You should remove your custom scope and call launch(Dispatchers.Default) instead, if you really need to change the dispatcher.

[Sam]

Okay, so technically, it should have failed with 1.1.0. So was that just a timing coincidence?

[louiscad]

Or just a race condition that was less likely or impossible to happen in 1.1.0 because of implementation details (that you should not rely on, as you can see).

[Sam]

Okay got it

[Sam]

Thanks