Skip to content

Commit cff037c

Browse files
committed
test(tools): Extend test_paste_text retry window to 5 seconds
why: The CI matrix (GitHub Actions runners) timed out the retry_until(..., 2) for "PASTE_TEST_marker_xyz" in pane capture across all six tmux matrix cells — the bash shell cold-start on a fresh pane can take several seconds before it processes the pasted echo command and renders output. pyproject's pytest-rerunfailures config reran each failure twice with no change, so this is a reliable timing issue rather than a transient flake. paste_text itself is still verified end-to-end (the marker MUST appear); only the patience budget grew. what: - Bump the timeout passed to retry_until from 2 to 5 seconds. - Add a comment explaining the cold-start rationale so future readers don't shave it back down.
1 parent 8956288 commit cff037c

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

tests/test_pane_tools.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -965,10 +965,12 @@ def test_paste_text(mcp_server: Server, mcp_pane: Pane) -> None:
965965
)
966966
assert "pasted" in result.lower()
967967

968-
# Verify the text appeared in the pane
968+
# Verify the text appeared in the pane. Use a generous retry
969+
# window: CI runners cold-start the pane's shell and the echo
970+
# output can take several seconds to render on the first run.
969971
retry_until(
970972
lambda: "PASTE_TEST_marker_xyz" in "\n".join(mcp_pane.capture_pane()),
971-
2,
973+
5,
972974
raises=True,
973975
)
974976

0 commit comments

Comments
 (0)