Migrate to inline-snapshot for snapshot files (#62)

Author: George-Ogden

.github/workflows/lint.yaml

@@ -10,4 +10,4 @@ on:
 
 jobs:
   lint:
-    uses: George-Ogden/actions/.github/workflows/lint.yaml@v4.0.0
+    uses: George-Ogden/actions/.github/workflows/lint.yaml@v4.0.1

.github/workflows/release.yaml

@@ -11,6 +11,6 @@ permissions:
 
 jobs:
   release:
-    uses: George-Ogden/actions/.github/workflows/python-release.yaml@v4.0.0
+    uses: George-Ogden/actions/.github/workflows/python-release.yaml@v4.0.1
     with:
       validation-code: "import mypy.version; from mypy_pytest_plugin import plugin; plugin(mypy.version.__version__)"

.github/workflows/test.yaml

@@ -14,15 +14,15 @@ jobs:
       matrix:
         shard: [0, 1, 2]
         total-shards: [3]
-    uses: George-Ogden/actions/.github/workflows/python-test.yaml@v4.0.0
+    uses: George-Ogden/actions/.github/workflows/python-test.yaml@v4.0.1
     with:
       python-versions: "['3.12', '3.13', '3.14']"
       timeout-minutes: 15
-      pytest-flags: -vv mypy_pytest_plugin -n auto --shard-id=${{ matrix.shard }} --num-shards=${{ matrix.total-shards }}
+      pytest-flags: -vv mypy_pytest_plugin -n auto --shard-id=${{ matrix.shard }} --num-shards=${{ matrix.total-shards }} --inline-snapshot=disable
 
   integration_tests:
-    uses: George-Ogden/actions/.github/workflows/python-test.yaml@v3.2.0
+    uses: George-Ogden/actions/.github/workflows/python-test.yaml@v4.0.1
     with:
       python-versions: "['3.12', '3.13', '3.14']"
       timeout-minutes: 10
-      pytest-flags: -vv tests -n auto
+      pytest-flags: -vv tests -n auto --inline-snapshot=disable

.mirror.lock

@@ -1,5 +1,5 @@
 # DANGER: EDIT AT YOUR OWN RISK. Track this file in version control so that others can sync files correctly.
-- commit: ca4110b072cfbd98e49a45ec2071e466d2f7f27f
+- commit: 160787740d5a4289276696fe07ebd6609580e10e
   files:
   - .github/python-release.yaml
   - .github/python-test.yaml

.pre-commit-config.yaml

@@ -1,5 +1,5 @@
 default_stages: [pre-commit]
-default_install_hook_types: [pre-commit, commit-msg]
+default_install_hook_types: [pre-commit, prepare-commit-msg, commit-msg]
 
 ci:
   skip:
@@ -36,7 +36,7 @@ repos:
         args: [--severity=warning]
 
   - repo: https://github.com/astral-sh/ruff-pre-commit/
-    rev: v0.15.0
+    rev: v0.15.2
     hooks:
       - id: ruff-check
         args: [--fix]
@@ -50,12 +50,17 @@ repos:
       - id: prettier
 
   - repo: https://github.com/rhysd/actionlint/
-    rev: v1.7.10
+    rev: v1.7.11
     hooks:
       - id: actionlint
 
+  - repo: https://github.com/codespell-project/codespell
+    rev: v2.4.1
+    hooks:
+      - id: codespell
+
   - repo: https://github.com/George-Ogden/pre-commit-hooks/
-    rev: v2.2.0
+    rev: v2.3.0
     hooks:
       - id: check-merge-conflict
       - id: dbg-check
@@ -68,3 +73,4 @@ repos:
         args: [-r, requirements-dev.txt, --show-traceback]
         exclude: test_samples/
       - id: spell-check-commit-msgs
+      - id: version-txt-auto-msg

mypy.ini

@@ -9,5 +9,5 @@ strict_equality = true
 exclude_gitignore = true
 plugins = plugin.py, mypy.plugins.proper_plugin
 
-[pytest_snapshot.*,xdist.*]
+[xdist.*]
 ignore_missing_imports = true

mypy_pytest_plugin/argnames_parser.py

@@ -65,7 +65,7 @@ def _check_duplicate_argnames(
 
     def _check_duplicate_argnames_sequence(
         self, argnames: list[str], context: Context
-    ) -> None | list[str]:
+    ) -> list[str] | None:
         argname_counts = Counter(argnames)
         duplicates = [argname for argname, count in argname_counts.items() if count > 1]
         if duplicates:

mypy_pytest_plugin/checker_wrapper.py

@@ -17,7 +17,7 @@ class CheckerWrapper(abc.ABC):
     @abc.abstractmethod
     def __init__(self) -> None: ...
 
-    def fail(self, msg: str, *, context: Context, code: ErrorCode, file: None | str = None) -> None:
+    def fail(self, msg: str, *, context: Context, code: ErrorCode, file: str | None = None) -> None:
         self.checker.msg.fail(msg, context=context, code=code, file=file)
 
     def note(self, msg: str, *, context: Context, code: ErrorCode | None) -> None:
@@ -55,7 +55,7 @@ def lookup_fullname[T](
         fullname: Fullname,
         *,
         context: Context | None = None,
-        predicate: None | Callable[[Any], TypeGuard[T]] = None,
+        predicate: Callable[[Any], TypeGuard[T]] | None = None,
     ) -> tuple[MypyFile, T] | None: ...
 
     @overload
@@ -64,15 +64,15 @@ def lookup_fullname(
         fullname: Fullname,
         *,
         context: Context | None = None,
-        predicate: None | Callable[[Any], bool] = None,
+        predicate: Callable[[Any], bool] | None = None,
     ) -> tuple[MypyFile, Any] | None: ...
 
     def lookup_fullname(
         self,
         fullname: Fullname,
         *,
         context: Context | None = None,
-        predicate: None | Callable[[Any], bool] = None,
+        predicate: Callable[[Any], bool] | None = None,
     ) -> tuple[MypyFile, Any] | None:
         module_name, target = (Fullname(()), fullname)
         while target:
@@ -92,10 +92,10 @@ def _lookup_fullname_in_module[T](
     ) -> T | None: ...
     @overload
     def _lookup_fullname_in_module(
-        self, module: MypyFile, target: Fullname, *, predicate: None | Callable[[Any], bool]
+        self, module: MypyFile, target: Fullname, *, predicate: Callable[[Any], bool] | None
     ) -> Any | None: ...
     def _lookup_fullname_in_module(
-        self, module: MypyFile, target: Fullname, *, predicate: None | Callable[[Any], bool]
+        self, module: MypyFile, target: Fullname, *, predicate: Callable[[Any], bool] | None
     ) -> Any | None:
         resource: Any = module
         for name in target:

mypy_pytest_plugin/fixture_manager_test.py

@@ -80,7 +80,6 @@ def test_fixture_manager_default_fixture_module_names() -> None:
             "_pytest.subtests",
             "_pytest.tmpdir",
             "inline_snapshot.pytest_plugin",
-            "pytest_snapshot.plugin",
             "xdist.plugin",
         ]
     )
@@ -321,12 +320,12 @@ def test_fixture_manager_resolve_fixtures_request_builtins() -> None:
     _fixture_manager_resolve_fixtures_test_body(
         [
             (
-                "pytest_snapshot.plugin",
+                "xdist.plugin",
                 """
                 import pytest
 
                 @pytest.fixture
-                def snapshot() -> None:
+                def worker() -> None:
                     ...
                 """,
             ),
@@ -345,12 +344,12 @@ def capsys() -> None:
                 """
                 from typing import Any
 
-                def test_request(capsys: Any, snapshot: Any) -> None:
+                def test_request(capsys: Any, worker: Any) -> None:
                     ...
                 """,
             ),
         ],
-        dict(capsys=["_pytest.capture.capsys"], snapshot=["pytest_snapshot.plugin.snapshot"]),
+        dict(capsys=["_pytest.capture.capsys"], worker=["xdist.plugin.worker"]),
     )
 
 
@@ -525,7 +524,7 @@ def _fixture_manager_resolve_autouse_fixtures_test_body(
                 isinstance(decorator := node.node, Decorator)
                 and (fixture := Fixture.from_decorator(decorator, checker)) is not None
             ):
-                overrides[(module_name, name)] = fixture.as_fixture_type(
+                overrides[module_name, name] = fixture.as_fixture_type(
                     decorator=decorator, checker=checker
                 )
         if autouse_node is not None:

mypy_pytest_plugin/object_patch_call_checker_test.py

@@ -66,7 +66,7 @@ def foo[T](target: Any, attribute: str, test: T, *, extra_arg: int = 0) -> int:
 
 
 def _attribute_type_test_body(
-    defs: str, attribute: str, *, errors: None | list[str] = None
+    defs: str, attribute: str, *, errors: list[str] | None = None
 ) -> None:
     parse_result = parse(defs)
     checker = parse_result.checker