Skip to content

Commit 2638148

Browse files
committed
test(cleanup): Remove obsolete timeout restart xfail
The test_run_result_timeout_triggers_restart xfail was a placeholder added before ScriptedStdout/Stdin infrastructure existed. The exact behavior it intended to test is already covered by the parametrized test_run_result_retries_with_process_factory test case "timeout_then_retry_succeeds" (lines 537-542, 556-606).
1 parent 2619ea4 commit 2638148

File tree

1 file changed

+0
-25
lines changed

1 file changed

+0
-25
lines changed

tests/test_control_mode_engine.py

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -658,28 +658,3 @@ def test_notifications_overflow_then_iter(case: BackpressureFixture) -> None:
658658
notif = next(engine.iter_notifications(timeout=0.1), None)
659659
assert notif is not None
660660
assert notif.kind.name == "SESSIONS_CHANGED"
661-
662-
663-
class TimeoutRestartFixture(t.NamedTuple):
664-
"""Fixture for per-command timeout restart behavior."""
665-
666-
test_id: str
667-
668-
669-
@pytest.mark.xfail(
670-
reason="per-command timeout restart needs injectable control-mode transport",
671-
strict=False,
672-
)
673-
@pytest.mark.parametrize(
674-
"case",
675-
[
676-
TimeoutRestartFixture(test_id="timeout_triggers_restart_then_succeeds"),
677-
],
678-
ids=lambda c: c.test_id,
679-
)
680-
def test_run_result_timeout_triggers_restart(case: TimeoutRestartFixture) -> None:
681-
"""Placeholder: timeout should restart control process and allow next command."""
682-
_ = ControlModeEngine(command_timeout=0.0001)
683-
pytest.xfail(
684-
"control-mode needs injectable process to simulate per-call timeout",
685-
)

0 commit comments

Comments
 (0)