Skip to content

Commit eb08f5e

Browse files
removed "PERSON" prefix req in nhs number (#175)
1 parent cede0e0 commit eb08f5e

3 files changed

Lines changed: 6 additions & 6 deletions

File tree

src/eligibility_signposting_api/repos/person_repo.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ def __init__(self, table: Annotated[Any, Inject(qualifier="person_table")]) -> N
3636
self.table = table
3737

3838
def get_eligibility_data(self, nhs_number: NHSNumber) -> list[dict[str, Any]]:
39-
response = self.table.query(KeyConditionExpression=Key("NHS_NUMBER").eq(f"PERSON#{nhs_number}"))
39+
response = self.table.query(KeyConditionExpression=Key("NHS_NUMBER").eq(nhs_number))
4040
logger.debug("response %r for %r", response, nhs_number, extra={"response": response, "nhs_number": nhs_number})
4141

4242
if not (items := response.get("Items")):

tests/fixtures/builders/repos/person.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ def person_rows_builder( # noqa:PLR0913
3131
faker = Faker("en_UK")
3232
faker.add_provider(PersonDetailProvider)
3333

34-
key = f"PERSON#{nhs_number}"
34+
key = nhs_number
3535
date_of_birth = date_of_birth if date_of_birth is not ... else faker.date_of_birth(minimum_age=18, maximum_age=99)
3636
gender = gender if gender is not ... else choice(get_args(Gender))
3737
postcode = postcode if postcode is not ... else faker.postcode()

tests/integration/repo/test_person_repo.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,10 @@ def test_person_found(person_table: Any, persisted_person: NHSNumber):
2020
assert_that(
2121
actual,
2222
contains_inanyorder(
23-
has_entries({"NHS_NUMBER": f"PERSON#{persisted_person}", "ATTRIBUTE_TYPE": "PERSON"}),
24-
has_entries({"NHS_NUMBER": f"PERSON#{persisted_person}", "ATTRIBUTE_TYPE": "COHORTS"}),
25-
has_entries({"NHS_NUMBER": f"PERSON#{persisted_person}", "ATTRIBUTE_TYPE": "COVID"}),
26-
has_entries({"NHS_NUMBER": f"PERSON#{persisted_person}", "ATTRIBUTE_TYPE": "RSV"}),
23+
has_entries({"NHS_NUMBER": persisted_person, "ATTRIBUTE_TYPE": "PERSON"}),
24+
has_entries({"NHS_NUMBER": persisted_person, "ATTRIBUTE_TYPE": "COHORTS"}),
25+
has_entries({"NHS_NUMBER": persisted_person, "ATTRIBUTE_TYPE": "COVID"}),
26+
has_entries({"NHS_NUMBER": persisted_person, "ATTRIBUTE_TYPE": "RSV"}),
2727
),
2828
)
2929

0 commit comments

Comments
 (0)