From 622c90a2d5799da158b05a19d4c7e94e0cdbd533 Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Tue, 26 May 2026 18:48:12 -0400 Subject: [PATCH] =?UTF-8?q?fix(android):=20LibraryViewModelTest=20?= =?UTF-8?q?=E2=80=94=20pass=20SyncController=20to=20constructor?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit LibraryViewModel gained a SyncController constructor parameter for pull-to-refresh (cf07a2a). Tests use a relaxed MockK SyncController since none of the four cases exercise refresh(). Co-Authored-By: Claude Opus 4.7 (1M context) --- .../minstrel/library/ui/LibraryViewModelTest.kt | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/android/app/src/test/java/com/fabledsword/minstrel/library/ui/LibraryViewModelTest.kt b/android/app/src/test/java/com/fabledsword/minstrel/library/ui/LibraryViewModelTest.kt index 17c33486..56dbd113 100644 --- a/android/app/src/test/java/com/fabledsword/minstrel/library/ui/LibraryViewModelTest.kt +++ b/android/app/src/test/java/com/fabledsword/minstrel/library/ui/LibraryViewModelTest.kt @@ -1,6 +1,7 @@ package com.fabledsword.minstrel.library.ui import app.cash.turbine.test +import com.fabledsword.minstrel.cache.sync.SyncController import com.fabledsword.minstrel.library.data.LibraryRepository import com.fabledsword.minstrel.models.AlbumRef import com.fabledsword.minstrel.models.ArtistRef @@ -25,7 +26,7 @@ class LibraryViewModelTest { every { repo.observeArtists() } returns MutableSharedFlow() every { repo.observeAlbums() } returns MutableSharedFlow() - val vm = LibraryViewModel(repo) + val vm = LibraryViewModel(repo, mockk(relaxed = true)) assertEquals(LibraryUiState.Loading, vm.uiState.value) } @@ -43,7 +44,7 @@ class LibraryViewModelTest { every { repo.observeArtists() } returns flowOf(emptyList()) every { repo.observeAlbums() } returns flowOf(emptyList()) - val vm = LibraryViewModel(repo) + val vm = LibraryViewModel(repo, mockk(relaxed = true)) vm.uiState.test { assertEquals(LibraryUiState.Empty, awaitItem()) @@ -59,7 +60,7 @@ class LibraryViewModelTest { every { repo.observeAlbums() } returns flowOf(listOf(AlbumRef(id = "al1", title = "Geogaddi", artistId = "a1"))) - val vm = LibraryViewModel(repo) + val vm = LibraryViewModel(repo, mockk(relaxed = true)) vm.uiState.test { val success = awaitItem() as LibraryUiState.Success @@ -78,7 +79,7 @@ class LibraryViewModelTest { flow { throw IllegalStateException("DAO blew up") } every { repo.observeAlbums() } returns flowOf(emptyList()) - val vm = LibraryViewModel(repo) + val vm = LibraryViewModel(repo, mockk(relaxed = true)) vm.uiState.test { val error = awaitItem() as LibraryUiState.Error