Repository navigation
test: run the suite in parallel (bats --jobs), 48 min -> ~4.5 min - #296
Merged
Merged
Conversation
CI installs GNU parallel and runs one bats job per CPU (serial when parallel is absent). Measured on the Mac: 48 min serial for 2055 tests, 254-264 s with --jobs 14 (3 clean runs in a row). Three tests were not safe side by side: - tests/helpers.bash: one cache dir per test (BATS_TEST_TMPDIR); auto-fix run logs (FI_AF_RUNS) live there and the classify tests mixed them. - tests/autofix-cancel.bats: sleep lengths unique to the test process and an exact pkill, instead of pkill 'sleep 471[34]' killing other tests'. tests/fix-batch8-long-root.bats carried the same pkill without ever starting those sleeps; removed. - tests/cli-sync-default-branch.bats: its own XDG_CACHE_HOME instead of the real ~/.cache plus a teardown that emptied it for everyone. Logs tests/helpers.bash:25 (tests keeping the real HOME write ~/.cache). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MVx3HWniZnrXYfPiCo5Lrs
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MVx3HWniZnrXYfPiCo5Lrs
AltDoug
enabled auto-merge (squash)
October 30, 2026 03:33
AltDoug
added a commit
that referenced
this pull request
Oct 30, 2026
…0, tests/autofix-sweep.bats) (#297) Claude-Session: https://claude.ai/code/session_01MVx3HWniZnrXYfPiCo5Lrs Co-authored-by: Diogo Silva Sena <227921880+DougBTW@users.noreply.github.com> Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
bats --jobs <CPUs> tests/; serialbats tests/when parallel is missing.tests/helpers.bash: oneFOUND_ISSUES_CACHE_DIRper test (BATS_TEST_TMPDIR). Auto-fix run logs (FI_AF_RUNS) live there; the classify tests read each other's.tests/autofix-cancel.bats: sleep lengths unique to the test process plus an exactpkill, instead ofpkill -f 'sleep 471[34]', which killed other tests' processes.tests/fix-batch8-long-root.batshad the samepkillwithout ever starting those sleeps; removed.tests/cli-sync-default-branch.bats: its ownXDG_CACHE_HOME, instead of the real~/.cacheplus a teardown that emptied it for every test.Why
Ledger
tests/autofix-sweep.bats(operator decision 2026-10-29): the serial suite reached 48 min on the Mac against the 60-minfix shipgate.Evidence
1..2055, 2055 ok, about 48 min.bats --jobs 14 tests/before the fixes: 258 s, 4 failures (cancel ×2, classify ×2). After them: 254 s, 261 s, 264 s, each1..2055 ok=2055 notok=0, no leftover sleeps.bats jobs: 14; without it,bats jobs: 1and the serial path.Also logs
tests/helpers.bash:25: tests that keep the real HOME write~/.cache/found-issues(measured:mode_foo_barrewritten during a suite run).🤖 Generated with Claude Code
https://claude.ai/code/session_01MVx3HWniZnrXYfPiCo5Lrs