ruff

Author: JamesW1-NHS

lambdas/filenameprocessor/src/audit_table.py

@@ -15,7 +15,7 @@ def upsert_audit_table(
     queue_name: str,
     file_status: str,
     error_details: Optional[str] = None,
-    condition_expression: str = ""
+    condition_expression: str = "",
 ) -> None:
     """
     Updates the audit table with the file details

lambdas/filenameprocessor/src/file_name_processor.py

@@ -7,9 +7,10 @@
 """
 
 import argparse
-from botocore.exceptions import ClientError
 from uuid import uuid4
 
+from botocore.exceptions import ClientError
+
 from audit_table import upsert_audit_table
 from common.aws_s3_utils import (
     copy_file_outside_bucket,
@@ -249,7 +250,7 @@ def handle_extended_attributes_file(
     Processes a single record for extended attributes file.
     Returns a dictionary containing information to be included in the logs.
     """
-    
+
     # here: the sequence of events should be
     # 1. upsert 'processing'
     # 2. move the file to the dest bucket
@@ -314,8 +315,8 @@ def handle_extended_attributes_file(
             file_key,
             created_at_formatted_string,
             expiry_timestamp,
-            extended_attribute_identifier,
             queue_name,
+            file_status,
             error_details=str(error),
         )
 

lambdas/shared/src/common/aws_s3_utils.py

@@ -29,6 +29,7 @@ def copy_file_outside_bucket(source_bucket: str, source_key: str, destination_bu
         ExpectedSourceBucketOwner=EXPECTED_BUCKET_OWNER_ACCOUNT,
     )
 
+
 def delete_file(source_bucket: str, source_key: str) -> None:
     s3_client = get_s3_client()
     s3_client.delete_object(
@@ -37,7 +38,7 @@ def delete_file(source_bucket: str, source_key: str) -> None:
         ExpectedBucketOwner=EXPECTED_BUCKET_OWNER_ACCOUNT,
     )
 
+
 def is_file_in_bucket(bucket_name: str, file_key: str) -> None:
     s3_client = get_s3_client()
     s3_client.head_object(Bucket=bucket_name, Key=file_key)
-