Skip to content

Commit e933ce5

Browse files
authored
Rj tests (#559)
* administrative penalty integration tests * restorative justice test cases * test cases
1 parent f388c39 commit e933ce5

14 files changed

+2808
-5
lines changed

compliance-api/src/compliance_api/resources/charge_recommendation.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -152,9 +152,10 @@ def patch(charge_recommendation_id):
152152
@API.response(code=204, description="NoContent")
153153
def delete(charge_recommendation_id):
154154
"""Delete a charge recommendation."""
155-
return ChargeRecommendationService.delete_charge_recommendation(
155+
ChargeRecommendationService.delete_charge_recommendation(
156156
charge_recommendation_id
157157
)
158+
return {}, HTTPStatus.NO_CONTENT
158159

159160

160161
@cors_preflight("GET, OPTIONS")

compliance-api/src/compliance_api/resources/restorative_justice.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,9 +146,10 @@ def patch(restorative_justice_id):
146146
@API.response(code=204, description="NoContent")
147147
def delete(restorative_justice_id):
148148
"""Delete a restorative justice."""
149-
return RestorativeJusticeService.delete_restorative_justice(
149+
RestorativeJusticeService.delete_restorative_justice(
150150
restorative_justice_id
151151
)
152+
return {}, HTTPStatus.NO_CONTENT
152153

153154

154155
@cors_preflight("GET, OPTIONS")

compliance-api/src/compliance_api/schemas/charge_recommendation.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,7 @@ class Meta: # pylint: disable=too-few-public-methods
126126

127127
model = ChargeRecommendation
128128
load_instance = True
129+
include_fk = True
129130

130131
court_appearances = fields.String(
131132
allow_none=True,

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

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
"""Restorative Justice Service."""
22

3-
from http import HTTPStatus
4-
53
from compliance_api.exceptions import BadRequestError, ResourceNotFoundError, UnprocessableEntityError
64
from compliance_api.models.case_file import CaseFile as CaseFileModel
75
from compliance_api.models.db import session_scope
@@ -146,7 +144,6 @@ def delete_restorative_justice(cls, restorative_justice_id: int):
146144
RestorativeJusticeInspectionRequirementMap.delete_by_restorative_justice_id(
147145
restorative_justice_id, session
148146
)
149-
return HTTPStatus.NO_CONTENT
150147

151148
@classmethod
152149
def insert_or_update_inspection_requirements(

compliance-api/tests/conftest.py

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,25 @@
3030
from .utilities.factory_scenario.administrative_penalty_fixture import ( # noqa: F401
3131
created_administrative_penalty, created_administrative_penalty_inspection_requirement,
3232
created_administrative_penalty_requirement_map)
33+
from .utilities.factory_scenario.charge_recommendation_fixture import ( # noqa: F401
34+
created_charge_recommendation, created_charge_recommendation_deputy_review, created_charge_recommendation_drafting,
35+
created_charge_recommendation_inspection_requirement, created_charge_recommendation_not_proceeding,
36+
created_charge_recommendation_requirement_map, created_charge_recommendation_submitted,
37+
created_charge_recommendation_with_court_details)
3338
from .utilities.factory_scenario.common_fixture import ( # noqa: F401
3439
created_case_file, created_inspection, created_inspection_requirement, created_staff, mock_auth_service,
3540
mock_doc_gen_service, mock_doc_service, mock_track_service)
3641
from .utilities.factory_scenario.order_fixture import ( # noqa: F401
3742
created_order, created_order_requirement_map, created_section)
43+
from .utilities.factory_scenario.restorative_justice_fixture import ( # noqa: F401
44+
created_restorative_justice, created_restorative_justice_closed, created_restorative_justice_drafting,
45+
created_restorative_justice_inspection_requirement, created_restorative_justice_open,
46+
created_restorative_justice_requirement_map)
47+
from .utilities.factory_scenario.violation_ticket_fixture import ( # noqa: F401
48+
created_violation_ticket, created_violation_ticket_disputed, created_violation_ticket_inspection_requirement,
49+
created_violation_ticket_issued, created_violation_ticket_no_fine, created_violation_ticket_paid,
50+
created_violation_ticket_requirement_map, created_violation_ticket_with_large_fine,
51+
created_violation_ticket_zero_fine)
3852
from .utilities.factory_utils import factory_auth_header
3953

4054

0 commit comments

Comments
 (0)