From 1f457f0f1138cd685358709695c91a95464017f6 Mon Sep 17 00:00:00 2001 From: Claudio Ortolina Date: Mon, 20 Apr 2026 06:28:54 +0100 Subject: [PATCH] Pin prune_assets test to %Asset{hash: ^hash} match Refs #176 --- test/music_library/worker/prune_assets_test.exs | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/test/music_library/worker/prune_assets_test.exs b/test/music_library/worker/prune_assets_test.exs index 5fc21acd..712706ae 100644 --- a/test/music_library/worker/prune_assets_test.exs +++ b/test/music_library/worker/prune_assets_test.exs @@ -4,14 +4,16 @@ defmodule MusicLibrary.Worker.PruneAssetsTest do import MusicLibrary.Fixtures.Records alias MusicLibrary.Assets + alias MusicLibrary.Assets.Asset alias MusicLibrary.Worker.PruneAssets describe "perform/1" do @describetag :capture_log test "deletes unreferenced assets" do {:ok, orphan} = Assets.store(%{content: "orphan_data", format: "image/jpeg"}) + orphan_hash = orphan.hash - assert Assets.get(orphan.hash) != nil + assert %Asset{hash: ^orphan_hash} = Assets.get(orphan_hash) assert :ok = perform_job(PruneAssets, %{}) @@ -20,10 +22,11 @@ defmodule MusicLibrary.Worker.PruneAssetsTest do test "keeps assets referenced by records" do record = record() + cover_hash = record.cover_hash assert :ok = perform_job(PruneAssets, %{}) - assert Assets.get(record.cover_hash) != nil + assert %Asset{hash: ^cover_hash} = Assets.get(cover_hash) end test "keeps assets referenced by artist info" do @@ -35,7 +38,8 @@ defmodule MusicLibrary.Worker.PruneAssetsTest do # The artist info image data comes from Discogs fixture which stores an asset # The record's cover asset should still be present - assert Assets.get(record.cover_hash) != nil + cover_hash = record.cover_hash + assert %Asset{hash: ^cover_hash} = Assets.get(cover_hash) end test "succeeds with no unreferenced assets" do