tests(refactor): Use namespace imports for exception classes

why: Follow project import conventions from CLAUDE.md which
require namespace imports instead of direct class imports.
what:
- Change from `from libtmux.exc import X` to `from libtmux import exc`
- Update all usages to use `exc.X` pattern (e.g., exc.DeprecatedError)
- Reformat long lines to comply with 88 character limit

Author: tony

tests/legacy_api/test_common.py

@@ -13,6 +13,7 @@
 import pytest
 
 import libtmux
+from libtmux import exc
 from libtmux._compat import LooseVersion
 from libtmux.common import (
     TMUX_MAX_VERSION,
@@ -28,7 +29,6 @@
     session_check_name,
     tmux_cmd,
 )
-from libtmux.exc import BadSessionName, LibTmuxException, TmuxCommandNotFound
 
 if t.TYPE_CHECKING:
     from libtmux.session import Session
@@ -104,7 +104,7 @@ def mock_tmux_cmd(*args: t.Any, **kwargs: t.Any) -> Hi:
         return Hi()
 
     monkeypatch.setattr(libtmux.common, "tmux_cmd", mock_tmux_cmd)
-    with pytest.raises(LibTmuxException) as exc_info:
+    with pytest.raises(exc.LibTmuxException) as exc_info:
         get_version()
     exc_info.match("does not meet the minimum tmux version requirement")
 
@@ -139,11 +139,11 @@ def mock_get_version() -> LooseVersion:
         return LooseVersion("1.7")
 
     monkeypatch.setattr(libtmux.common, "get_version", mock_get_version)
-    with pytest.raises(LibTmuxException) as excinfo:
+    with pytest.raises(exc.LibTmuxException) as excinfo:
         has_minimum_version()
         excinfo.match(r"libtmux only supports")
 
-    with pytest.raises(LibTmuxException) as excinfo:
+    with pytest.raises(exc.LibTmuxException) as excinfo:
         has_minimum_version()
 
         excinfo.match(r"libtmux only supports")
@@ -195,7 +195,7 @@ def test_has_lte_version() -> None:
 def test_tmux_cmd_raises_on_not_found(monkeypatch: pytest.MonkeyPatch) -> None:
     """Verify raises if tmux command not found."""
     monkeypatch.setenv("PATH", "")
-    with pytest.raises(TmuxCommandNotFound):
+    with pytest.raises(exc.TmuxCommandNotFound):
         tmux_cmd("-V")
 
 
@@ -230,7 +230,7 @@ def test_session_check_name(
 ) -> None:
     """Verify session_check_name()."""
     if raises:
-        with pytest.raises(BadSessionName) as exc_info:
+        with pytest.raises(exc.BadSessionName) as exc_info:
             session_check_name(session_name)
         if exc_msg_regex is not None:
             assert exc_info.match(exc_msg_regex)

tests/legacy_api/test_pane.py

@@ -9,7 +9,7 @@
 
 import pytest
 
-from libtmux.exc import DeprecatedError
+from libtmux import exc
 
 if t.TYPE_CHECKING:
     from libtmux.session import Session
@@ -21,7 +21,9 @@ def test_resize_pane_raises_deprecated_error(session: Session) -> None:
     pane = window.active_pane
     assert pane is not None
 
-    with pytest.raises(DeprecatedError, match=r"Pane\.resize_pane\(\) was deprecated"):
+    with pytest.raises(
+        exc.DeprecatedError, match=r"Pane\.resize_pane\(\) was deprecated"
+    ):
         pane.resize_pane(height=4)
 
 
@@ -31,7 +33,9 @@ def test_select_pane_raises_deprecated_error(session: Session) -> None:
     pane = window.active_pane
     assert pane is not None
 
-    with pytest.raises(DeprecatedError, match=r"Pane\.select_pane\(\) was deprecated"):
+    with pytest.raises(
+        exc.DeprecatedError, match=r"Pane\.select_pane\(\) was deprecated"
+    ):
         pane.select_pane()
 
 
