From 7fbfd9c10577830aea620c324a4e48aeba25e481 Mon Sep 17 00:00:00 2001 From: Caleb Allen Date: Sat, 26 Sep 2026 17:02:56 -0400 Subject: [PATCH] test(library): skip the cover-tier scan test when fixtures are absent TestScan_StoresOnlyCoverTiers built the fixture path by hand, so in a tree where make testdata had not run it failed on a missing covers directory, where every other fixture test skips via testfixtures.Load. In a fresh worktree that failure blocked the pre-push hook for every branch. Closes #266 Co-Authored-By: Claude Opus 5.5 Claude-Session: https://claude.ai/code/session_017HJiuc3ZZhxsPXz3ozTirT --- backend/library/coverart_storage_test.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/backend/library/coverart_storage_test.go b/backend/library/coverart_storage_test.go index 2c2b60d..1f0e6b7 100644 --- a/backend/library/coverart_storage_test.go +++ b/backend/library/coverart_storage_test.go @@ -8,6 +8,7 @@ import ( "yellowjacket/backend/coverart" "yellowjacket/backend/database/sql/sqlcgen" + "yellowjacket/internal/testfixtures" ) // TestScan_StoresOnlyCoverTiers pins the size decision: a scan writes @@ -26,10 +27,9 @@ func TestScan_StoresOnlyCoverTiers(t *testing.T) { lib, db := setupTestLibrary(t) - root, err := filepath.Abs("../../test_data/music_library_test") - if err != nil { - t.Fatalf("resolve fixture path: %v", err) - } + // Load skips when the fixture library has not been generated, as + // every other fixture test does. + root := testfixtures.Load(t).Root() library, err := db.Queries.CreateLibrary(lib.ctx, sqlcgen.CreateLibraryParams{ Name: "Fixtures",