refactor!(ruff): Run all automated fixes as of ruff 0.3.4

ruff check --select ALL . --fix --unsafe-fixes --preview --show-fixes; ruff format .

Fixed 144 errors:
- conftest.py:
     2 × I001 (unsorted-imports)
     2 × PT003 (pytest-extraneous-scope-function)
     1 × PIE790 (unnecessary-placeholder)
     1 × TCH005 (empty-type-checking-block)
     1 × TCH004 (runtime-import-in-type-checking-block)
- docs/conf.py:
     1 × RET505 (superfluous-else-return)
- src/libtmux/_internal/query_list.py:
     3 × RSE102 (unnecessary-paren-on-raise-exception)
     2 × ANN204 (missing-return-type-special-method)
     1 × T201 (print)
     1 × PLR5501 (collapsible-else-if)
     1 × RET505 (superfluous-else-return)
     1 × RET506 (superfluous-else-raise)
     1 × F841 (unused-variable)
- src/libtmux/_vendor/version.py:
     2 × PLR6201 (literal-membership)
     1 × ANN204 (missing-return-type-special-method)
     1 × RET505 (superfluous-else-return)
     1 × RUF022 (unsorted-dunder-all)
- src/libtmux/common.py:
     6 × RET506 (superfluous-else-raise)
     1 × RSE102 (unnecessary-paren-on-raise-exception)
     1 × TCH005 (empty-type-checking-block)
     1 × I001 (unsorted-imports)
- src/libtmux/exc.py:
     8 × ANN204 (missing-return-type-special-method)
     2 × PIE790 (unnecessary-placeholder)
     2 × COM812 (missing-trailing-comma)
     2 × E303 (too-many-blank-lines)
- src/libtmux/neo.py:
     1 × E303 (too-many-blank-lines)
     1 × RET504 (unnecessary-assign)
- src/libtmux/pane.py:
     3 × E266 (multiple-leading-hashes-for-block-comment)
     2 × COM812 (missing-trailing-comma)
     1 × RSE102 (unnecessary-paren-on-raise-exception)
     1 × E303 (too-many-blank-lines)
     1 × PLR1711 (useless-return)
- src/libtmux/pytest_plugin.py:
     3 × PT003 (pytest-extraneous-scope-function)
     1 × PT001 (pytest-fixture-incorrect-parentheses-style)
- src/libtmux/server.py:
     2 × ISC001 (single-line-implicit-string-concatenation)
     2 × RSE102 (unnecessary-paren-on-raise-exception)
     2 × COM812 (missing-trailing-comma)
     1 × RET505 (superfluous-else-return)
     1 × SIM103 (needless-bool)
- src/libtmux/session.py:
     5 × COM812 (missing-trailing-comma)
     2 × RSE102 (unnecessary-paren-on-raise-exception)
     1 × RET506 (superfluous-else-raise)
     1 × RET505 (superfluous-else-return)
     1 × RET503 (implicit-return)
     1 × PLR1711 (useless-return)
     1 × E303 (too-many-blank-lines)
- src/libtmux/test.py:
     1 × RET506 (superfluous-else-raise)
     1 × RSE102 (unnecessary-paren-on-raise-exception)
- src/libtmux/window.py:
     4 × COM812 (missing-trailing-comma)
     2 × E266 (multiple-leading-hashes-for-block-comment)
     1 × RSE102 (unnecessary-paren-on-raise-exception)
     1 × E303 (too-many-blank-lines)
     1 × PLR6201 (literal-membership)
     1 × PLR1711 (useless-return)
- tests/_internal/test_query_list.py:
    38 × PT007 (pytest-parametrize-values-wrong-type)
     1 × PT006 (pytest-parametrize-names-wrong-type)
- tests/legacy_api/test_session.py:
     1 × PT006 (pytest-parametrize-names-wrong-type)
- tests/legacy_api/test_version.py:
     1 × PT014 (pytest-duplicate-parametrize-test-cases)
- tests/legacy_api/test_window.py:
     1 × COM812 (missing-trailing-comma)
     1 × PT006 (pytest-parametrize-names-wrong-type)
- tests/test_dataclasses.py:
     1 × COM812 (missing-trailing-comma)
     1 × PT001 (pytest-fixture-incorrect-parentheses-style)
- tests/test_session.py:
     2 × COM812 (missing-trailing-comma)
- tests/test_version.py:
     1 × PT014 (pytest-duplicate-parametrize-test-cases)
- tests/test_window.py:
     3 × COM812 (missing-trailing-comma)
     1 × PT006 (pytest-parametrize-names-wrong-type)

