Skip to content

test(coverage): skip subshell-tracking tests on macOS #693

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Chemaclass merged 1 commit into from
Jun 3, 2026
Merged

test(coverage): skip subshell-tracking tests on macOS #693

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 12 additions & 6 deletions tests/unit/coverage_subshell_test.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ _ORIG_COVERAGE_EXCLUDE=""
# /tmp to deadlock CI runners. The contracts tested here are
# deterministic in single-process mode, so the parallel run is not
# a useful execution context.
function _skip_when_parallel_or_windows() {
function _skip_when_unsupported_context() {
if [ "${BASHUNIT_PARALLEL_RUN:-false}" = "true" ]; then
bashunit::skip "subshell tracking tests require single-process execution"
return 0
Expand All @@ -30,6 +30,12 @@ function _skip_when_parallel_or_windows() {
bashunit::skip "DEBUG trap + set -T behavior is unstable on Git Bash"
return 0
;;
Darwin*)
# The DEBUG-trap recorder does not propagate into subshells on the
# macOS system Bash (3.2); this works on Linux for all supported Bash.
bashunit::skip "DEBUG trap + set -T does not reach subshells on macOS Bash"
return 0
;;
esac
return 1
}
Expand Down Expand Up @@ -105,7 +111,7 @@ function _run_fixture_under_coverage() {
}

function test_coverage_records_lines_inside_command_substitution() {
_skip_when_parallel_or_windows && return 0
_skip_when_unsupported_context && return 0
local fixture
fixture=$(mktemp)
cat >"$fixture" <<'EOF'
Expand All @@ -125,7 +131,7 @@ EOF
}

function test_coverage_records_explicit_subshell_block() {
_skip_when_parallel_or_windows && return 0
_skip_when_unsupported_context && return 0
local fixture
fixture=$(mktemp)
cat >"$fixture" <<'EOF'
Expand All @@ -147,7 +153,7 @@ EOF
}

function test_coverage_records_pipeline_lhs() {
_skip_when_parallel_or_windows && return 0
_skip_when_unsupported_context && return 0
local fixture
fixture=$(mktemp)
cat >"$fixture" <<'EOF'
Expand All @@ -165,7 +171,7 @@ EOF
}

function test_coverage_records_process_substitution_consumer() {
_skip_when_parallel_or_windows && return 0
_skip_when_unsupported_context && return 0
local fixture
fixture=$(mktemp)
cat >"$fixture" <<'EOF'
Expand All @@ -186,7 +192,7 @@ EOF
}

function test_coverage_records_lines_inside_function_called_from_subshell() {
_skip_when_parallel_or_windows && return 0
_skip_when_unsupported_context && return 0
local fixture
fixture=$(mktemp)
cat >"$fixture" <<'EOF'
Expand Down
Loading