@@ -41,7 +45,9 @@ def test_split_window_raises_deprecated_error(session: Session) -> None:
     pane = window.active_pane
     assert pane is not None
 
-    with pytest.raises(DeprecatedError, match=r"Pane\.split_window\(\) was deprecated"):
+    with pytest.raises(
+        exc.DeprecatedError, match=r"Pane\.split_window\(\) was deprecated"
+    ):
         pane.split_window()
 
 
@@ -51,7 +57,7 @@ def test_pane_get_raises_deprecated_error(session: Session) -> None:
     pane = window.active_pane
     assert pane is not None
 
-    with pytest.raises(DeprecatedError, match=r"Pane\.get\(\) was deprecated"):
+    with pytest.raises(exc.DeprecatedError, match=r"Pane\.get\(\) was deprecated"):
         pane.get("pane_id")
 
 
@@ -61,5 +67,5 @@ def test_pane_getitem_raises_deprecated_error(session: Session) -> None:
     pane = window.active_pane
     assert pane is not None
 
-    with pytest.raises(DeprecatedError, match=r"Pane\[key\] lookup was deprecated"):
+    with pytest.raises(exc.DeprecatedError, match=r"Pane\[key\] lookup was deprecated"):
         _ = pane["pane_id"]

tests/legacy_api/test_server.py

@@ -1,97 +1,101 @@
 """Tests for deprecated libtmux Server APIs.
 
-These tests verify that deprecated methods raise DeprecatedError.
+These tests verify that deprecated methods raise exc.DeprecatedError.
 """
 
 from __future__ import annotations
 
 import pytest
 
-from libtmux.exc import DeprecatedError
+from libtmux import exc
 from libtmux.server import Server
 
 
 def test_kill_server_raises_deprecated_error(server: Server) -> None:
-    """Test Server.kill_server() raises DeprecatedError."""
+    """Test Server.kill_server() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Server\.kill_server\(\) was deprecated"
+        exc.DeprecatedError, match=r"Server\.kill_server\(\) was deprecated"
     ):
         server.kill_server()
 
 
 def test_server_get_by_id_raises_deprecated_error(server: Server) -> None:
-    """Test Server.get_by_id() raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Server\.get_by_id\(\) was deprecated"):
+    """Test Server.get_by_id() raises exc.DeprecatedError."""
+    with pytest.raises(
+        exc.DeprecatedError, match=r"Server\.get_by_id\(\) was deprecated"
+    ):
         server.get_by_id("$0")
 
 
 def test_server_where_raises_deprecated_error(server: Server) -> None:
-    """Test Server.where() raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Server\.where\(\) was deprecated"):
+    """Test Server.where() raises exc.DeprecatedError."""
+    with pytest.raises(exc.DeprecatedError, match=r"Server\.where\(\) was deprecated"):
         server.where({"session_name": "test"})
 
 
 def test_server_find_where_raises_deprecated_error(server: Server) -> None:
-    """Test Server.find_where() raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Server\.find_where\(\) was deprecated"):
+    """Test Server.find_where() raises exc.DeprecatedError."""
+    with pytest.raises(
+        exc.DeprecatedError, match=r"Server\.find_where\(\) was deprecated"
+    ):
         server.find_where({"session_name": "test"})
 
 
 def test_server_list_sessions_raises_deprecated_error(server: Server) -> None:
-    """Test Server.list_sessions() raises DeprecatedError."""
+    """Test Server.list_sessions() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Server\.list_sessions\(\) was deprecated"
+        exc.DeprecatedError, match=r"Server\.list_sessions\(\) was deprecated"
     ):
         server.list_sessions()
 
 
 def test_server_children_raises_deprecated_error(server: Server) -> None:
-    """Test Server.children raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Server\.children was deprecated"):
+    """Test Server.children raises exc.DeprecatedError."""
+    with pytest.raises(exc.DeprecatedError, match=r"Server\.children was deprecated"):
         _ = server.children
 
 
 def test_server__sessions_raises_deprecated_error(server: Server) -> None:
