sarj-python-lint 1.29.0__tar.gz → 1.29.1__tar.gz

{sarj_python_lint-1.29.0 → sarj_python_lint-1.29.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: sarj-python-lint
-Version: 1.29.0
+Version: 1.29.1
 Summary: Custom Python lint rules — AST-based, pre-commit-friendly, hypermodern defaults
 Project-URL: Homepage, https://code-standards.sarj.ai/rules/python/
 Project-URL: Documentation, https://code-standards.sarj.ai/rules/python/

{sarj_python_lint-1.29.0 → sarj_python_lint-1.29.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "sarj-python-lint"
-version = "1.29.0"
+version = "1.29.1"
 description = "Custom Python lint rules — AST-based, pre-commit-friendly, hypermodern defaults"
 readme = "README.md"
 authors = [{ name = "sarj-ai" }]

sarj_python_lint-1.29.1/src/sarj_python_lint/rules/_exports.py

@@ -0,0 +1,27 @@
+from __future__ import annotations
+
+import ast
+
+from sarj_python_lint.rules._ast_index import walk
+
+
+def mentions_dunder_all(node: ast.AST) -> bool:
+    for child in walk(node):
+        match child:
+            case ast.Name(id="__all__"):
+                return True
+            case ast.alias() if (child.asname or child.name).split(".")[0] == "__all__":
+                return True
+            case ast.FunctionDef() | ast.AsyncFunctionDef() | ast.ClassDef() if child.name == "__all__":
+                return True
+            case ast.arg(arg="__all__") | ast.ExceptHandler(name="__all__"):
+                return True
+            case ast.MatchAs() | ast.MatchStar() if child.name == "__all__":
+                return True
+            case ast.MatchMapping(rest="__all__"):
+                return True
+            case ast.Global() | ast.Nonlocal() if "__all__" in child.names:
+                return True
+            case _:
+                pass
+    return False

{sarj_python_lint-1.29.0 → sarj_python_lint-1.29.1}/src/sarj_python_lint/rules/_pytest.py

@@ -1,10 +1,26 @@
 from __future__ import annotations
 
 import ast
+from typing import TYPE_CHECKING
 
 from sarj_python_lint.rules._ast_index import walk
 
 
+if TYPE_CHECKING:
+    from collections.abc import Iterator
+
+
+def collected_tests(statements: list[ast.stmt]) -> Iterator[ast.FunctionDef | ast.AsyncFunctionDef]:
+    for statement in statements:
+        match statement:
+            case ast.FunctionDef() | ast.AsyncFunctionDef() if statement.name.startswith("test"):
+                yield statement
+            case ast.ClassDef() if statement.name.startswith("Test"):
+                yield from collected_tests(statement.body)
+            case _:
+                pass
+
+
 # The pytest-benchmark fixture: the test measures time, it does not verify.
 _BENCHMARK = "benchmark"
 

{sarj_python_lint-1.29.0 → sarj_python_lint-1.29.1}/src/sarj_python_lint/rules/async_mock_call_without_await_assertion.py

@@ -20,6 +20,7 @@ from sarj_python_lint.rule_base import (
 )
 from sarj_python_lint.rules._ast_index import walk as walk_ast
 from sarj_python_lint.rules._imports import ImportIndex
+from sarj_python_lint.rules._pytest import collected_tests
 
 
 if TYPE_CHECKING:
@@ -224,24 +225,13 @@ class AsyncMockCallWithoutAwaitAssertion(Rule):
                 severity=Severity.WARNING,
                 message="AsyncMock call assertions do not prove awaiting; keep this assertion and add an await assertion",
             )
-            for test in _collected_tests(tree.body)
+            for test in collected_tests(tree.body)
             if not _is_pytest_fixture(test, imports)
             for call in _uncovered_calls(_test_imports(imports, test), test, writes)
             if not is_suppressed(lines, call.lineno, self.code)
         ]
 
 
