Skip to content

Commit eb53d8f

Browse files
All tests passed 👽
1 parent 60425b1 commit eb53d8f

3 files changed

Lines changed: 9 additions & 5 deletions

File tree

‎src/eligibility_signposting_api/services/calculators/eligibility_calculator.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ def evaluate_eligibility(self) -> eligibility.EligibilityStatus:
129129
)
130130
if status.is_exclusion:
131131
actionable_flag = False
132-
suppression_reasons.append(group_exclusions)
132+
suppression_reasons.extend(group_exclusions)
133133
if rule_stop:
134134
break
135135
# No exclusions - actionable

‎src/eligibility_signposting_api/views/eligibility.py‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,13 @@ def build_eligibility_response(
5151
) -> eligibility.EligibilityResponse:
5252
"""Return an object representing the API response we are going to send, given an evaluation of the person's
5353
eligibility."""
54+
55+
for condition in eligibility_status.conditions:
56+
for cohort_result in condition.cohort_results:
57+
for reason in cohort_result.reasons:
58+
print(f"Debug: {reason}")
59+
print(f"Type: {type(reason)}")
60+
5461
return eligibility.EligibilityResponse( # pyright: ignore[reportCallIssue]
5562
response_id=uuid.uuid4(), # pyright: ignore[reportCallIssue]
5663
meta=eligibility.Meta(last_updated=eligibility.LastUpdated(datetime.now(tz=UTC))), # pyright: ignore[reportCallIssue]

‎tests/unit/services/calculators/test_eligibility_calculator.py‎

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import datetime
2-
import itertools
32

43
import pytest
54
from faker import Faker
@@ -889,9 +888,7 @@ def test_rules_stop_behavior(
889888
for condition in actual.conditions:
890889
if condition.condition_name == ConditionName("RSV"):
891890
for cohort_result in condition.cohort_results:
892-
actual_reason_results.extend(
893-
[reason.rule_result for reason in list(itertools.chain(*cohort_result.reasons))]
894-
)
891+
actual_reason_results.extend([reason.rule_result for reason in cohort_result.reasons])
895892

896893
# Then
897894
assert_that(

0 commit comments

Comments
 (0)