-    """Test Server._sessions raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Server\._sessions was deprecated"):
+    """Test Server._sessions raises exc.DeprecatedError."""
+    with pytest.raises(exc.DeprecatedError, match=r"Server\._sessions was deprecated"):
         _ = server._sessions
 
 
 def test_server__list_sessions_raises_deprecated_error(server: Server) -> None:
-    """Test Server._list_sessions() raises DeprecatedError."""
+    """Test Server._list_sessions() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Server\._list_sessions\(\) was deprecated"
+        exc.DeprecatedError, match=r"Server\._list_sessions\(\) was deprecated"
     ):
         server._list_sessions()
 
 
 def test_server__list_windows_raises_deprecated_error(server: Server) -> None:
-    """Test Server._list_windows() raises DeprecatedError."""
+    """Test Server._list_windows() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Server\._list_windows\(\) was deprecated"
+        exc.DeprecatedError, match=r"Server\._list_windows\(\) was deprecated"
     ):
         server._list_windows()
 
 
 def test_server__update_windows_raises_deprecated_error(server: Server) -> None:
-    """Test Server._update_windows() raises DeprecatedError."""
+    """Test Server._update_windows() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Server\._update_windows\(\) was deprecated"
+        exc.DeprecatedError, match=r"Server\._update_windows\(\) was deprecated"
     ):
         server._update_windows()
 
 
 def test_server__list_panes_raises_deprecated_error(server: Server) -> None:
-    """Test Server._list_panes() raises DeprecatedError."""
+    """Test Server._list_panes() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Server\._list_panes\(\) was deprecated"
+        exc.DeprecatedError, match=r"Server\._list_panes\(\) was deprecated"
     ):
         server._list_panes()
 
 
 def test_server__update_panes_raises_deprecated_error(server: Server) -> None:
-    """Test Server._update_panes() raises DeprecatedError."""
+    """Test Server._update_panes() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Server\._update_panes\(\) was deprecated"
+        exc.DeprecatedError, match=r"Server\._update_panes\(\) was deprecated"
     ):
         server._update_panes()

tests/legacy_api/test_session.py

@@ -1,6 +1,6 @@
 """Tests for deprecated libtmux Session APIs.
 
-These tests verify that deprecated methods raise DeprecatedError.
+These tests verify that deprecated methods raise exc.DeprecatedError.
 """
 
 from __future__ import annotations
@@ -9,42 +9,44 @@
 
 import pytest
 
-from libtmux.exc import DeprecatedError
+from libtmux import exc
 
 if t.TYPE_CHECKING:
     from libtmux.server import Server
     from libtmux.session import Session
 
 
 def test_attached_window_raises_deprecated_error(session: Session) -> None:
-    """Test Session.attached_window raises DeprecatedError."""
+    """Test Session.attached_window raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Session\.attached_window was deprecated"
+        exc.DeprecatedError, match=r"Session\.attached_window was deprecated"
     ):
         _ = session.attached_window
 
 
 def test_attached_pane_raises_deprecated_error(session: Session) -> None:
-    """Test Session.attached_pane raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Session\.attached_pane was deprecated"):
+    """Test Session.attached_pane raises exc.DeprecatedError."""
+    with pytest.raises(
+        exc.DeprecatedError, match=r"Session\.attached_pane was deprecated"
+    ):
         _ = session.attached_pane
 
 
 def test_attach_session_raises_deprecated_error(session: Session) -> None:
