From f0d22e4b6c9a46e0ce2956131aec502797655082 Mon Sep 17 00:00:00 2001 From: Brooklyn Nicholson Date: Mon, 13 Jul 2026 17:02:57 -0400 Subject: [PATCH] test(windows): capture bootstrap only in msys wrap-command tests Same fix as the base-env bootstrap tests: init_session's post-failure non-login probe was overwriting captured["script"] with `true`. Capture the first (bootstrap) _run_bash call via setdefault. --- tests/tools/test_local_env_windows_msys.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/tools/test_local_env_windows_msys.py b/tests/tools/test_local_env_windows_msys.py index 46e310dcdf48..938cd8783f18 100644 --- a/tests/tools/test_local_env_windows_msys.py +++ b/tests/tools/test_local_env_windows_msys.py @@ -353,7 +353,7 @@ class TestWrapCommandWindowsNativeCwd: captured = {} def fake_run_bash(self, cmd_string, *, login=False, timeout=120, stdin_data=None): - captured["script"] = cmd_string + captured.setdefault("script", cmd_string) # bootstrap only; ignore the failure-path probe raise RuntimeError("stop after capturing bootstrap") monkeypatch.setattr(LocalEnvironment, "_run_bash", fake_run_bash) @@ -373,7 +373,7 @@ class TestWrapCommandWindowsNativeCwd: captured = {} def fake_run_bash(self, cmd_string, *, login=False, timeout=120, stdin_data=None): - captured["script"] = cmd_string + captured.setdefault("script", cmd_string) # bootstrap only; ignore the failure-path probe raise RuntimeError("stop after capturing bootstrap") monkeypatch.setattr(LocalEnvironment, "_run_bash", fake_run_bash) @@ -402,7 +402,7 @@ class TestWrapCommandWindowsNativeCwd: captured = {} def fake_run_bash(self, cmd_string, *, login=False, timeout=120, stdin_data=None): - captured["script"] = cmd_string + captured.setdefault("script", cmd_string) # bootstrap only; ignore the failure-path probe raise RuntimeError("stop after capturing bootstrap") monkeypatch.setattr(LocalEnvironment, "_run_bash", fake_run_bash)