Skip to content

Commit 44663ef

Browse files
MARSclaude
andcommitted
fix(lint): remove unused fresh_finding variable in test_vector3 (F841)
Ruff correctly flagged the scaffolding dict as unused. Test intent preserved — asserts new_findings is a list on a doc with no triggers. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 38472ea commit 44663ef

1 file changed

Lines changed: 2 additions & 9 deletions

File tree

tests/odia_legal/test_vector3.py

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -107,20 +107,13 @@ def test_all_prior_findings_not_in_current_become_resolved():
107107

108108

109109
def test_new_finding_appears_only_in_new():
110-
fresh_finding = {
111-
"id": "legal:l99:nonexistent:new_one",
112-
"issue": "new finding not in prior",
113-
"severity": "medium",
114-
"layer": "l99",
115-
"details": {},
116-
}
117-
# Patch: run against empty prior so we know current > prior
110+
# Run against empty prior and a doc with no detector triggers —
111+
# new_findings must be an empty list (not None or missing)
118112
result = LegalVector3().reeval(
119113
{"text": "nothing relevant"},
120114
[],
121115
prior_run_date="2023-01-01",
122116
)
123-
# The doc produces no findings, so new_findings should be empty
124117
assert isinstance(result.new_findings, list)
125118

126119

0 commit comments

Comments
 (0)