test(issue-17): the frozen wrapper literals follow _decided/_exit_for (review agy)

a670ef5 routed the wrapper's sub-status and exit-code decisions through
_decided / _exit_for, but test_tier1_unit G-10 and test_issue5's
test_wrapper_exit_code_mapping_literals_are_frozen still looked for the raw
`_decide sub-status "$sub_rc"` and `_decide exit-code "$job_status"`. They now
pin `_decided job_status sub-status "$sub_rc"` and `_exit_for "$job_status"`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HCGpJXhSZasJLU38ksDghe
This commit is contained in:
GodopuandClaude Opus 5.5 committed 2026-10-08 09:26:41 +09:00
1 parent 8241bd53b1
commit 541c3febe8
2 files changed
+3 -3

No files matched your search

+2 -2
View File
@@ -386,8 +386,8 @@ def test_wrapper_exit_code_mapping_literals_are_frozen(mam_sandbox):
wrapper = SKILLS / "multi-agent-mux-delegate-job" / "multi-agent-mux-delegate-job" wrapper = SKILLS / "multi-agent-mux-delegate-job" / "multi-agent-mux-delegate-job"
text = wrapper.read_text(encoding="utf-8") text = wrapper.read_text(encoding="utf-8")
# Issue #17 item 4: the mapping moved to lib_py/delegation/wrapper.py; the # Issue #17 item 4: the mapping moved to lib_py/delegation/wrapper.py; the
# wrapper reaches it only through `_decide`. # wrapper reaches it only through `_decide`, via _decided / _exit_for (N2).
for literal in ('_decide sub-status "$sub_rc"', '_decide exit-code "$job_status"', "--sub-rc \"$sub_rc\""): for literal in ('_decided job_status sub-status "$sub_rc"', '_exit_for "$job_status"', "--sub-rc \"$sub_rc\""):
assert literal in text, literal assert literal in text, literal
from lib_py.delegation import wrapper as decide # no scripts/ shim: the wrapper calls it with -m from lib_py.delegation import wrapper as decide # no scripts/ shim: the wrapper calls it with -m
assert decide.EXIT_CODES == {"completed": 0, "error": 1, "timeout": 2, "broker_unavailable": 3, "cancelled": 5} assert decide.EXIT_CODES == {"completed": 0, "error": 1, "timeout": 2, "broker_unavailable": 3, "cancelled": 5}
+1 -1
View File
@@ -844,7 +844,7 @@ def test_g10_delegate_job_rc3_not_mistaken_for_error(mam_sandbox):
"""G-10: multi-agent-mux-delegate-job maps rc=3 to broker_unavailable and checks disk status. """G-10: multi-agent-mux-delegate-job maps rc=3 to broker_unavailable and checks disk status.
Issue #17 item 4: the mapping is lib_py/delegation/wrapper.py's sub_status.""" Issue #17 item 4: the mapping is lib_py/delegation/wrapper.py's sub_status."""
delegate_script = mam_sandbox / "skills" / "multi-agent-mux-delegate-job" / "multi-agent-mux-delegate-job" delegate_script = mam_sandbox / "skills" / "multi-agent-mux-delegate-job" / "multi-agent-mux-delegate-job"
assert '_decide sub-status "$sub_rc"' in delegate_script.read_text() assert '_decided job_status sub-status "$sub_rc"' in delegate_script.read_text()
from lib_py.delegation import wrapper from lib_py.delegation import wrapper
reg = mam_sandbox / "g10-reg" reg = mam_sandbox / "g10-reg"
reg.mkdir() reg.mkdir()