Skip to content

Commit 54482e2

Browse files
KarthikeyannhsKarthikeyannhs
committed
"CampaignConfigID" is the campaign id in consumer mapping
1 parent baf882b commit 54482e2

5 files changed

Lines changed: 40 additions & 40 deletions

File tree

‎src/eligibility_signposting_api/model/consumer_mapping.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99

1010
class ConsumerCampaign(BaseModel):
11-
campaign_config_id: CampaignID = Field(alias="CampaignConfigId")
11+
campaign_config_id: CampaignID = Field(alias="CampaignConfigID")
1212
description: str | None = Field(default=None, alias="Description")
1313

1414

‎tests/integration/conftest.py‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1353,7 +1353,7 @@ def create_and_put_consumer_mapping_in_s3(
13531353
) -> ConsumerMapping:
13541354
consumer_mapping = ConsumerMapping.model_validate({})
13551355
campaign_entry = ConsumerCampaign(
1356-
CampaignConfigId=campaign_config.id, Description="Test description for campaign mapping"
1356+
CampaignConfigID=campaign_config.id, Description="Test description for campaign mapping"
13571357
)
13581358

13591359
consumer_mapping.root[ConsumerId(consumer_id)] = [campaign_entry]
@@ -1530,7 +1530,7 @@ def consumer_to_campaign_having_inactive_iteration_mapping(
15301530
):
15311531
mapping = ConsumerMapping.model_validate({})
15321532
mapping.root[consumer_id] = [
1533-
ConsumerCampaign(CampaignConfigId=cc.id, Description=f"Description for {cc.id}")
1533+
ConsumerCampaign(CampaignConfigID=cc.id, Description=f"Description for {cc.id}")
15341534
for cc in inactive_iteration_config
15351535
]
15361536

@@ -1553,7 +1553,7 @@ def consumer_to_multiple_campaign_configs_mapping(
15531553
) -> Generator[ConsumerMapping]:
15541554
mapping = ConsumerMapping.model_validate({})
15551555
mapping.root[consumer_id] = [
1556-
ConsumerCampaign(CampaignConfigId=cc.id, Description=f"Description for {cc.id}")
1556+
ConsumerCampaign(CampaignConfigID=cc.id, Description=f"Description for {cc.id}")
15571557
for cc in multiple_campaign_configs
15581558
]
15591559

‎tests/integration/in_process/test_eligibility_endpoint.py‎