Author: tony

conftest.py

@@ -21,9 +21,6 @@
 from libtmux.session import Session
 from libtmux.window import Window
 
-if t.TYPE_CHECKING:
-    from libtmux.session import Session
-
 pytest_plugins = ["pytester"]
 
 
@@ -47,7 +44,7 @@ def add_doctest_fixtures(
         doctest_namespace["request"] = request
 
 
-@pytest.fixture(autouse=True, scope="function")
+@pytest.fixture(autouse=True)
 def set_home(
     monkeypatch: pytest.MonkeyPatch,
     user_path: pathlib.Path,
@@ -56,12 +53,11 @@ def set_home(
     monkeypatch.setenv("HOME", str(user_path))
 
 
-@pytest.fixture(autouse=True, scope="function")
+@pytest.fixture(autouse=True)
 def setup_fn(
     clear_env: None,
 ) -> None:
     """Function-level test configuration fixtures for pytest."""
-    pass
 
 
 @pytest.fixture(autouse=True, scope="session")

docs/conf.py

@@ -197,15 +197,14 @@ def linkcode_resolve(domain: str, info: t.Dict[str, str]) -> t.Union[None, str]:
             fn,
             linespec,
         )
-    else:
-        return "{}/blob/v{}/{}/{}/{}{}".format(
-            about["__github__"],
-            about["__version__"],
-            "src",
-            about["__package_name__"],
-            fn,
-            linespec,
-        )
+    return "{}/blob/v{}/{}/{}/{}{}".format(
+        about["__github__"],
+        about["__version__"],
+        "src",
+        about["__package_name__"],
+        fn,
+        linespec,
+    )
 
 
 def remove_tabs_js(app: "Sphinx", exc: Exception) -> None:

src/libtmux/_internal/query_list.py

@@ -100,9 +100,8 @@ def keygetter(
             elif hasattr(dct, sub_field):
                 dct = getattr(dct, sub_field)
 
-    except Exception as e:
+    except Exception:
         traceback.print_stack()
-        print(f"Above error was {e}")
         return None
 
     return dct
@@ -307,12 +306,12 @@ def lookup_iregex(
 
 
 class PKRequiredException(Exception):
-    def __init__(self, *args: object):
+    def __init__(self, *args: object) -> None:
         return super().__init__("items() require a pk_key exists")
 
 
 class OpNotFound(ValueError):
-    def __init__(self, op: str, *args: object):
+    def __init__(self, op: str, *args: object) -> None:
         return super().__init__(f"{op} not in LOOKUP_NAME_MAP")
 
 
@@ -473,7 +472,7 @@ def __init__(self, items: t.Optional["Iterable[T]"] = None) -> None:
 
     def items(self) -> t.List[t.Tuple[str, T]]:
         if self.pk_key is None:
-            raise PKRequiredException()
+            raise PKRequiredException
         return [(getattr(item, self.pk_key), item) for item in self]
 
     def __eq__(
@@ -493,9 +492,8 @@ def __eq__(
                         for key in a_keys:
                             if abs(a[key] - b[key]) > 1:
                                 return False
-                else:
-                    if a != b:
-                        return False
+                elif a != b:
+                    return False
 
             return True
         return False
@@ -534,8 +532,7 @@ def filter_lookup(obj: t.Any) -> bool:
             def val_match(obj: t.Union[str, t.List[t.Any], T]) -> bool:
                 if isinstance(matcher, list):
                     return obj in matcher
-                else:
-                    return bool(obj == matcher)
+                return bool(obj == matcher)
 
             _filter = val_match
         else:
@@ -557,9 +554,9 @@ def get(
         """
         objs = self.filter(matcher=matcher, **kwargs)
         if len(objs) > 1:
-            raise MultipleObjectsReturned()
-        elif len(objs) == 0:
+            raise MultipleObjectsReturned
+        if len(objs) == 0:
             if default == no_arg:
-                raise ObjectDoesNotExist()
+                raise ObjectDoesNotExist
             return default
         return objs[0]

src/libtmux/_vendor/version.py

@@ -16,7 +16,7 @@
 
 from ._structures import Infinity, InfinityType, NegativeInfinity, NegativeInfinityType
 
-__all__ = ["VERSION_PATTERN", "parse", "Version", "InvalidVersion"]
+__all__ = ["VERSION_PATTERN", "InvalidVersion", "Version", "parse"]
 
 InfiniteTypes = Union[InfinityType, NegativeInfinityType]
 PrePostDevType = Union[InfiniteTypes, Tuple[str, int]]
@@ -78,7 +78,7 @@ class InvalidVersion(ValueError):
     libtmux._vendor.version.InvalidVersion: Invalid version: 'invalid'
     """
 
-    def __init__(self, version: str, *args: object):
+    def __init__(self, version: str, *args: object) -> None:
         return super().__init__(f"Invalid version: '{version}'")
 
 
@@ -362,8 +362,7 @@ def local(self) -> Optional[str]:
         """
         if self._version.local:
             return ".".join(str(x) for x in self._version.local)
-        else:
-            return None
+        return None
 
     @property
     def public(self) -> str:
@@ -494,9 +493,9 @@ def _parse_letter_version(
             letter = "a"
         elif letter == "beta":
             letter = "b"
-        elif letter in ["c", "pre", "preview"]:
+        elif letter in {"c", "pre", "preview"}:
             letter = "rc"
-        elif letter in ["rev", "r"]:
+        elif letter in {"rev", "r"}:
             letter = "post"
 
         return letter, int(number)

src/libtmux/common.py

@@ -16,10 +16,6 @@
 from . import exc
 from ._compat import LooseVersion, console_to_str, str_from_console
 
-if t.TYPE_CHECKING:
-    pass
-
-
 logger = logging.getLogger(__name__)
 
 
@@ -219,7 +215,7 @@ class tmux_cmd:
     def __init__(self, *args: t.Any) -> None:
         tmux_bin = shutil.which("tmux")
         if not tmux_bin:
-            raise exc.TmuxCommandNotFound()
+            raise exc.TmuxCommandNotFound
 
         cmd = [tmux_bin]
         cmd += args  # add the command arguments to cmd
@@ -416,8 +412,7 @@ def has_minimum_version(raises: bool = True) -> bool:
                 + "libtmux."
             )
             raise exc.VersionTooLow(msg)
-        else:
-            return False
+        return False
     return True
 
 
@@ -439,9 +434,9 @@ def session_check_name(session_name: t.Optional[str]) -> None:
     """
     if session_name is None or len(session_name) == 0:
         raise exc.BadSessionName(reason="empty", session_name=session_name)
-    elif "." in session_name:
+    if "." in session_name:
         raise exc.BadSessionName(reason="contains periods", session_name=session_name)
-    elif ":" in session_name:
+    if ":" in session_name:
         raise exc.BadSessionName(reason="contains colons", session_name=session_name)
 
 
@@ -474,12 +469,11 @@ def handle_option_error(error: str) -> t.Type[exc.OptionError]:
     """
     if "unknown option" in error:
         raise exc.UnknownOption(error)
-    elif "invalid option" in error:
+    if "invalid option" in error:
         raise exc.InvalidOption(error)
-    elif "ambiguous option" in error:
+    if "ambiguous option" in error:
         raise exc.AmbiguousOption(error)
-    else:
-        raise exc.OptionError(error)  # Raise generic option error
+    raise exc.OptionError(error)  # Raise generic option error
 
 
 def get_libtmux_version() -> LooseVersion:

src/libtmux/exc.py

@@ -35,7 +35,7 @@ def __init__(
         list_cmd: t.Optional[str] = None,
         list_extra_args: "t.Optional[ListExtraArgs]" = None,
         *args: object,
-    ):
+    ) -> None:
         if all(arg is not None for arg in [obj_key, obj_id, list_cmd, list_extra_args]):
             return super().__init__(
                 f"Could not find {obj_key}={obj_id} for {list_cmd} "
@@ -56,7 +56,7 @@ def __init__(
         reason: str,
         session_name: t.Optional[str] = None,
         *args: object,
-    ):
+    ) -> None:
         msg = f"Bad session name: {reason}"
         if session_name is not None:
             msg += f" (session name: {session_name})"
@@ -74,7 +74,7 @@ class UnknownOption(OptionError):
 class UnknownColorOption(UnknownOption):
     """Unknown color option."""
 
-    def __init__(self, *args: object):
+    def __init__(self, *args: object) -> None:
         return super().__init__("Server.colors must equal 88 or 256")
 
 
@@ -93,7 +93,7 @@ class WaitTimeout(LibTmuxException):
 class VariableUnpackingError(LibTmuxException):
     """Error unpacking variable."""
 
-    def __init__(self, variable: t.Optional[t.Any] = None, *args: object):
+    def __init__(self, variable: t.Optional[t.Any] = None, *args: object) -> None:
         return super().__init__(f"Unexpected variable: {variable!s}")
 
 
@@ -104,7 +104,7 @@ class PaneError(LibTmuxException):
 class PaneNotFound(PaneError):
     """Pane not found."""
 
-    def __init__(self, pane_id: t.Optional[str] = None, *args: object):
+    def __init__(self, pane_id: t.Optional[str] = None, *args: object) -> None:
         if pane_id is not None:
             return super().__init__(f"Pane not found: {pane_id}")
         return super().__init__("Pane not found")
@@ -117,21 +117,21 @@ class WindowError(LibTmuxException):
 class MultipleActiveWindows(WindowError):
     """Multiple active windows."""
 
-    def __init__(self, count: int, *args: object):
+    def __init__(self, count: int, *args: object) -> None:
         return super().__init__(f"Multiple active windows: {count} found")
 
 
 class NoActiveWindow(WindowError):
     """No active window found."""
 
-    def __init__(self, *args: object):
+    def __init__(self, *args: object) -> None:
         return super().__init__("No active windows found")
 
 
 class NoWindowsExist(WindowError):
     """No windows exist for object."""
 
-    def __init__(self, *args: object):
+    def __init__(self, *args: object) -> None:
         return super().__init__("No windows exist for object")
 
 
@@ -143,20 +143,18 @@ def __init__(self) -> None:
 
 
 class WindowAdjustmentDirectionRequiresAdjustment(
-    WindowError, AdjustmentDirectionRequiresAdjustment
+    WindowError,
+    AdjustmentDirectionRequiresAdjustment,
 ):
     """ValueError for :meth:`libtmux.Window.resize_window`."""
 
-    pass
-
 
 class PaneAdjustmentDirectionRequiresAdjustment(
-    WindowError, AdjustmentDirectionRequiresAdjustment
+    WindowError,
+    AdjustmentDirectionRequiresAdjustment,
 ):
     """ValueError for :meth:`libtmux.Pane.resize_pane`."""
 
-    pass
-
 
 class RequiresDigitOrPercentage(LibTmuxException, ValueError):
     """Requires digit (int or str digit) or a percentage."""

src/libtmux/neo.py

@@ -226,11 +226,7 @@ def fetch_objs(
     ]
 
     # Filter empty values
-    obj_formatters_filtered = [
-        {k: v for k, v in formatter.items() if v} for formatter in obj_formatters
-    ]
-
-    return obj_formatters_filtered
+    return [{k: v for k, v in formatter.items() if v} for formatter in obj_formatters]
 
 
 def fetch_obj(

src/libtmux/pane.py

@@ -215,16 +215,16 @@ def resize(
         """
         tmux_args: t.Tuple[str, ...] = ()
 
-        ## Adjustments
+        # Adjustments
         if adjustment_direction:
             if adjustment is None:
-                raise exc.PaneAdjustmentDirectionRequiresAdjustment()
+                raise exc.PaneAdjustmentDirectionRequiresAdjustment
             tmux_args += (
                 f"{RESIZE_ADJUSTMENT_DIRECTION_FLAG_MAP[adjustment_direction]}",
                 str(adjustment),
             )
         elif height or width:
-            ## Manual resizing
+            # Manual resizing
             if height:
                 if isinstance(height, str):
                     if height.endswith("%") and not has_gte_version("3.1"):
@@ -242,7 +242,7 @@ def resize(
 
                 tmux_args += (f"-x{width}",)
         elif zoom:
-            ## Zoom / Unzoom
+            # Zoom / Unzoom
             tmux_args += ("-Z",)
         elif mouse:
             tmux_args += ("-M",)
@@ -436,8 +436,6 @@ def kill(
         if proc.stderr:
             raise exc.LibTmuxException(proc.stderr)
 
-        return None
-
     """
     Commands ("climber"-helpers)
 
@@ -650,7 +648,9 @@ def split(
                 raise exc.LibTmuxException(pane_cmd.stderr)
 
             raise exc.LibTmuxException(
-                pane_cmd.stderr, self.__dict__, self.window.panes
+                pane_cmd.stderr,
+                self.__dict__,
+                self.window.panes,
             )
 
         pane_output = pane_cmd.stdout[0]
@@ -877,7 +877,9 @@ def get(self, key: str, default: t.Optional[t.Any] = None) -> t.Any:
 
         """
         warnings.warn(
-            "Pane.get() is deprecated", category=DeprecationWarning, stacklevel=2
+            "Pane.get() is deprecated",
+            category=DeprecationWarning,
+            stacklevel=2,
         )
         return getattr(self, key, default)