From 93f54e00a432ea5fed98ffb76e59bad548146167 Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Mon, 4 May 2026 17:31:27 -0400 Subject: [PATCH] =?UTF-8?q?fix(server,web/m7-353):=20forward-fix=20CI=20?= =?UTF-8?q?=E2=80=94=20Mount=20call=20+=20AlbumRef=20test=20fixtures?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- internal/api/library_test.go | 2 +- web/src/lib/components/AlbumCard.test.ts | 1 + web/src/routes/albums/[id]/album.test.ts | 2 ++ web/src/routes/artists/[id]/artist.test.ts | 3 ++- web/src/routes/library/liked/liked.test.ts | 2 +- web/src/routes/search/albums/albums.test.ts | 2 +- web/src/routes/search/search.test.ts | 2 +- 7 files changed, 9 insertions(+), 5 deletions(-) diff --git a/internal/api/library_test.go b/internal/api/library_test.go index c267ce60..d15fec3b 100644 --- a/internal/api/library_test.go +++ b/internal/api/library_test.go @@ -443,7 +443,7 @@ func TestRoutesRegisteredInMount(t *testing.T) { r := chi.NewRouter() w := playevents.NewWriter(h.pool, slog.New(slog.NewTextHandler(io.Discard, nil)), 30*time.Minute, 0.5, 30000) - Mount(r, h.pool, h.logger, w, config.RecommendationConfig{RadioSize: 50, RadioSizeMax: 200, RecentlyPlayedHours: 1}, h.lidarrCfg, h.lidarrRequests, h.lidarrQuarantine, h.tracks, h.playlists, h.dataDir) + Mount(r, h.pool, h.logger, w, config.RecommendationConfig{RadioSize: 50, RadioSizeMax: 200, RecentlyPlayedHours: 1}, h.lidarrCfg, h.lidarrRequests, h.lidarrQuarantine, h.tracks, h.playlists, h.coverart, h.coverArtBackfillCap, h.dataDir) paths := []string{ "/api/artists", diff --git a/web/src/lib/components/AlbumCard.test.ts b/web/src/lib/components/AlbumCard.test.ts index 125e533b..37bedf51 100644 --- a/web/src/lib/components/AlbumCard.test.ts +++ b/web/src/lib/components/AlbumCard.test.ts @@ -60,6 +60,7 @@ const album: AlbumRef = { track_count: 5, duration_sec: 2630, cover_url: '/api/albums/xyz/cover', + cover_art_source: null, }; afterEach(() => vi.clearAllMocks()); diff --git a/web/src/routes/albums/[id]/album.test.ts b/web/src/routes/albums/[id]/album.test.ts index fa39ac20..ce57c879 100644 --- a/web/src/routes/albums/[id]/album.test.ts +++ b/web/src/routes/albums/[id]/album.test.ts @@ -60,6 +60,7 @@ describe('album detail page', () => { artist_id: 'md', artist_name: 'Miles Davis', year: 1959, track_count: 2, duration_sec: 544 + 565, cover_url: '/api/albums/xyz/cover', + cover_art_source: null, tracks: [track('t1', 'So What', 1, 544), track('t2', 'Freddie Freeloader', 2, 565)] }; (createAlbumQuery as ReturnType).mockReturnValue(mockQuery({ data: detail })); @@ -87,6 +88,7 @@ describe('album detail page', () => { artist_id: 'md', artist_name: 'Miles Davis', track_count: 0, duration_sec: 0, cover_url: '/api/albums/xyz/cover', + cover_art_source: null, tracks: [] }; (createAlbumQuery as ReturnType).mockReturnValue(mockQuery({ data: detail })); diff --git a/web/src/routes/artists/[id]/artist.test.ts b/web/src/routes/artists/[id]/artist.test.ts index 90b61b80..35819abd 100644 --- a/web/src/routes/artists/[id]/artist.test.ts +++ b/web/src/routes/artists/[id]/artist.test.ts @@ -43,7 +43,8 @@ function album(id: string, title: string, year?: number): AlbumRef { sort_title: title, artist_id: 'abc', artist_name: 'Alice', year, track_count: 10, duration_sec: 2400, - cover_url: `/api/albums/${id}/cover` + cover_url: `/api/albums/${id}/cover`, + cover_art_source: null }; } diff --git a/web/src/routes/library/liked/liked.test.ts b/web/src/routes/library/liked/liked.test.ts index 8dfb625e..e88411f4 100644 --- a/web/src/routes/library/liked/liked.test.ts +++ b/web/src/routes/library/liked/liked.test.ts @@ -45,7 +45,7 @@ describe('liked library page', () => { const ar: ArtistRef = { id: 'a1', name: 'X', sort_name: 'X', album_count: 1, cover_url: '' }; const al: AlbumRef = { id: 'al1', title: 'Y', sort_title: 'Y', artist_id: 'a1', artist_name: 'X', - year: 2020, track_count: 1, duration_sec: 100, cover_url: '/x' + year: 2020, track_count: 1, duration_sec: 100, cover_url: '/x', cover_art_source: null }; const tr: TrackRef = { id: 't1', title: 'Z', album_id: 'al1', album_title: 'Y', diff --git a/web/src/routes/search/albums/albums.test.ts b/web/src/routes/search/albums/albums.test.ts index 78a3e6e2..456f629d 100644 --- a/web/src/routes/search/albums/albums.test.ts +++ b/web/src/routes/search/albums/albums.test.ts @@ -45,7 +45,7 @@ function page(items: T[], total: number, offset = 0, limit = 50): Page { const album: AlbumRef = { id: 'al1', title: 'Kind of Blue', sort_title: 'Kind of Blue', artist_id: 'a1', artist_name: 'Miles Davis', - year: 1959, track_count: 5, duration_sec: 2630, cover_url: '/api/albums/al1/cover' + year: 1959, track_count: 5, duration_sec: 2630, cover_url: '/api/albums/al1/cover', cover_art_source: null }; afterEach(() => { diff --git a/web/src/routes/search/search.test.ts b/web/src/routes/search/search.test.ts index 18bfd292..531de004 100644 --- a/web/src/routes/search/search.test.ts +++ b/web/src/routes/search/search.test.ts @@ -56,7 +56,7 @@ function emptyResp(): SearchResponse { const artist: ArtistRef = { id: 'a1', name: 'Miles Davis', sort_name: 'Miles Davis', album_count: 12, cover_url: '' }; const album: AlbumRef = { id: 'al1', title: 'Kind of Blue', sort_title: 'Kind of Blue', artist_id: 'a1', artist_name: 'Miles Davis', - year: 1959, track_count: 5, duration_sec: 2630, cover_url: '/api/albums/al1/cover' + year: 1959, track_count: 5, duration_sec: 2630, cover_url: '/api/albums/al1/cover', cover_art_source: null }; const track: TrackRef = { id: 't1', title: 'So What',