-def _collected_tests(statements: list[ast.stmt]) -> Iterator[_TestFunction]:
-    for statement in statements:
-        match statement:
-            case ast.FunctionDef() | ast.AsyncFunctionDef() if statement.name.startswith("test"):
-                yield statement
-            case ast.ClassDef() if statement.name.startswith("Test"):
-                yield from _collected_tests(statement.body)
-            case _:
-                pass
-
-
 def _is_pytest_fixture(test: _TestFunction, imports: ImportIndex) -> bool:
     return any(
         imports.resolved_qualified_name(decorator.func if isinstance(decorator, ast.Call) else decorator)

{sarj_python_lint-1.29.0 → sarj_python_lint-1.29.1}/src/sarj_python_lint/rules/no_duplicate_dunder_all_entry.py

@@ -14,7 +14,7 @@ from sarj_python_lint.rule_base import (
     RuleDocumentation,
     RuleExample,
 )
-from sarj_python_lint.rules._ast_index import walk as walk_ast
+from sarj_python_lint.rules._exports import mentions_dunder_all
 
 
 if TYPE_CHECKING:
@@ -140,7 +140,7 @@ def _has_other_dunder_all_rebindings(tree: ast.Module, declaration: ast.stmt) ->
             continue
         if _is_supported_dunder_all_growth(statement):
             continue
-        if _mentions_dunder_all(statement):
+        if mentions_dunder_all(statement):
             return True
     return False
 
@@ -155,28 +155,6 @@ def _is_supported_dunder_all_growth(statement: ast.stmt) -> bool:
             return False
 
 
-def _mentions_dunder_all(node: ast.AST) -> bool:
-    for child in walk_ast(node):
-        match child:
-            case ast.Name(id="__all__"):
-                return True
-            case ast.alias() if (child.asname or child.name).split(".")[0] == "__all__":
-                return True
-            case ast.FunctionDef() | ast.AsyncFunctionDef() | ast.ClassDef() if child.name == "__all__":
-                return True
-            case ast.arg(arg="__all__") | ast.ExceptHandler(name="__all__"):
-                return True
-            case ast.MatchAs() | ast.MatchStar() if child.name == "__all__":
-                return True
-            case ast.MatchMapping(rest="__all__"):
-                return True
-            case ast.Global() | ast.Nonlocal() if "__all__" in child.names:
-                return True
-            case _:
-                pass
-    return False
-
-
 def _literal_elements(statement: ast.stmt) -> list[_LiteralElement] | None:
     value: ast.expr | None
     match statement:

{sarj_python_lint-1.29.0 → sarj_python_lint-1.29.1}/src/sarj_python_lint/rules/no_fixed_sleep_before_assert.py

@@ -17,6 +17,7 @@ from sarj_python_lint.rule_base import (
     RuleExample,
     Severity,
 )
+from sarj_python_lint.rules._pytest import collected_tests
 
 
 if TYPE_CHECKING:
@@ -144,7 +145,7 @@ class NoFixedSleepBeforeAssert(Rule):
         imports = context.imports
         diagnostics = [
             Diagnostic(path=path, line=statement.lineno, col=statement.col_offset + 1, code=self.code, message=_MESSAGE)
-            for test in _collected_tests(tree.body)
+            for test in collected_tests(tree.body)
             for statement, following in _adjacent_statements(test.body)
             if isinstance(following, ast.Assert) and _is_fixed_sleep(statement, test, imports)
         ]
@@ -155,17 +156,6 @@ def _is_collected_test_module(path: Path) -> bool:
     return path.suffix == ".py" and (path.stem.startswith("test_") or path.stem.endswith("_test"))
 
 
-def _collected_tests(statements: list[ast.stmt]) -> Iterator[_TestFunction]:
-    for statement in statements:
-        match statement:
-            case ast.FunctionDef() | ast.AsyncFunctionDef() if statement.name.startswith("test"):
-                yield statement
-            case ast.ClassDef() if statement.name.startswith("Test"):
-                yield from _collected_tests(statement.body)
-            case _:
-                pass
-
-
 def _adjacent_statements(block: list[ast.stmt]) -> Iterator[tuple[ast.stmt, ast.stmt]]:
     yield from pairwise(block)
     for statement in block:

{sarj_python_lint-1.29.0 → sarj_python_lint-1.29.1}/src/sarj_python_lint/rules/no_generic_single_export_module.py

@@ -15,6 +15,7 @@ from sarj_python_lint.rule_base import (
     RuleExample,
 )
 from sarj_python_lint.rules._ast_index import walk as walk_ast
+from sarj_python_lint.rules._exports import mentions_dunder_all
 
 
 if TYPE_CHECKING:
@@ -145,7 +146,7 @@ def _has_additional_public_export(tree: ast.Module, primary_name: str) -> bool:
 
 
 def _dunder_all_matches_primary(tree: ast.Module, primary_name: str) -> bool:
-    owners = [statement for statement in tree.body if _mentions_dunder_all(statement)]
+    owners = [statement for statement in tree.body if mentions_dunder_all(statement)]
     if not owners:
         return True
     if len(owners) != 1:
@@ -163,28 +164,6 @@ def _dunder_all_matches_primary(tree: ast.Module, primary_name: str) -> bool:
     return isinstance(entry, ast.Constant) and entry.value == primary_name
 
 
-def _mentions_dunder_all(node: ast.AST) -> bool:
-    for child in walk_ast(node):
-        match child:
-            case ast.Name(id="__all__"):
-                return True
-            case ast.alias() if (child.asname or child.name).split(".")[0] == "__all__":
-                return True
-            case ast.FunctionDef() | ast.AsyncFunctionDef() | ast.ClassDef() if child.name == "__all__":
-                return True
-            case ast.arg(arg="__all__") | ast.ExceptHandler(name="__all__"):
-                return True
-            case ast.MatchAs() | ast.MatchStar() if child.name == "__all__":
-                return True
-            case ast.MatchMapping(rest="__all__"):
-                return True
-            case ast.Global() | ast.Nonlocal() if "__all__" in child.names:
-                return True
-            case _:
-                pass
-    return False
-
-
 def _annotation_name(node: ast.expr) -> str:
     match node:
         case ast.Name(id=name) | ast.Attribute(attr=name):