Skip to content

ci: require zero orphan tests without a baseline - #4261

Closed
miguel-heygen wants to merge 0 commit into
mainfrom
hfoss39/orphans-zero
Closed

miguel-heygen wants to merge 0 commit into
mainfrom
hfoss39/orphans-zero

Conversation

@miguel-heygen

Copy link
Copy Markdown
Collaborator

Every test must be reachable through CI for the files it protects. This removes the empty reachability baseline and rejects its presence, so a new orphan cannot be accepted as debt.

The report command now shares the same failure decision as the required check. A CLI witness starts from a reachable fixture, restores an empty baseline and then plants an orphan, asserting a nonzero exit for each violation.

Stacked on #4260. The predecessor wires the remaining tests; this change makes zero permanent. The deletion is named in the existing deletion guard.

Validation: source review complete; local execution is unavailable. CI must supply the before/after report and focused test output before this draft is ready. Expected report: zero findings before and after, with baseline presence forbidden afterward. No measured CI result is claimed yet.

@miguel-heygen
miguel-heygen force-pushed the hfoss39/orphans-test-runners branch from bd9ce21 to a1b74a0 Compare September 22, 2026 01:16
@miguel-heygen
miguel-heygen force-pushed the hfoss39/orphans-test-runners branch from a1b74a0 to bf88e33 Compare September 22, 2026 01:53
@miguel-heygen
miguel-heygen force-pushed the hfoss39/orphans-test-runners branch from bf88e33 to 48e519f Compare September 22, 2026 02:07
@miguel-heygen
miguel-heygen force-pushed the hfoss39/orphans-zero branch 2 times, most recently from 189fb85 to 6e0a22e Compare September 22, 2026 02:37
@miguel-heygen
miguel-heygen force-pushed the hfoss39/orphans-test-runners branch from 7edec02 to 38c218b Compare September 22, 2026 04:16
Base automatically changed from hfoss39/orphans-test-runners to main September 22, 2026 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant