Skip to content

chore: remove leaked test artifacts from repo root - #1365

Merged
Chemaclass merged 1 commit into
mainfrom
chore/remove-leaked-root-artifacts
Sep 9, 2026
Merged

chore: remove leaked test artifacts from repo root#1365
Chemaclass merged 1 commit into
mainfrom
chore/remove-leaked-root-artifacts

Conversation

@Chemaclass

Copy link
Copy Markdown
Member

🤔 Background

.RAN_TEARDOWN and .setup are two empty files sitting at the repo root. Nothing in the project uses them — they were swept into 40560c2 by a git add -A while #1329 was being fixed. Two 0-byte files that look deliberate cost the next contributor a grep to find out they are not.

💡 Changes

  • Delete both files. git log -S'RAN_TEARDOWN' --all finds no commit whose content ever mentioned the name, and .setup is what a dev-time run of the A test killed by --test-timeout does not run tear_down #1324 timeout test wrote into the working directory back when its fixture expanded "$TIMEOUT_MARKER.setup" with the marker unset.
  • Neither is regenerated: that fixture now points TIMEOUT_MARKER at a path under bashunit::temp_dir, and the sequential and --parallel suites both finish clean without them.
  • No .gitignore entry to go with it — a rule guarding a name nothing writes is the same false landmark in another file.
  • No CHANGELOG entry: nothing user-facing. They shipped in no artifact, since package.json lists only bin/bashunit, LICENSE and README.md, and bpkg.json only bashunit.

https://claude.ai/code/session_01EXYWTGLjf7qM8Ru3GakDRm

`.RAN_TEARDOWN` and `.setup` are empty files swept into 40560c2 by a
`git add -A`, not something the project uses. Nothing reads them:
`git log -S'RAN_TEARDOWN' --all` finds no commit whose content ever
mentioned the name, and `.setup` is what a dev-time run of the #1324
timeout test wrote into the working directory back when its fixture
expanded `"$TIMEOUT_MARKER.setup"` with the marker unset. The test now
sets `TIMEOUT_MARKER` to a path under `bashunit::temp_dir`, so neither
file is regenerated.

They shipped in no artifact -- `package.json` lists only `bin/bashunit`,
`LICENSE` and `README.md`, and `bpkg.json` only `bashunit` -- so this is
repo hygiene, not a release fix. No `.gitignore` entry to go with it: a
rule guarding a name nothing writes is the same false landmark in
another file.

Claude-Session: https://claude.ai/code/session_01EXYWTGLjf7qM8Ru3GakDRm
@Chemaclass Chemaclass added the refactoring Refactoring or cleaning related label Sep 9, 2026
@Chemaclass Chemaclass self-assigned this Sep 9, 2026
@Chemaclass
Chemaclass merged commit b9c6506 into main Sep 9, 2026
37 checks passed
@Chemaclass
Chemaclass deleted the chore/remove-leaked-root-artifacts branch September 9, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Refactoring or cleaning related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant