Add comment to unit tests

edhall-nhs · edhall-nhs · commit d9d93cf59a9d · 2025-11-24T10:41:22.000Z
diff --git a/lambdas/recordforwarder/tests/repository/test_fhir_batch_repository.py b/lambdas/recordforwarder/tests/repository/test_fhir_batch_repository.py
@@ -396,11 +396,13 @@ def update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_
         )
 
     def test_update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_tpp(self):
+        """it should use a V5 identifier_pk for a legacy TPP identifier"""
         self.update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_logic(
             "YGA", "https://tpp-uk.com/Id/ve/vacc"
         )
 
     def test_update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_emis(self):
+        """it should use a V5 identifier_pk for a legacy EMIS identifier"""
         self.update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_logic(
             "YGJ", "https://emishealth.com/identifiers/vacc"
         )
@@ -561,11 +563,13 @@ def delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_
         self.assertEqual(response, f"Immunization#{self.immunization['id']}")
 
     def test_delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_tpp(self):
+        """it should use a V5 identifier_pk for a legacy TPP identifier"""
         self.delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_logic(
             "YGA", "https://tpp-uk.com/Id/ve/vacc"
         )
 
     def test_delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_emis(self):
+        """it should use a V5 identifier_pk for a legacy EMIS identifier"""
         self.delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_logic(
             "YGJ", "https://emishealth.com/identifiers/vacc"
         )

Original file line number	Diff line number	Diff line change
`@@ -396,11 +396,13 @@ def update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_`
`396`	`396`	`)`
`397`	`397`
`398`	`398`	`def test_update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_tpp(self):`
	`399`	`+ """it should use a V5 identifier_pk for a legacy TPP identifier"""`
`399`	`400`	`self.update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_logic(`
`400`	`401`	`"YGA", "https://tpp-uk.com/Id/ve/vacc"`
`401`	`402`	`)`
`402`	`403`
`403`	`404`	`def test_update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_emis(self):`
	`405`	`+ """it should use a V5 identifier_pk for a legacy EMIS identifier"""`
`404`	`406`	`self.update_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_logic(`
`405`	`407`	`"YGJ", "https://emishealth.com/identifiers/vacc"`
`406`	`408`	`)`
`@@ -561,11 +563,13 @@ def delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_`
`561`	`563`	`self.assertEqual(response, f"Immunization#{self.immunization['id']}")`
`562`	`564`
`563`	`565`	`def test_delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_tpp(self):`
	`566`	`+ """it should use a V5 identifier_pk for a legacy TPP identifier"""`
`564`	`567`	`self.delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_logic(`
`565`	`568`	`"YGA", "https://tpp-uk.com/Id/ve/vacc"`
`566`	`569`	`)`
`567`	`570`
`568`	`571`	`def test_delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_emis(self):`
	`572`	`+ """it should use a V5 identifier_pk for a legacy EMIS identifier"""`
`569`	`573`	`self.delete_immunization_uses_separate_identifier_pk_for_legacy_identifiers_test_logic(`
`570`	`574`	`"YGJ", "https://emishealth.com/identifiers/vacc"`
`571`	`575`	`)`