[NDR-359] Remove comments

Author: jameslinnell

lambdas/services/delete_fhir_document_reference_service.py

@@ -102,11 +102,6 @@ def delete_fhir_document_references_by_nhs_id(
                     document_references=documents,
                     document_ttl_days=DocumentRetentionDays.SOFT_DELETE,
                 )
-                # self.delete_document_references(
-                #     table_name=dynamo_table,
-                #     document_references=documents,
-                #     document_ttl_days=DocumentRetentionDays.SOFT_DELETE,
-                # )
                 logger.info(
                     f"Deleted document of type {doc_type.display_name}",
                     {"Result": "Successful deletion"},
@@ -119,38 +114,6 @@ def delete_fhir_document_references_by_nhs_id(
             )
             raise DocumentDeletionServiceException(500, LambdaError.DocDelClient)
 
-    # def delete_document_references(
-    #     self,
-    #     table_name: str,
-    #     document_references: list[DocumentReference],
-    #     document_ttl_days: int,
-    # ):
-    #     deletion_date = datetime.now(timezone.utc)
-    #
-    #     ttl_seconds = document_ttl_days * 24 * 60 * 60
-    #     document_reference_ttl = int(deletion_date.timestamp() + ttl_seconds)
-    #
-    #     logger.info(f"Deleting items in table: {table_name}")
-    #
-    #     for reference in document_references:
-    #         reference.doc_status = "deprecated"
-    #         reference.deleted = deletion_date.strftime("%Y-%m-%dT%H:%M:%S.%fZ")
-    #         reference.ttl = document_reference_ttl
-    #
-    #         update_fields = reference.model_dump(
-    #             by_alias=True,
-    #             exclude_none=True,
-    #             include={"doc_status", "deleted", "ttl"},
-    #         )
-    #         self.dynamo_service.update_item(
-    #             table_name=table_name,
-    #             key_pair={
-    #                 DocumentReferenceMetadataFields.NHS_NUMBER.value: reference.nhs_number,
-    #                 DocumentReferenceMetadataFields.ID.value: reference.id,
-    #             },
-    #             updated_fields=update_fields,
-    #         )
-
     def _determine_document_type_based_on_common_name(
         self, common_name: MtlsCommonNames | None
     ) -> SnomedCode:

lambdas/tests/unit/services/test_document_service.py

@@ -217,11 +217,11 @@ def test_delete_documents_soft_delete_core(mock_service, mock_dynamo_service):
     }
 
     mock_service.delete_document_references(
-        "dev_COREDocument_metadata", [test_doc_ref], DocumentRetentionDays.SOFT_DELETE
+        "dev_COREDocumentMetadata", [test_doc_ref], DocumentRetentionDays.SOFT_DELETE
     )
 
     mock_dynamo_service.update_item.assert_called_once_with(
-        table_name="dev_COREDocument_metadata",
+        table_name="dev_COREDocumentMetadata",
         key_pair={"ID": test_doc_ref.id, "NhsNumber": test_doc_ref.nhs_number},
         updated_fields=test_update_fields,
     )