Skip to content

test(health): neutral repo placeholders in the conductor fixtures #202

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
Jammy2211 merged 1 commit into from
Aug 5, 2026
Merged

test(health): neutral repo placeholders in the conductor fixtures #202

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
15 changes: 10 additions & 5 deletions tests/test_health_conductor.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,11 @@
# free slot 1 (bash's own generic failure), because STALE passes the ship gate.
EXIT_GREEN, EXIT_YELLOW, EXIT_RED, EXIT_UNKNOWN, EXIT_USAGE, EXIT_STALE = 0, 2, 3, 4, 5, 6

# Reason strings are stubbed Heart output. The repo half of a "<repo>: <problem>"
# reason is deliberately a NEUTRAL placeholder, never a real satellite repo name:
# nothing here asserts on it (the conductor classifies by verdict and reason
# text, not by who reported it), so a real name would be an instance fact the
# tenant firewall counts against organ code for no test value.
STALE_ONLY = {
"verdict": "stale",
"score": 75,
Expand All @@ -37,7 +42,7 @@
"stale_reasons": [
"no release validation for current source",
"install verification not run",
"PyAutoLens: status unknown",
"library-a: status unknown",
],
}

Expand Down Expand Up @@ -174,9 +179,9 @@ def test_every_verdict_maps_to_a_distinct_exit_code(tmp_path):
EXIT_GREEN: {"verdict": "green", "red_reasons": [], "yellow_reasons": [],
"stale_reasons": []},
EXIT_YELLOW: {"verdict": "yellow", "red_reasons": [],
"yellow_reasons": ["PyAutoFit: uncommitted changes"],
"yellow_reasons": ["library-a: uncommitted changes"],
"stale_reasons": []},
EXIT_RED: {"verdict": "red", "red_reasons": ["PyAutoFit: CI failing on main"],
EXIT_RED: {"verdict": "red", "red_reasons": ["library-a: CI failing on main"],
"yellow_reasons": [], "stale_reasons": []},
EXIT_UNKNOWN: {},
EXIT_STALE: STALE_ONLY,
Expand Down Expand Up @@ -242,7 +247,7 @@ def test_red_still_dominates_a_stale_reason(tmp_path):
never recommend a refresh while a blocker is open."""
t, code = _triage(tmp_path, {
"verdict": "red", "score": 40,
"red_reasons": ["PyAutoFit: CI failing on main"],
"red_reasons": ["library-a: CI failing on main"],
"yellow_reasons": [],
"stale_reasons": ["install verification not run"],
})
Expand All @@ -254,7 +259,7 @@ def test_red_still_dominates_a_stale_reason(tmp_path):
def test_yellow_real_warning_still_wins_over_a_stale_reason(tmp_path):
t, code = _triage(tmp_path, {
"verdict": "yellow", "score": 60, "red_reasons": [],
"yellow_reasons": ["PyAutoLens: uncommitted changes"],
"yellow_reasons": ["library-b: uncommitted changes"],
"stale_reasons": ["install verification not run"],
})
assert code == EXIT_YELLOW
Expand Down
Loading