mirror/mihon
1
0
Fork 0
mirror of https://github.com/mihonapp/mihon.git synced 2025-11-15 21:47:28 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix flaky migration tests (#2663)

Browse Source
This commit is contained in:
NGB-Was-Taken
2025-11-07 15:06:17 +05:45
committed by GitHub
parent 9f9155121c
commit 506d51a007
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
app/src/test/java/mihon/core/migration/MigratorTest.kt
View File

@@ -1,9 +1,11 @@
package mihon.core.migration package mihon.core.migration
import io.kotest.assertions.nondeterministic.eventually
import io.mockk.slot import io.mockk.slot
import io.mockk.spyk import io.mockk.spyk
import io.mockk.verify import io.mockk.verify
import kotlinx.coroutines.CoroutineScope import kotlinx.coroutines.CoroutineScope
import kotlinx.coroutines.DelicateCoroutinesApi
import kotlinx.coroutines.Dispatchers import kotlinx.coroutines.Dispatchers
import kotlinx.coroutines.Job import kotlinx.coroutines.Job
import kotlinx.coroutines.newSingleThreadContext import kotlinx.coroutines.newSingleThreadContext
@@ -17,6 +19,7 @@ import org.junit.jupiter.api.Assertions.assertInstanceOf
import org.junit.jupiter.api.BeforeAll import org.junit.jupiter.api.BeforeAll
import org.junit.jupiter.api.BeforeEach import org.junit.jupiter.api.BeforeEach
import org.junit.jupiter.api.Test import org.junit.jupiter.api.Test
import kotlin.time.Duration.Companion.seconds
class MigratorTest { class MigratorTest {
@@ -26,7 +29,7 @@ class MigratorTest {
lateinit var migrationStrategyFactory: MigrationStrategyFactory lateinit var migrationStrategyFactory: MigrationStrategyFactory
@BeforeEach @BeforeEach
fun initilize() { fun initialize() {
migrationContext = MigrationContext(false) migrationContext = MigrationContext(false)
migrationJobFactory = spyk(MigrationJobFactory(migrationContext, CoroutineScope(Dispatchers.Main + Job()))) migrationJobFactory = spyk(MigrationJobFactory(migrationContext, CoroutineScope(Dispatchers.Main + Job())))
migrationCompletedListener = spyk<MigrationCompletedListener>(block = {}) migrationCompletedListener = spyk<MigrationCompletedListener>(block = {})
@@ -45,7 +48,7 @@ class MigratorTest {
verify { migrationJobFactory.create(capture(migrations)) } verify { migrationJobFactory.create(capture(migrations)) }
assertEquals(1, migrations.captured.size) assertEquals(1, migrations.captured.size)
verify { migrationCompletedListener() } eventually(2.seconds) { verify { migrationCompletedListener() } }
} }
@Test @Test
@@ -86,7 +89,7 @@ class MigratorTest {
verify { migrationJobFactory.create(capture(migrations)) } verify { migrationJobFactory.create(capture(migrations)) }
assertEquals(2, migrations.captured.size) assertEquals(2, migrations.captured.size)
verify { migrationCompletedListener() } eventually(2.seconds) { verify { migrationCompletedListener() } }
} }
@Test @Test
@@ -114,7 +117,7 @@ class MigratorTest {
verify { migrationJobFactory.create(capture(migrations)) } verify { migrationJobFactory.create(capture(migrations)) }
assertEquals(10, migrations.captured.size) assertEquals(10, migrations.captured.size)
verify { migrationCompletedListener() } eventually(2.seconds) { verify { migrationCompletedListener() } }
} }
@Test @Test
@@ -135,11 +138,12 @@ class MigratorTest {
verify { migrationJobFactory.create(capture(migrations)) } verify { migrationJobFactory.create(capture(migrations)) }
assertEquals(2, migrations.captured.size) assertEquals(2, migrations.captured.size)
verify { migrationCompletedListener() } eventually(2.seconds) { verify { migrationCompletedListener() } }
} }
companion object { companion object {
@OptIn(DelicateCoroutinesApi::class)
val mainThreadSurrogate = newSingleThreadContext("UI thread") val mainThreadSurrogate = newSingleThreadContext("UI thread")
@BeforeAll @BeforeAll