style: apply ruff formatting and fix lint warnings

Author: practicalmind-dev

gateframe/audit/log.py

@@ -11,7 +11,14 @@
 
 
 class AuditEntry:
-    __slots__ = ("timestamp", "contract_name", "passed", "rules_applied", "rules_failed", "failures")
+    __slots__ = (
+        "timestamp",
+        "contract_name",
+        "passed",
+        "rules_applied",
+        "rules_failed",
+        "failures",
+    )
 
     def __init__(self, result: ValidationResult) -> None:
         self.timestamp = datetime.now(timezone.utc)

tests/audit/test_log.py

@@ -9,7 +9,9 @@ def _make_result(passed: bool = True, **overrides: object) -> ValidationResult:
         "contract_name": "test_contract",
         "rules_applied": 1,
         "rules_failed": 0 if passed else 1,
-        "failures": [] if passed else [
+        "failures": []
+        if passed
+        else [
             FailureResult(
                 rule_name="rule_a",
                 failure_mode=FailureMode.HARD_FAIL,
@@ -22,7 +24,6 @@ def _make_result(passed: bool = True, **overrides: object) -> ValidationResult:
 
 
 class TestAuditLog:
-
     def test_record_adds_entry(self) -> None:
         log = AuditLog()
         log.record(_make_result())
@@ -49,7 +50,6 @@ def test_entries_returns_copy(self) -> None:
 
 
 class TestAuditEntry:
-
     def test_entry_structure(self) -> None:
         result = _make_result(passed=False)
         entry = AuditEntry(result)

tests/core/test_contract.py

@@ -1,5 +1,8 @@
 from typing import Any
 
+import pytest
+from pydantic import ValidationError
+
 from gateframe.core.contract import ValidationContract, ValidationResult
 from gateframe.core.failure import FailureMode, FailureResult
 from gateframe.core.rule import Rule
@@ -11,7 +14,11 @@ def validate(self, output: Any, **context: Any) -> FailureResult | None:
 
 
 class _FailingRule(Rule):
-    def __init__(self, name: str = "failing_rule", mode: FailureMode = FailureMode.HARD_FAIL) -> None:
+    def __init__(
+        self,
+        name: str = "failing_rule",
+        mode: FailureMode = FailureMode.HARD_FAIL,
+    ) -> None:
         super().__init__(name)
         self._mode = mode
 
@@ -24,7 +31,6 @@ def validate(self, output: Any, **context: Any) -> FailureResult | None:
 
 
 class TestValidationContract:
-
     def test_all_rules_pass(self) -> None:
         contract = ValidationContract("test", [_PassingRule("a"), _PassingRule("b")])
         result = contract.validate({"key": "value"})
@@ -68,11 +74,8 @@ def test_result_has_timestamp(self) -> None:
     def test_result_is_frozen(self) -> None:
         contract = ValidationContract("test", [_PassingRule("a")])
         result = contract.validate({})
-        try:
+        with pytest.raises(ValidationError):
             result.passed = False  # type: ignore[misc]
-            assert False, "Should have raised"
-        except Exception:
-            pass
 
     def test_context_forwarded_to_rules(self) -> None:
         class _ContextRule(Rule):
@@ -91,7 +94,6 @@ def validate(self, output: Any, **context: Any) -> FailureResult | None:
 
 
 class TestValidationResult:
-
     def test_serialization_roundtrip(self) -> None:
         result = ValidationResult(
             passed=False,

tests/core/test_failure.py

@@ -1,12 +1,12 @@
 from datetime import datetime, timezone
 
 import pytest
+from pydantic import ValidationError
 
 from gateframe.core.failure import FailureMode, FailureResult
 
 
 class TestFailureMode:
-
     def test_has_four_members(self) -> None:
         assert len(FailureMode) == 4
 
@@ -31,7 +31,6 @@ def test_invalid_value_raises(self) -> None:
 
 
 class TestFailureResult:
-
     def _make_result(self, **overrides: object) -> FailureResult:
         defaults = {
             "rule_name": "test_rule",
@@ -76,18 +75,18 @@ def test_timestamp_is_utc(self) -> None:
 
     def test_is_frozen(self) -> None:
         result = self._make_result()
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             result.rule_name = "changed"  # type: ignore[misc]
 
     def test_missing_rule_name_raises(self) -> None:
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             FailureResult(
                 failure_mode=FailureMode.HARD_FAIL,
                 message="something failed",
             )  # type: ignore[call-arg]
 
     def test_missing_message_raises(self) -> None:
-        with pytest.raises(Exception):
+        with pytest.raises(ValidationError):
             FailureResult(
                 rule_name="test",
                 failure_mode=FailureMode.HARD_FAIL,

tests/core/test_rule.py

@@ -22,7 +22,6 @@ def validate(self, output: Any, **context: Any) -> FailureResult | None:
 
 
 class TestRule:
-
     def test_cannot_instantiate_directly(self) -> None:
         with pytest.raises(TypeError):
             Rule("test")  # type: ignore[abstract]

tests/rules/test_structural.py

@@ -10,7 +10,6 @@ class _SampleOutput(BaseModel):
 
 
 class TestStructuralRule:
-
     def _make_rule(self, **overrides: object) -> StructuralRule:
         defaults: dict = {"schema": _SampleOutput}
         defaults.update(overrides)