From bbe0d5cd9e935d707307af9ea24467e102069074 Mon Sep 17 00:00:00 2001 From: Claudio Ortolina Date: Sun, 19 Apr 2026 22:52:08 +0100 Subject: [PATCH] Pin artist_refresh_discogs_data assertions Bind the returned ArtistInfo. For the no-discogs branch, assert discogs_data was not touched. Refs #176 --- .../worker/artist_refresh_discogs_data_test.exs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/test/music_library/worker/artist_refresh_discogs_data_test.exs b/test/music_library/worker/artist_refresh_discogs_data_test.exs index 45e51763..7b2177b3 100644 --- a/test/music_library/worker/artist_refresh_discogs_data_test.exs +++ b/test/music_library/worker/artist_refresh_discogs_data_test.exs @@ -21,7 +21,8 @@ defmodule MusicLibrary.Worker.ArtistRefreshDiscogsDataTest do Req.Test.json(conn, ArtistFixture.get_artist()) end) - assert {:ok, _} = perform_job(ArtistRefreshDiscogsData, %{"id" => artist_info.id}) + assert {:ok, refreshed} = perform_job(ArtistRefreshDiscogsData, %{"id" => artist_info.id}) + assert refreshed.id == artist_info.id updated = Artists.get_artist_info!(artist_info.id) assert updated.discogs_data == ArtistFixture.get_artist() @@ -31,7 +32,9 @@ defmodule MusicLibrary.Worker.ArtistRefreshDiscogsDataTest do artist_info = artist_info_fixture(%{musicbrainz_data: %{"name" => "No Discogs Artist"}}) - assert {:ok, _} = perform_job(ArtistRefreshDiscogsData, %{"id" => artist_info.id}) + assert {:ok, unchanged} = perform_job(ArtistRefreshDiscogsData, %{"id" => artist_info.id}) + assert unchanged.id == artist_info.id + assert unchanged.discogs_data == artist_info.discogs_data end end end