From 428f11b0265104e5a5f5f2b14ee0036bef7e83e9 Mon Sep 17 00:00:00 2001 From: Antawari Date: Wed, 27 May 2026 21:26:16 -0600 Subject: [PATCH] style(tests): clear ruff I001 + format in test_verdict_parity MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Three mechanical ruff fixes, no behavior change: - I001: drop the duplicate blank line between the `import pytest` block and the section comment that follows. - E501: ruff format consolidates two assertion-message f-strings in `test_severity_variants` and `test_finding_fields` from two lines to one each — both well under the 100-char line limit after the collapse. Caught while running CI on an unrelated feature branch: the lint gate trips on this file's existing state, blocking every PR whose branch CI runs the full `ruff check src/ tests/`. --- tests/test_verdict_parity.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/tests/test_verdict_parity.py b/tests/test_verdict_parity.py index a399ebb..57df384 100644 --- a/tests/test_verdict_parity.py +++ b/tests/test_verdict_parity.py @@ -40,7 +40,6 @@ import pytest - # --------------------------------------------------------------------------- # Hardcoded reference sets — read from forge-side at test-WRITE time # (ishtar/forge/core/verdict.py, BON-1240 prep merge, SCHEMA_VERSION="1.1") @@ -140,8 +139,7 @@ def test_severity_variants() -> None: actual = {member.name for member in Severity} assert actual == EXPECTED_SEVERITY_VARIANTS, ( - f"Severity drift: bonfire-public has {actual}, expected " - f"{EXPECTED_SEVERITY_VARIANTS}" + f"Severity drift: bonfire-public has {actual}, expected {EXPECTED_SEVERITY_VARIANTS}" ) @@ -162,8 +160,7 @@ def test_finding_fields() -> None: actual = set(Finding.model_fields.keys()) assert actual == EXPECTED_FINDING_FIELDS, ( - f"Finding field drift: bonfire-public has {actual}, expected " - f"{EXPECTED_FINDING_FIELDS}" + f"Finding field drift: bonfire-public has {actual}, expected {EXPECTED_FINDING_FIELDS}" )