runBlockingTest raises job has not completed yet with Futures #1395

powturns · 2019-07-29T21:29:45Z

The following code will raise a "This job has not completed yet" exception

    @Test
    fun testFuture() = runBlockingTest {
        try {
            CompletableFuture.supplyAsync {
                Thread.sleep(1000)
                throw RuntimeException("Async Exception")
            }.await()
            fail("Expected Exception not thrown")
        } catch (e: RuntimeException) {
            //noop
        }
    }

I believe it is because advanceUntilIdle() doesn't wait for the future to complete because the async behaviour is kicked off from outside of the coroutines.

This may be related to some of the other "This job has not completed yet" issues, but I don't have enough experience to evaluate.

The text was updated successfully, but these errors were encountered:

qwwdfsad · 2019-07-29T22:36:17Z

This is being fixed in #1206

Implement a multiplatform runTest as an initial implementation of #1996. Fixes #1204 Fixes #1222 Fixes #1395 Fixes #1881 Fixes #1910 Fixes #1772

This commit introduces the new version of the test module. Please see README.md and MIGRATION.md for a thorough discussion of the changes. Fixes Kotlin#1203 Fixes Kotlin#1609 Fixes Kotlin#2379 Fixes Kotlin#1749 Fixes Kotlin#1204 Fixes Kotlin#1390 Fixes Kotlin#1222 Fixes Kotlin#1395 Fixes Kotlin#1881 Fixes Kotlin#1910 Fixes Kotlin#1772 Fixes Kotlin#1626 Fixes Kotlin#1742 Fixes Kotlin#2082 Fixes Kotlin#2102 Fixes Kotlin#2405 Fixes Kotlin#2462 Co-authored-by: Vsevolod Tolstopyatov <qwwdfsad@gmail.com>

qwwdfsad added the test label Jul 29, 2019

qwwdfsad self-assigned this Feb 18, 2020

dkhalanskyjb mentioned this issue Oct 12, 2021

Implement 'runTest' that waits for asynchronous completion #2978

Merged

dkhalanskyjb assigned dkhalanskyjb and unassigned qwwdfsad Oct 12, 2021

dkhalanskyjb added a commit that referenced this issue Nov 1, 2021

Implement 'runTest' that waits for asynchronous completion (#2978)

9fb2182

Implement a multiplatform runTest as an initial implementation of #1996. Fixes #1204 Fixes #1222 Fixes #1395 Fixes #1881 Fixes #1910 Fixes #1772

dkhalanskyjb mentioned this issue Nov 1, 2021

Update kotlinx-coroutines-test #2973

Merged

dkhalanskyjb added a commit that referenced this issue Nov 17, 2021

Implement 'runTest' that waits for asynchronous completion (#2978)

104c507

Implement a multiplatform runTest as an initial implementation of #1996. Fixes #1204 Fixes #1222 Fixes #1395 Fixes #1881 Fixes #1910 Fixes #1772

dkhalanskyjb added a commit that referenced this issue Nov 17, 2021

Implement 'runTest' that waits for asynchronous completion (#2978)

888d7a0

Implement a multiplatform runTest as an initial implementation of #1996. Fixes #1204 Fixes #1222 Fixes #1395 Fixes #1881 Fixes #1910 Fixes #1772

mpierucci mentioned this issue Nov 17, 2021

Integration Test Fix mpierucci/Android-Redux#11

Open

dkhalanskyjb added a commit that referenced this issue Nov 19, 2021

Implement 'runTest' that waits for asynchronous completion (#2978)

d9bc7ac

Implement a multiplatform runTest as an initial implementation of #1996. Fixes #1204 Fixes #1222 Fixes #1395 Fixes #1881 Fixes #1910 Fixes #1772

qwwdfsad closed this as completed in 2e25bae Nov 22, 2021

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

runBlockingTest raises job has not completed yet with Futures #1395

runBlockingTest raises job has not completed yet with Futures #1395

powturns commented Jul 29, 2019 •

edited

Loading

qwwdfsad commented Jul 29, 2019

runBlockingTest raises job has not completed yet with Futures #1395

runBlockingTest raises job has not completed yet with Futures #1395

Comments

powturns commented Jul 29, 2019 • edited Loading

qwwdfsad commented Jul 29, 2019

powturns commented Jul 29, 2019 •

edited

Loading