Skip to content

Commit 73277af

Browse files
authored
COMP-807 error closing case file (#740)
1 parent 8a72c2a commit 73277af

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

compliance-api/requirements/dev.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ pylint-flask
77
pytest
88
pytest-cov
99
pytest-env
10-
pytest-dotenv
1110
pytest-mock
1211
requests
1312
flake8==4.0.1

compliance-api/src/compliance_api/services/case_file.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
from compliance_api.models.complaint import ComplaintStatusEnum
2525
from compliance_api.models.db import db, session_scope
2626
from compliance_api.models.inspection import Inspection as InspectionModel
27+
from compliance_api.models.inspection.inspection_enum import InspectionStatusEnum
2728
from compliance_api.models.order import Order as OrderModel
2829
from compliance_api.models.project import Project as ProjectModel
2930
from compliance_api.models.restorative_justice import RestorativeJustice as RestorativeJusticeModel
@@ -761,8 +762,11 @@ def _process_case_level_items(case_file_id: int, open_items: dict) -> list:
761762
for row in case_level_query:
762763
if row.inspection_id and row.inspection_id not in all_inspection_ids:
763764
all_inspection_ids.add(row.inspection_id)
764-
# Add all inspection items, regardless of IR status
765-
open_items["inspections"].append(_build_inspection_item(row))
765+
# Only add inspections that are open to the open items list,
766+
# closed inspections are not considered open items but we
767+
# want to track their IDs for enforcement action checks
768+
if row.inspection_status == InspectionStatusEnum.OPEN:
769+
open_items["inspections"].append(_build_inspection_item(row))
766770

767771
if row.complaint_id and row.complaint_id not in processed_complaints:
768772
processed_complaints.add(row.complaint_id)

0 commit comments

Comments
 (0)