sarj-python-lint 0.67.4__tar.gz → 0.67.6__tar.gz

{sarj_python_lint-0.67.4 → sarj_python_lint-0.67.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sarj-python-lint
-Version: 0.67.4
+Version: 0.67.6
 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-0.67.4 → sarj_python_lint-0.67.6}/pyproject.toml

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

{sarj_python_lint-0.67.4 → sarj_python_lint-0.67.6}/src/sarj_python_lint/rules/kwarg_heavy_construction_in_test.py

@@ -65,7 +65,18 @@ class KwargHeavyConstructionInTest(Rule):
                 files=(
                     ExampleFile.python(
                         "tests/test_call.py",
-                        "def test_first():\n    assert Call(a=1, b=2, c=3, d=4, e=5, f=6, g=7, h=8, i=9)\n\ndef test_second():\n    assert Call(a=1, b=2, c=3, d=4, e=5, f=6, g=7, h=8, i=9)\n",
+                        "def test_style():\n"
+                        "    style = Style(\n"
+                        '        color="red", bgcolor="black", bold=True, dim=True, italic=True,\n'
+                        "        underline=True, blink=True, blink2=True, reverse=True,\n"
+                        "    )\n"
+                        "    assert str(style)\n\n"
+                        "def test_style_again():\n"
+                        "    other = Style(\n"
+                        '        color="blue", bgcolor="white", bold=False, dim=True, italic=True,\n'
+                        "        underline=True, blink=True, blink2=True, reverse=True,\n"
+                        "    )\n"
+                        "    assert str(other)\n",
                     ),
                 ),
                 focus_path=PurePosixPath("tests/test_call.py"),
@@ -79,7 +90,18 @@ class KwargHeavyConstructionInTest(Rule):
                 files=(
                     ExampleFile.python(
                         "tests/test_call.py",
-                        "def build_call(**overrides):\n    return Call(**overrides)\n\ndef test_first():\n    assert build_call(a=1)\n",
+                        "def build_style(**overrides):\n"
+                        "    defaults = dict(\n"
+                        "        dim=True, italic=True, underline=True,\n"
+                        "        blink=True, blink2=True, reverse=True,\n"
+                        "    )\n"
+                        "    return Style(**(defaults | overrides))\n\n"
+                        "def test_style():\n"
+                        '    style = build_style(color="red", bgcolor="black", bold=True)\n'
+                        "    assert str(style)\n\n"
+                        "def test_style_again():\n"
+                        '    other = build_style(color="blue", bgcolor="white", bold=False)\n'
+                        "    assert str(other)\n",
                     ),
                 ),
                 focus_path=PurePosixPath("tests/test_call.py"),

{sarj_python_lint-0.67.4 → sarj_python_lint-0.67.6}/src/sarj_python_lint/rules/prefer_library_fake.py

@@ -297,7 +297,16 @@ class PreferLibraryFake(Rule):
                 files=(
                     ExampleFile.python(
                         "tests/fakes/s3.py",
-                        "class FakeS3Client:\n    def put_object(self, **kwargs):\n        return kwargs\n\n    def get_object(self, **kwargs):\n        return kwargs\n\n    def delete_object(self, **kwargs):\n        return kwargs\n",
+                        "class FakeS3Client:\n"
+                        "    def __init__(self):\n"
+                        "        self.objects = {}\n\n"
+                        "    def put_object(self, Bucket, Key, Body):\n"
+                        "        self.objects[(Bucket, Key)] = Body\n"
+                        '        return {"ResponseMetadata": {"HTTPStatusCode": 200}}\n\n'
+                        "    def get_object(self, Bucket, Key):\n"
+                        '        return {"Body": self.objects[(Bucket, Key)]}\n\n'
+                        "    def delete_object(self, Bucket, Key):\n"
+                        "        self.objects.pop((Bucket, Key), None)\n",
                     ),
                 ),
                 focus_path=PurePosixPath("tests/fakes/s3.py"),

{sarj_python_lint-0.67.4 → sarj_python_lint-0.67.6}/src/sarj_python_lint/rules/prefer_timedelta_for_durations.py

@@ -46,6 +46,11 @@ _EXCLUDE_RE = re.compile(
     re.IGNORECASE,
 )
 
+# Standard HTTP status symbols describe integer protocol codes, even when the
+# reason phrase contains a duration-shaped word (for example, 408 Request
+# Timeout).  Requiring ``timedelta`` here would change the public wire value.
+_HTTP_STATUS_CONSTANT_RE = re.compile(r"^HTTP_[1-5]\d{2}(?:_|$)", re.IGNORECASE)
+
 _NUMERIC_NAMES = frozenset({"int", "float"})
 _BARE_UNIT_NAMES = frozenset(
     {
@@ -181,6 +186,8 @@ class PreferTimedeltaForDurations(Rule):
     ) -> None:
         if annotation is None:
             return
+        if _HTTP_STATUS_CONSTANT_RE.search(name):
+            return
         if name.lower() in _BARE_UNIT_NAMES or name.lower().endswith(("_worked", "_elapsed")):
             return
         if not _UNIT_RE.search(name) or _EXCLUDE_RE.search(name):