test(builder[on-before-script]): Add test for on_before_script not called without script key

tony · tony · commit 419f93c056b3 · 2026-03-07T17:10:35.000-06:00
why: Verify callback is not fired when session config has no before_script key.
what:
- Add test_builder_on_before_script_not_called_without_script using real server fixture
diff --git a/tests/workspace/test_progress.py b/tests/workspace/test_progress.py
@@ -32,3 +32,24 @@ def test_builder_on_progress_callback(
     assert any("Creating window:" in c for c in calls)
     assert any("Creating pane:" in c for c in calls)
     assert "Workspace built" in calls
+
+
+def test_builder_on_before_script_not_called_without_script(
+    server: Server,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    """on_before_script callback is not invoked when config has no before_script key."""
+    monkeypatch.delenv("TMUX", raising=False)
+
+    session_config = {
+        "session_name": "no-script-callback-test",
+        "windows": [{"window_name": "main", "panes": [{"shell_command": []}]}],
+    }
+    called: list[bool] = []
+    builder = WorkspaceBuilder(
+        session_config=session_config,
+        server=server,
+        on_before_script=lambda: called.append(True),
+    )
+    builder.build()
+    assert called == []