Lines changed: 29 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -944,8 +944,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
944944
],
945945
{
946946
"consumer-id": [
947-
{"CampaignConfigId": "RSV_campaign_id"},
948-
{"CampaignConfigId": "COVID_campaign_id"},
947+
{"CampaignConfigID": "RSV_campaign_id"},
948+
{"CampaignConfigID": "COVID_campaign_id"},
949949
]
950950
},
951951
"consumer-id",
@@ -962,8 +962,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
962962
],
963963
{
964964
"consumer-id": [
965-
{"CampaignConfigId": "RSV_campaign_id"},
966-
{"CampaignConfigId": "COVID_campaign_id"},
965+
{"CampaignConfigID": "RSV_campaign_id"},
966+
{"CampaignConfigID": "COVID_campaign_id"},
967967
]
968968
},
969969
"consumer-id",
@@ -980,8 +980,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
980980
],
981981
{
982982
"consumer-id": [
983-
{"CampaignConfigId": "RSV_campaign_id"},
984-
{"CampaignConfigId": "COVID_campaign_id"},
983+
{"CampaignConfigID": "RSV_campaign_id"},
984+
{"CampaignConfigID": "COVID_campaign_id"},
985985
]
986986
},
987987
"consumer-id",
@@ -1001,8 +1001,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
10011001
],
10021002
{
10031003
"consumer-id": [
1004-
{"CampaignConfigId": "RSV_campaign_id"},
1005-
{"CampaignConfigId": "COVID_campaign_id"},
1004+
{"CampaignConfigID": "RSV_campaign_id"},
1005+
{"CampaignConfigID": "COVID_campaign_id"},
10061006
]
10071007
},
10081008
"consumer-id",
@@ -1017,8 +1017,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
10171017
],
10181018
{
10191019
"consumer-id": [
1020-
{"CampaignConfigId": "RSV_campaign_id"},
1021-
{"CampaignConfigId": "COVID_campaign_id"},
1020+
{"CampaignConfigID": "RSV_campaign_id"},
1021+
{"CampaignConfigID": "COVID_campaign_id"},
10221022
]
10231023
},
10241024
"consumer-id",
@@ -1033,8 +1033,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
10331033
],
10341034
{
10351035
"consumer-id": [
1036-
{"CampaignConfigId": "RSV_campaign_id"},
1037-
{"CampaignConfigId": "COVID_campaign_id"},
1036+
{"CampaignConfigID": "RSV_campaign_id"},
1037+
{"CampaignConfigID": "COVID_campaign_id"},
10381038
]
10391039
},
10401040
"consumer-id",
@@ -1054,8 +1054,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
10541054
],
10551055
{
10561056
"consumer-id": [
1057-
{"CampaignConfigId": "RSV_campaign_id"},
1058-
{"CampaignConfigId": "COVID_campaign_id"},
1057+
{"CampaignConfigID": "RSV_campaign_id"},
1058+
{"CampaignConfigID": "COVID_campaign_id"},
10591059
]
10601060
},
10611061
"another-consumer-id",
@@ -1072,8 +1072,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
10721072
],
10731073
{
10741074
"consumer-id": [
1075-
{"CampaignConfigId": "RSV_campaign_id"},
1076-
{"CampaignConfigId": "COVID_campaign_id"},
1075+
{"CampaignConfigID": "RSV_campaign_id"},
1076+
{"CampaignConfigID": "COVID_campaign_id"},
10771077
]
10781078
},
10791079
"another-consumer-id",
@@ -1093,8 +1093,8 @@ def test_not_actionable_and_check_response_when_rule_mapper_is_given(
10931093
],
10941094
{
10951095
"consumer-id": [
1096-
{"CampaignConfigId": "RSV_campaign_id"},
1097-
{"CampaignConfigId": "COVID_campaign_id"},
1096+
{"CampaignConfigID": "RSV_campaign_id"},
1097+
{"CampaignConfigID": "COVID_campaign_id"},
10981098
]
10991099
},
11001100
"consumer-id",
@@ -1201,21 +1201,21 @@ def test_valid_response_when_consumer_has_a_valid_campaign_config_mapping( # no
12011201
],
12021202
{
12031203
# Consumer mappings in S3
1204-
"consumer-id-1": [{"CampaignConfigId": "RSV_campaign_id_1"}],
1205-
"consumer-id-2": [{"CampaignConfigId": "RSV_campaign_id_2"}],
1204+
"consumer-id-1": [{"CampaignConfigID": "RSV_campaign_id_1"}],
1205+
"consumer-id-2": [{"CampaignConfigID": "RSV_campaign_id_2"}],
12061206
"consumer-id-3a": [
1207-
{"CampaignConfigId": "RSV_campaign_id_3"},
1208-
{"CampaignConfigId": "RSV_campaign_id_4"},
1207+
{"CampaignConfigID": "RSV_campaign_id_3"},
1208+
{"CampaignConfigID": "RSV_campaign_id_4"},
12091209
],
12101210
"consumer-id-3b": [
1211-
{"CampaignConfigId": "RSV_campaign_id_4"},
1212-
{"CampaignConfigId": "RSV_campaign_id_3"},
1211+
{"CampaignConfigID": "RSV_campaign_id_4"},
1212+
{"CampaignConfigID": "RSV_campaign_id_3"},
12131213
],
12141214
"consumer-id-4": [
1215-
{"CampaignConfigId": "inactive_RSV_campaign_id_5"},
1216-
{"CampaignConfigId": "RSV_campaign_id_6"},
1215+
{"CampaignConfigID": "inactive_RSV_campaign_id_5"},
1216+
{"CampaignConfigID": "RSV_campaign_id_6"},
12171217
],
1218-
"consumer-id-5": [{"CampaignConfigId": "inactive_RSV_campaign_id_5"}],
1218+
"consumer-id-5": [{"CampaignConfigID": "inactive_RSV_campaign_id_5"}],
12191219
},
12201220
"RSV",
12211221
"VACCINATIONS",
@@ -1278,8 +1278,8 @@ def test_if_campaign_having_best_status_is_chosen_if_there_exists_multiple_campa
12781278
Body=json.dumps(
12791279
{
12801280
consumer_id: [
1281-
{"CampaignConfigId": "RSV_campaign_id_not_actionable"},
1282-
{"CampaignConfigId": "RSV_campaign_id_actionable"},
1281+
{"CampaignConfigID": "RSV_campaign_id_not_actionable"},
1282+
{"CampaignConfigID": "RSV_campaign_id_actionable"},
12831283
],
12841284
}
12851285
),

