chore: resolve linting errors in test modules

pyproject.toml

@@ -129,9 +129,15 @@ module = [
     "textual.*",
     "pyte.*",
     "libtmux.*",
+    "pytest.*",
 ]
 ignore_missing_imports = true
 
+# Relax strict rules for test files (pytest decorators are not fully typed)
+[[tool.mypy.overrides]]
+module = ["tests.*"]
+disallow_untyped_decorators = false
+
 # ============================================================================
 # Ruff Configuration (Fast Python Linter & Formatter)
 # ============================================================================

tests/conftest.py

@@ -1,10 +1,13 @@
 """Pytest configuration and shared fixtures for Strix tests."""
 
+from collections.abc import Callable
+from typing import Any
+
 import pytest
 
 
 @pytest.fixture
-def sample_function_with_types():
+def sample_function_with_types() -> Callable[..., None]:
     """Create a sample function with type annotations for testing argument conversion."""
 
     def func(
@@ -12,8 +15,8 @@ def sample_function_with_types():
         count: int,
         enabled: bool,
         ratio: float,
-        items: list,
+        items: list[Any],
-        config: dict,
+        config: dict[str, Any],
         optional: str | None = None,
     ) -> None:
         pass
@@ -22,10 +25,10 @@ def sample_function_with_types():
 
 
 @pytest.fixture
-def sample_function_no_annotations():
+def sample_function_no_annotations() -> Callable[..., None]:
     """Create a sample function without type annotations."""
 
-    def func(arg1, arg2, arg3):
+    def func(arg1: Any, arg2: Any, arg3: Any) -> None:
         pass
 
     return func

tests/test_argument_parser.py

@@ -5,8 +5,7 @@ string arguments to their appropriate Python types based on function
 type annotations.
 """
 
-import json
+from collections.abc import Callable
-from typing import Optional, Union
 
 import pytest
 
@@ -177,17 +176,17 @@ class TestConvertStringToType:
 
     def test_optional_type(self) -> None:
         """Test conversion with Optional type."""
-        result = convert_string_to_type("42", Optional[int])
+        result = convert_string_to_type("42", int | None)
         assert result == 42
 
     def test_union_type(self) -> None:
         """Test conversion with Union type."""
-        result = convert_string_to_type("42", Union[int, str])
+        result = convert_string_to_type("42", int | str)
         assert result == 42
 
     def test_union_type_with_none(self) -> None:
         """Test conversion with Union including None."""
-        result = convert_string_to_type("hello", Union[str, None])
+        result = convert_string_to_type("hello", str | None)
         assert result == "hello"
 
     def test_modern_union_syntax(self) -> None:
@@ -199,7 +198,9 @@ class TestConvertStringToType:
 class TestConvertArguments:
     """Tests for the convert_arguments function."""
 
-    def test_converts_typed_arguments(self, sample_function_with_types) -> None:
+    def test_converts_typed_arguments(
+        self, sample_function_with_types: Callable[..., None]
+    ) -> None:
         """Test that arguments are converted based on type annotations."""
         kwargs = {
             "name": "test",
@@ -218,32 +219,42 @@ class TestConvertArguments:
         assert result["items"] == [1, 2, 3]
         assert result["config"] == {"key": "value"}
 
-    def test_passes_through_none_values(self, sample_function_with_types) -> None:
+    def test_passes_through_none_values(
+        self, sample_function_with_types: Callable[..., None]
+    ) -> None:
         """Test that None values are passed through unchanged."""
         kwargs = {"name": "test", "count": None}
         result = convert_arguments(sample_function_with_types, kwargs)
         assert result["count"] is None
 
-    def test_passes_through_non_string_values(self, sample_function_with_types) -> None:
+    def test_passes_through_non_string_values(
+        self, sample_function_with_types: Callable[..., None]
+    ) -> None:
         """Test that non-string values are passed through unchanged."""
         kwargs = {"name": "test", "count": 42}
         result = convert_arguments(sample_function_with_types, kwargs)
         assert result["count"] == 42
 
-    def test_unknown_parameter_passed_through(self, sample_function_with_types) -> None:
+    def test_unknown_parameter_passed_through(
+        self, sample_function_with_types: Callable[..., None]
+    ) -> None:
         """Test that parameters not in signature are passed through."""
         kwargs = {"name": "test", "unknown_param": "value"}
         result = convert_arguments(sample_function_with_types, kwargs)
         assert result["unknown_param"] == "value"
 
-    def test_function_without_annotations(self, sample_function_no_annotations) -> None:
+    def test_function_without_annotations(
+        self, sample_function_no_annotations: Callable[..., None]
+    ) -> None:
         """Test handling of functions without type annotations."""
         kwargs = {"arg1": "value1", "arg2": "42"}
         result = convert_arguments(sample_function_no_annotations, kwargs)
         assert result["arg1"] == "value1"
         assert result["arg2"] == "42"
 
-    def test_raises_error_on_conversion_failure(self, sample_function_with_types) -> None:
+    def test_raises_error_on_conversion_failure(
+        self, sample_function_with_types: Callable[..., None]
+    ) -> None:
         """Test that ArgumentConversionError is raised on conversion failure."""
         kwargs = {"count": "not_a_number"}
         with pytest.raises(ArgumentConversionError) as exc_info: