From 13582efcae3cd799079440b3a121360758df5b44 Mon Sep 17 00:00:00 2001 From: Chemaclass Date: Sun, 3 May 2026 10:51:56 +0200 Subject: [PATCH] chore(shellcheck): silence intentional SC2016 in coverage tests The two `done` fixtures pass literal `$var`/`$file` strings to `is_executable_line` on purpose; flag them with `# shellcheck disable=SC2016` so `make sa` runs noise-free. --- tests/unit/coverage_executable_test.sh | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/unit/coverage_executable_test.sh b/tests/unit/coverage_executable_test.sh index 453ab930..9e257752 100644 --- a/tests/unit/coverage_executable_test.sh +++ b/tests/unit/coverage_executable_test.sh @@ -241,6 +241,7 @@ function test_coverage_is_executable_line_returns_false_for_done_with_file_redir function test_coverage_is_executable_line_returns_false_for_done_with_herestring() { local result + # shellcheck disable=SC2016 result=$(bashunit::coverage::is_executable_line ' done <<<"$var"' 2 && echo "yes" || echo "no") assert_equals "no" "$result" } @@ -253,6 +254,7 @@ function test_coverage_is_executable_line_returns_false_for_done_with_process_su function test_coverage_is_executable_line_returns_false_for_done_with_redirect_and_comment() { local result + # shellcheck disable=SC2016 result=$(bashunit::coverage::is_executable_line ' done < "$file" # read input' 2 && echo "yes" || echo "no") assert_equals "no" "$result" }