‎tests/test_data/test_consumer_mapping/test_consumer_mapping_config.json‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,21 +2,21 @@
22
{
33
"consumer-id-123": [
44
{
5-
"CampaignConfigId": "8fcb742b-45fa-4e0d-8f2f-9c2efb1f46d0",
5+
"CampaignConfigID": "8fcb742b-45fa-4e0d-8f2f-9c2efb1f46d0",
66
"Description": "RSV Ongoing for My Vaccines"
77
},
88
{
9-
"CampaignConfigId": "COVID_campaign_id",
9+
"CampaignConfigID": "COVID_campaign_id",
1010
"Description": "COVID Ongoing for My Vaccines"
1111
}
1212
],
1313
"consumer-id-456": [
1414
{
15-
"CampaignConfigId": "RSV_campaign_id_NBS",
15+
"CampaignConfigID": "RSV_campaign_id_NBS",
1616
"Description": "RSV Ongoing for NBS"
1717
},
1818
{
19-
"CampaignConfigId": "COVID_campaign_id_NBS",
19+
"CampaignConfigID": "COVID_campaign_id_NBS",
2020
"Description": "RSV Ongoing for NBS"
2121
}
2222
]

‎tests/unit/repos/test_consumer_mapping_repo.py‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ def test_get_permitted_campaign_ids_success(self, repo, mock_s3_client):
2626
# The mocked S3 data must match the new schema (objects with description)
2727
mapping_data = {
2828
consumer_id: [
29-
{"CampaignConfigId": "flu-2024", "Description": "Flu Shot Description"},
30-
{"CampaignConfigId": "covid-2024", "Description": "Covid Shot Description"},
29+
{"CampaignConfigID": "flu-2024", "Description": "Flu Shot Description"},
30+
{"CampaignConfigID": "covid-2024", "Description": "Covid Shot Description"},
3131
]
3232
}
3333

@@ -49,7 +49,7 @@ def test_get_permitted_campaign_ids_returns_none_when_missing(self, repo, mock_s
4949
Setup data where the consumer_id doesn't exist
5050
We must still use the valid schema (dicts inside the list) to pass Pydantic validation
5151
"""
52-
valid_schema_data = {"other-user": [{"CampaignConfigId": "camp-1", "Description": "Some description"}]}
52+
valid_schema_data = {"other-user": [{"CampaignConfigID": "camp-1", "Description": "Some description"}]}
5353

5454
mock_s3_client.list_objects.return_value = {"Contents": [{"Key": "mappings.json"}]}
5555
body_json = json.dumps(valid_schema_data).encode("utf-8")

0 commit comments

Comments
 (0)