-    """Test Session.attach_session() raises DeprecatedError."""
+    """Test Session.attach_session() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Session\.attach_session\(\) was deprecated"
+        exc.DeprecatedError, match=r"Session\.attach_session\(\) was deprecated"
     ):
         session.attach_session()
 
 
 def test_kill_session_raises_deprecated_error(server: Server) -> None:
-    """Test Session.kill_session() raises DeprecatedError."""
+    """Test Session.kill_session() raises exc.DeprecatedError."""
     # Create a new session to kill (so we don't kill our test session)
     new_session = server.new_session(session_name="test_kill_session", detach=True)
 
     with pytest.raises(
-        DeprecatedError, match=r"Session\.kill_session\(\) was deprecated"
+        exc.DeprecatedError, match=r"Session\.kill_session\(\) was deprecated"
     ):
         new_session.kill_session()
 
@@ -53,60 +55,64 @@ def test_kill_session_raises_deprecated_error(server: Server) -> None:
 
 
 def test_session_get_raises_deprecated_error(session: Session) -> None:
-    """Test Session.get() raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Session\.get\(\) was deprecated"):
+    """Test Session.get() raises exc.DeprecatedError."""
+    with pytest.raises(exc.DeprecatedError, match=r"Session\.get\(\) was deprecated"):
         session.get("session_name")
 
 
 def test_session_getitem_raises_deprecated_error(session: Session) -> None:
-    """Test Session.__getitem__() raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Session\[key\] lookup was deprecated"):
+    """Test Session.__getitem__() raises exc.DeprecatedError."""
+    with pytest.raises(
+        exc.DeprecatedError, match=r"Session\[key\] lookup was deprecated"
+    ):
         _ = session["session_name"]
 
 
 def test_session_get_by_id_raises_deprecated_error(session: Session) -> None:
-    """Test Session.get_by_id() raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Session\.get_by_id\(\) was deprecated"):
+    """Test Session.get_by_id() raises exc.DeprecatedError."""
+    with pytest.raises(
+        exc.DeprecatedError, match=r"Session\.get_by_id\(\) was deprecated"
+    ):
         session.get_by_id("@0")
 
 
 def test_session_where_raises_deprecated_error(session: Session) -> None:
-    """Test Session.where() raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Session\.where\(\) was deprecated"):
+    """Test Session.where() raises exc.DeprecatedError."""
+    with pytest.raises(exc.DeprecatedError, match=r"Session\.where\(\) was deprecated"):
         session.where({"window_name": "test"})
 
 
 def test_session_find_where_raises_deprecated_error(session: Session) -> None:
-    """Test Session.find_where() raises DeprecatedError."""
+    """Test Session.find_where() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Session\.find_where\(\) was deprecated"
+        exc.DeprecatedError, match=r"Session\.find_where\(\) was deprecated"
     ):
         session.find_where({"window_name": "test"})
 
 
 def test_session_list_windows_raises_deprecated_error(session: Session) -> None:
-    """Test Session.list_windows() raises DeprecatedError."""
+    """Test Session.list_windows() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Session\.list_windows\(\) was deprecated"
+        exc.DeprecatedError, match=r"Session\.list_windows\(\) was deprecated"
     ):
         session.list_windows()
 
 
 def test_session_children_raises_deprecated_error(session: Session) -> None:
-    """Test Session.children raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Session\.children was deprecated"):
+    """Test Session.children raises exc.DeprecatedError."""
+    with pytest.raises(exc.DeprecatedError, match=r"Session\.children was deprecated"):
         _ = session.children
 
 
 def test_session__windows_raises_deprecated_error(session: Session) -> None:
-    """Test Session._windows raises DeprecatedError."""
-    with pytest.raises(DeprecatedError, match=r"Session\._windows was deprecated"):
+    """Test Session._windows raises exc.DeprecatedError."""
+    with pytest.raises(exc.DeprecatedError, match=r"Session\._windows was deprecated"):
         _ = session._windows
 
 
 def test_session__list_windows_raises_deprecated_error(session: Session) -> None:
-    """Test Session._list_windows() raises DeprecatedError."""
+    """Test Session._list_windows() raises exc.DeprecatedError."""
     with pytest.raises(
-        DeprecatedError, match=r"Session\._list_windows\(\) was deprecated"
+        exc.DeprecatedError, match=r"Session\._list_windows\(\) was deprecated"
     ):
         session._list_windows()