From 83c9065ff723135acfb8bce141036d2edb336a41 Mon Sep 17 00:00:00 2001 From: Bastian Dehn Date: Fri, 25 Sep 2026 13:15:02 +0200 Subject: [PATCH] add cleanup cover --- src/albentohandy | 8 ++++++++ tests/albentohandy.bats | 26 ++++++++++++++++++++++++++ 2 files changed, 34 insertions(+) diff --git a/src/albentohandy b/src/albentohandy index f76219e..5972896 100755 --- a/src/albentohandy +++ b/src/albentohandy @@ -70,6 +70,11 @@ make_symlinks_to_work() done } +cleanup_cover() +{ + find "$OUTPUT_PATH" -regex '.*cover\.\(txt\|jpg\)' -exec rm {} \; +} + main() { local albumlist="albumlist" @@ -103,6 +108,9 @@ main() encode "$OUTPUT_PATH" \ || errlog "could not encode $OUTPUT_PATH" \ || return $? + cleanup_cover \ + || errlog "could not cleanup cover" \ + || return $? } [ "$TEST_MODE" == "ON" ] && return 0 diff --git a/tests/albentohandy.bats b/tests/albentohandy.bats index 9f44ed9..fc093ff 100755 --- a/tests/albentohandy.bats +++ b/tests/albentohandy.bats @@ -188,6 +188,10 @@ teardown() { } @test "success: copy cover.jpg for Alben/Artist1" { + cleanup_cover() { + : + } + run main -i "build/tests/albumlist" assert_success @@ -226,4 +230,26 @@ teardown() { assert_failure assert_line --index 0 "❌ ERROR: could not encode build/tests/output" +} + +@test "success: cleanup cover" { + mkdir --parents "build/tests/output/Artist1/" + echo "metadatacover" > "build/tests/output/Artist1/cover.txt" + + run main -i "build/tests/albumlist" + + assert_success + [ ! -f "build/tests/output/Artist1/cover.txt" ] + [ ! -f "build/tests/output/Artist1/cover.jpg" ] +} + +@test "failed: cleanup cover" { + cleanup_cover() { + return 1 + } + + run main -i "build/tests/albumlist" + + assert_failure + assert_line --index 0 "❌ ERROR: could not cleanup cover" } \ No newline at end of file -- 2.47.3