Skip to content

Commit 23ed755

Browse files
committed
migration : fix presenter test for fresh/old install
1 parent 0246fec commit 23ed755

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

features/migration/impl/src/test/kotlin/io/element/android/features/migration/impl/MigrationPresenterTest.kt

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,9 @@ class MigrationPresenterTest {
2727
val warmUpRule = WarmUpRule()
2828

2929
@Test
30-
fun `present - no migration should occurs on fresh installation, and last version should be stored`() = runTest {
30+
fun `present - run all migrations on fresh installation, and last version should be stored`() = runTest {
3131
val migrations = (1..10).map { order ->
32-
FakeAppMigration(
33-
order = order,
34-
migrateLambda = LambdaNoParamRecorder(ensureNeverCalled = true) { },
35-
)
32+
FakeAppMigration(order = order)
3633
}
3734
val store = InMemoryMigrationStore(initialApplicationMigrationVersion = -1)
3835
val presenter = createPresenter(
@@ -44,12 +41,15 @@ class MigrationPresenterTest {
4441
}.test {
4542
val initialState = awaitItem()
4643
assertThat(initialState.migrationAction).isEqualTo(AsyncData.Uninitialized)
47-
skipItems(1)
44+
skipItems(migrations.size)
4845
awaitItem().also { state ->
4946
assertThat(state.migrationAction).isEqualTo(AsyncData.Success(Unit))
5047
}
5148
assertThat(store.applicationMigrationVersion().first()).isEqualTo(migrations.maxOf { it.order })
5249
}
50+
for (migration in migrations) {
51+
migration.migrateLambda.assertions().isCalledOnce()
52+
}
5353
}
5454

5555
@Test

0 commit comments

Comments
 (0)