@@ -766,14 +766,14 @@ def test_status_on_cohort_attribute_level(faker: Faker):
766766
767767
768768@pytest .mark .parametrize (
769- ("person_cohorts" , "cohort " , "expected_status" , "test_comment" ),
769+ ("person_cohorts" , "cohort_label " , "expected_status" , "test_comment" ),
770770 [
771771 (["cohort1" , "cohort2" ], "cohort1" , Status .not_actionable , "matches the cohort label" ),
772772 (["cohort2" , "cohort3" ], "cohort1" , Status .actionable , "doesn't match the cohort label" ),
773773 ],
774774)
775775def test_status_if_iteration_rules_contains_cohort_label_field (
776- person_cohorts , cohort : str , expected_status : Status , test_comment : str , faker : Faker
776+ person_cohorts , cohort_label : str , expected_status : Status , test_comment : str , faker : Faker
777777):
778778 # Given
779779 nhs_number = NHSNumber (faker .nhs_number ())
@@ -789,7 +789,7 @@ def test_status_if_iteration_rules_contains_cohort_label_field(
789789 rule_builder .IterationCohortFactory .build (cohort_label = "cohort1" ),
790790 rule_builder .IterationCohortFactory .build (cohort_label = "cohort2" ),
791791 ],
792- iteration_rules = [rule_builder .PersonAgeSuppressionRuleFactory .build (cohort_label = cohort )],
792+ iteration_rules = [rule_builder .PersonAgeSuppressionRuleFactory .build (cohort_label = cohort_label )],
793793 )
794794 ],
795795 )
0 commit comments