fix lint issues

Author: Akol125

recordprocessor/src/constants.py

@@ -1,7 +1,7 @@
 """Constants for recordprocessor"""
 
 import os
-from enum import Enum, auto
+from enum import Enum
 
 SOURCE_BUCKET_NAME = os.getenv("SOURCE_BUCKET_NAME")
 ACK_BUCKET_NAME = os.getenv("ACK_BUCKET_NAME")
@@ -86,13 +86,13 @@ class Urls:
     NULL_FLAVOUR_CODES = "http://terminology.hl7.org/CodeSystem/v3-NullFlavor"
     VACCINATION_PROCEDURE = "https://fhir.hl7.org.uk/StructureDefinition/Extension-UKCore-VaccinationProcedure"
 
+
 class Permission(Enum):
     CREATE = "C"
     UPDATE = "U"
     DELETE = "D"
 
 
-
 class ActionFlag(Enum):
     NEW = Permission.CREATE
     UPDATE = Permission.UPDATE

recordprocessor/src/file_level_validation.py

@@ -32,13 +32,16 @@ def validate_action_flag_permissions(
     operations_requested = get_unique_action_flags_from_s3(csv_data)
 
     # Map ACTION_FLAGs to single-letter permissions
-    requested_permissions = {ActionFlag[flag].value.value for flag in operations_requested if flag in ActionFlag.__members__}
-    
+    requested_permissions = {
+        ActionFlag[flag].value.value
+        for flag in operations_requested
+        if flag in ActionFlag.__members__
+        }
+
     if not requested_permissions:
         logger.warning("No valid ACTION_FLAGs found in file. Skipping permission validation.")
         return set()
 
-
     # Get allowed permission in single letters from allowed_permissions_list
     allowed_ops = set()
     for perm in allowed_permissions_list:
@@ -57,10 +60,11 @@ def validate_action_flag_permissions(
             allowed_ops.update({"C", "R", "U", "D", "S"})
         else:
             allowed_ops.add(op_code)
-    
-    if not requested_permissions.intersection(allowed_ops):
-        raise NoOperationPermissions(f"{supplier} does not have permissions to perform any of the requested actions.")
 
+    if not requested_permissions.intersection(allowed_ops):
+        raise NoOperationPermissions(
+            f"{supplier} does not have permissions to perform any of the requested actions."
+            )
 
     logger.info(
         "%s permissions %s match one of the requested permissions required to %s",
@@ -70,7 +74,7 @@ def validate_action_flag_permissions(
     )
 
     # Return allowed ops in full-word format for downstream logic
-    return {perm.name for perm in Permission if perm.value in allowed_ops }
+    return {perm.name for perm in Permission if perm.value in allowed_ops}
 
 
 def move_file(bucket_name: str, source_file_key: str, destination_file_key: str) -> None:

recordprocessor/tests/test_recordprocessor_main.py

@@ -91,9 +91,6 @@ def make_inf_ack_assertions(self, file_details: FileDetails, passed_validation:
             "created_at_formatted_string", file_details.created_at_formatted_string
         )
 
-        print(f"actual_rows: {actual_rows,}")
-        print(f"expected_rows: {expected_row}")
-        
         self.assertEqual(actual_rows, [InfAckFileRows.HEADERS, expected_row])
 
     def make_kinesis_assertions(self, test_cases):