!squash revert tests

Author: tony

tests/cli/test_cli.py

@@ -1,7 +1,5 @@
 """CLI tests for tmuxp's core shell functionality."""
 
-from __future__ import annotations
-
 import argparse
 import contextlib
 import pathlib
@@ -114,7 +112,7 @@ def test_pass_config_dir_argparse(
     def config_cmd(workspace_file: str) -> None:
         tmuxp_echo(find_workspace_file(workspace_file, workspace_dir=configdir))
 
-    def check_cmd(config_arg: str) -> _pytest.capture.CaptureResult[str]:
+    def check_cmd(config_arg: str) -> "_pytest.capture.CaptureResult[str]":
         args = parser.parse_args([config_arg])
         config_cmd(workspace_file=args.workspace_file)
         return capsys.readouterr()
@@ -132,7 +130,7 @@ def check_cmd(config_arg: str) -> _pytest.capture.CaptureResult[str]:
 
 def test_reattach_plugins(
     monkeypatch_plugin_test_packages: None,
-    server: Server,
+    server: "Server",
 ) -> None:
     """Test reattach plugin hook."""
     config_plugins = test_utils.read_workspace_file("workspace/builder/plugin_r.yaml")

tests/cli/test_debug_info.py

@@ -1,16 +1,11 @@
 """CLI tests for tmuxp debuginfo."""
 
-from __future__ import annotations
+import pathlib
 
-import typing as t
+import pytest
 
 from tmuxp import cli
 
-if t.TYPE_CHECKING:
-    import pathlib
-
-    import pytest
-
 
 def test_debug_info_cli(
     monkeypatch: pytest.MonkeyPatch,

tests/cli/test_load.py

@@ -1,7 +1,5 @@
 """CLI tests for tmuxp load."""
 
-from __future__ import annotations
-
 import contextlib
 import io
 import pathlib
@@ -12,6 +10,7 @@
 from libtmux.common import has_lt_version
 from libtmux.server import Server
 from libtmux.session import Session
+from pytest_mock import MockerFixture
 
 from tests.constants import FIXTURE_PATH
 from tests.fixtures import utils as test_utils
@@ -28,7 +27,7 @@
 
 
 def test_load_workspace(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Generic test for loading a tmuxp workspace via tmuxp load."""
@@ -50,7 +49,7 @@ def test_load_workspace(
 
 
 def test_load_workspace_passes_tmux_config(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Test tmuxp load with a tmux configuration file."""
@@ -74,7 +73,7 @@ def test_load_workspace_passes_tmux_config(
 
 
 def test_load_workspace_named_session(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Test tmuxp load with a custom tmux session name."""
@@ -102,7 +101,7 @@ def test_load_workspace_named_session(
 )
 def test_load_workspace_name_match_regression_252(
     tmp_path: pathlib.Path,
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Test tmuxp load for a regression where tmux shell names would not match."""
@@ -142,7 +141,7 @@ def test_load_workspace_name_match_regression_252(
 
 
 def test_load_symlinked_workspace(
-    server: Server,
+    server: "Server",
     tmp_path: pathlib.Path,
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
@@ -186,7 +185,6 @@ def test_load_symlinked_workspace(
 
 
 if t.TYPE_CHECKING:
-    from pytest_mock import MockerFixture
     from typing_extensions import TypeAlias
 
     ExpectedOutput: TypeAlias = t.Optional[t.Union[str, list[str]]]
@@ -199,14 +197,14 @@ class CLILoadFixture(t.NamedTuple):
     test_id: str
 
     # test params
-    cli_args: list[str | list[str]]
+    cli_args: list[t.Union[str, list[str]]]
     config_paths: list[str]
     session_names: list[str]
     expected_exit_code: int
-    expected_in_out: ExpectedOutput = None
-    expected_not_in_out: ExpectedOutput = None
-    expected_in_err: ExpectedOutput = None
-    expected_not_in_err: ExpectedOutput = None
+    expected_in_out: "ExpectedOutput" = None
+    expected_not_in_out: "ExpectedOutput" = None
+    expected_in_err: "ExpectedOutput" = None
+    expected_not_in_err: "ExpectedOutput" = None
 
 
 TEST_LOAD_FIXTURES: list[CLILoadFixture] = [
@@ -291,7 +289,7 @@ class CLILoadFixture(t.NamedTuple):
 def test_load(
     tmp_path: pathlib.Path,
     tmuxp_configdir: pathlib.Path,
-    server: Server,
+    server: "Server",
     session: Session,
     capsys: pytest.CaptureFixture[str],
     monkeypatch: pytest.MonkeyPatch,
@@ -300,10 +298,10 @@ def test_load(
     config_paths: list[str],
     session_names: list[str],
     expected_exit_code: int,
-    expected_in_out: ExpectedOutput,
-    expected_not_in_out: ExpectedOutput,
-    expected_in_err: ExpectedOutput,
-    expected_not_in_err: ExpectedOutput,
+    expected_in_out: "ExpectedOutput",
+    expected_not_in_out: "ExpectedOutput",
+    expected_in_err: "ExpectedOutput",
+    expected_not_in_err: "ExpectedOutput",
 ) -> None:
     """Parametrized test battery for tmuxp load CLI command."""
     assert server.socket_name is not None
@@ -348,7 +346,7 @@ def test_load(
 
 def test_regression_00132_session_name_with_dots(
     tmp_path: pathlib.Path,
-    server: Server,
+    server: "Server",
     session: Session,
     capsys: pytest.CaptureFixture[str],
 ) -> None:
@@ -368,7 +366,7 @@ def test_load_zsh_autotitle_warning(
     tmp_path: pathlib.Path,
     monkeypatch: pytest.MonkeyPatch,
     capsys: pytest.CaptureFixture[str],
-    server: Server,
+    server: "Server",
 ) -> None:
     """Test loading ZSH without DISABLE_AUTO_TITLE raises warning."""
     # create dummy tmuxp yaml so we don't get yelled at
@@ -550,7 +548,7 @@ def test_load_plugins_plugin_missing(
 
 def test_plugin_system_before_script(
     monkeypatch_plugin_test_packages: None,
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Test tmuxp load with sessions using before_script."""
@@ -572,7 +570,7 @@ def test_plugin_system_before_script(
 
 
 def test_load_attached(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
     mocker: MockerFixture,
 ) -> None:
@@ -594,7 +592,7 @@ def test_load_attached(
 
 
 def test_load_attached_detached(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
     mocker: MockerFixture,
 ) -> None:
@@ -616,7 +614,7 @@ def test_load_attached_detached(
 
 
 def test_load_attached_within_tmux(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
     mocker: MockerFixture,
 ) -> None:
@@ -638,7 +636,7 @@ def test_load_attached_within_tmux(
 
 
 def test_load_attached_within_tmux_detached(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
     mocker: MockerFixture,
 ) -> None:
@@ -660,7 +658,7 @@ def test_load_attached_within_tmux_detached(
 
 
 def test_load_append_windows_to_current_session(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Test tmuxp load when windows are appended to the current session."""

tests/cli/test_ls.py

@@ -1,15 +1,11 @@
 """CLI tests for tmuxp ls command."""
 
-from __future__ import annotations
-
 import contextlib
 import pathlib
-import typing as t
 
-from tmuxp import cli
+import pytest
 
-if t.TYPE_CHECKING:
-    import pytest
+from tmuxp import cli
 
 
 def test_ls_cli(

tests/workspace/test_builder.py

@@ -1,7 +1,5 @@
 """Test for tmuxp workspace builder."""
 
-from __future__ import annotations
-
 import functools
 import os
 import pathlib
@@ -15,6 +13,7 @@
 from libtmux.common import has_gte_version, has_lt_version
 from libtmux.exc import LibTmuxException
 from libtmux.pane import Pane
+from libtmux.server import Server
 from libtmux.session import Session
 from libtmux.test import retry_until, temp_session
 from libtmux.window import Window
@@ -28,7 +27,6 @@
 from tmuxp.workspace.builder import WorkspaceBuilder
 
 if t.TYPE_CHECKING:
-    from libtmux.server import Server
 
     class AssertCallbackProtocol(t.Protocol):
         """Assertion callback type protocol."""
@@ -483,7 +481,7 @@ def test_environment_variables_warns_prior_to_tmux_3_0(
 
 
 def test_automatic_rename_option(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Test workspace builder with automatic renaming enabled."""
@@ -1187,6 +1185,32 @@ def test_find_current_active_pane(
                 """
 session_name: Should execute
 windows:
+- panes:
+  - shell_command: echo "___$((1 + 3))___"
+  """,
+            ),
+            "___4___",
+            True,
+        ),
+        (
+            textwrap.dedent(
+                """
+session_name: Should not execute
+windows:
+- panes:
+  - shell_command:
+    - cmd: echo "___$((1 + 3))___"
+      enter: false
+    """,
+            ),
+            "___4___",
+            False,
+        ),
+        (
+            textwrap.dedent(
+                """
+session_name: Should execute
+windows:
 - panes:
   - shell_command: echo "___$((1 + 3))___"
   """,
@@ -1463,7 +1487,7 @@ class DefaultSizeNamespaceFixture(t.NamedTuple):
     test_id: str
 
     # test params
-    TMUXP_DEFAULT_SIZE: str | None
+    TMUXP_DEFAULT_SIZE: t.Optional[str]
     raises: bool
     confoverrides: dict[str, t.Any]
 
@@ -1497,10 +1521,10 @@ class DefaultSizeNamespaceFixture(t.NamedTuple):
 )
 @pytest.mark.skipif(has_lt_version("2.9"), reason="default-size only applies there")
 def test_issue_800_default_size_many_windows(
-    server: Server,
+    server: "Server",
     monkeypatch: pytest.MonkeyPatch,
     test_id: str,
-    TMUXP_DEFAULT_SIZE: str | None,
+    TMUXP_DEFAULT_SIZE: t.Optional[str],
     raises: bool,
     confoverrides: dict[str, t.Any],
 ) -> None:

tests/workspace/test_finder.py

@@ -1,7 +1,5 @@
 """Test config file searching for tmuxp."""
 
-from __future__ import annotations
-
 import argparse
 import pathlib
 import typing as t
@@ -269,7 +267,7 @@ def config_cmd(workspace_file: str) -> None:
 
     project_config = projectdir / ".tmuxp.yaml"
 
-    def check_cmd(config_arg: str) -> _pytest.capture.CaptureResult[str]:
+    def check_cmd(config_arg: str) -> "_pytest.capture.CaptureResult[str]":
         args = parser.parse_args([config_arg])
         config_cmd(workspace_file=args.workspace_file)
         return capsys.readouterr()