Server(test[connect]): cover connect() under control engine

tony · tony · commit 291be61c4e5c · 2025-12-06T11:53:32.000-06:00
why: Ensure Server.connect behaves identically for subprocess and control engines.
what:
- Parametrize connect tests with engines marker for both modes
diff --git a/tests/test_server.py b/tests/test_server.py
@@ -158,13 +158,15 @@ def test_new_session_shell_env(server: Server) -> None:
     assert pane_start_command.replace('"', "") == cmd
 
 
+@pytest.mark.engines(["subprocess", "control"])
 def test_connect_creates_new_session(server: Server) -> None:
     """Server.connect creates a new session when it doesn't exist."""
     session = server.connect("test_connect_new")
     assert session.name == "test_connect_new"
     assert session.session_id is not None
 
 
+@pytest.mark.engines(["subprocess", "control"])
 def test_connect_reuses_existing_session(server: Server, session: Session) -> None:
     """Server.connect reuses an existing session instead of creating a new one."""
     # First call creates
@@ -178,6 +180,7 @@ def test_connect_reuses_existing_session(server: Server, session: Session) -> No
     assert session2.name == "test_connect_reuse"
 
 
+@pytest.mark.engines(["subprocess", "control"])
 def test_connect_invalid_name(server: Server) -> None:
     """Server.connect raises BadSessionName for invalid session names."""
     with pytest.raises(exc.BadSessionName):
