[PRMP-1080] Fixed formatting

Author: FoxMaltas-NHS

lambdas/services/document_review_processor_service.py

@@ -4,13 +4,11 @@
 
 from botocore.exceptions import ClientError
 from enums.document_review_status import DocumentReviewStatus
-from models.document_reference import DocumentReferenceMetadataFields
 from models.document_review import (
     DocumentReviewFileDetails,
     DocumentUploadReviewReference,
 )
 from models.sqs.review_message_body import ReviewMessageBody
-from services.base.dynamo_service import DynamoDBService
 from services.base.s3_service import S3Service
 from services.document_upload_review_service import DocumentUploadReviewService
 from utils.audit_logging_setup import LoggingService
@@ -54,11 +52,9 @@ def process_review_message(self, review_message: ReviewMessageBody) -> None:
         document_upload_review = self._build_review_record(
             review_message, review_id, review_files
         )
-        
+
         try:
-            self.review_document_service.create_dynamo_entry(
-                document_upload_review
-            )
+            self.review_document_service.create_dynamo_entry(document_upload_review)
         except ClientError as e:
             if e.response["Error"]["Code"] == "ConditionalCheckFailedException":
                 logger.info("Entry already exists on Document Review table")

lambdas/tests/unit/services/test_document_review_processor_service.py

@@ -17,7 +17,9 @@ def mock_s3_service(mocker):
 
 @pytest.fixture
 def mock_document_review_service(mocker):
-    return mocker.patch("services.document_review_processor_service.DocumentUploadReviewService")
+    return mocker.patch(
+        "services.document_review_processor_service.DocumentUploadReviewService"
+    )
 
 
 @pytest.fixture
@@ -151,9 +153,11 @@ def test_process_review_message_dynamo_error_not_precondition(
             )
         ],
     )
-    service_under_test.review_document_service.create_dynamo_entry.side_effect = ClientError(
-        {"Error": {"Code": "InternalServerError", "Message": "DynamoDB error"}},
-        "PutItem",
+    service_under_test.review_document_service.create_dynamo_entry.side_effect = (
+        ClientError(
+            {"Error": {"Code": "InternalServerError", "Message": "DynamoDB error"}},
+            "PutItem",
+        )
     )
 
     with pytest.raises(ClientError):
@@ -175,14 +179,16 @@ def test_process_review_message_continues_dynamo_conditional_check_failure(
         ],
     )
     mocker.patch.object(service_under_test, "_delete_files_from_staging")
-    service_under_test.review_document_service.create_dynamo_entry.side_effect = ClientError(
-        {
-            "Error": {
-                "Code": "ConditionalCheckFailedException",
-                "Message": "DynamoDB error",
-            }
-        },
-        "PutItem",
+    service_under_test.review_document_service.create_dynamo_entry.side_effect = (
+        ClientError(
+            {
+                "Error": {
+                    "Code": "ConditionalCheckFailedException",
+                    "Message": "DynamoDB error",
+                }
+            },
+            "PutItem",
+        )
     )
 
     service_under_test.process_review_message(sample_review_message)
@@ -418,5 +424,7 @@ def test_workflow_handles_multiple_different_patients(service_under_test):
     for message in messages:
         service_under_test.process_review_message(message)
 
-    assert service_under_test.review_document_service.create_dynamo_entry.call_count == 3
+    assert (
+        service_under_test.review_document_service.create_dynamo_entry.call_count == 3
+    )
     assert service_under_test.s3_service.copy_across_bucket.call_count == 3