Removed test logging

Author: ThomasC-Kainos

README.md

@@ -1,6 +1,5 @@
 # DoS Integration
 
-
 ## Table of Contents
 
 - [DoS Integration](#dos-integration)

application/ingest_change_event/change_event_validation.py

@@ -58,8 +58,6 @@ def validate_organisation_keys(org_type_id: str, org_sub_type: str) -> None:
         ValidationError: Either Org Type ID or Org Sub Type is not part of the valid list
     """
     validate_organisation_type_id(org_type_id)
-    #logger.info("test log: ", len(PHARMACY_ORGANISATION_SUB_TYPES))
-    #logger.info("test log: ", PHARMACY_ORGANISATION_SUB_TYPES)
     if org_sub_type in PHARMACY_ORGANISATION_SUB_TYPES:
         logger.info(f"Subtype type id: {org_sub_type} validated")
     else:

application/service_sync/data_processing/check_for_change.py

@@ -63,7 +63,6 @@ def compare_nhs_uk_and_dos_data(
     Returns:
         ChangesToDoS: ChangesToDoS class with all the flags if changes need to be made and the changes to make
     """
-
     # Set up the holder class
     changes_to_dos = ChangesToDoS(dos_service=dos_service, nhs_entity=nhs_entity, service_histories=service_histories)
 
@@ -80,8 +79,6 @@ def compare_nhs_uk_and_dos_data(
     # Compare blood pressure
     changes_to_dos = check_blood_pressure_for_change(changes_to_dos=changes_to_dos)
     # Compare contraception
-    logger.warning(f"TEST LOG check for change {changes_to_dos.nhs_entity.org_sub_type}")
-
     return check_contraception_for_change(changes_to_dos=changes_to_dos)
 
 
@@ -415,22 +412,3 @@ def status_id_change(changes_to_dos: ChangesToDoS, new_value: bool, previous_val
         ),
     )
     return changes_to_dos
-
-def check_org_for_change(changes_to_dos: ChangesToDoS) -> ChangesToDoS:
-    """Compares and creates change for website if needed.
-
-    Args:
-        changes_to_dos (ChangesToDoS): ChangesToDoS holder object
-
-    Returns:
-        ChangesToDoS: ChangesToDoS holder object
-    """
-    if compare_website(changes=changes_to_dos):
-        changes_to_dos = services_change(
-            changes_to_dos=changes_to_dos,
-            change_key=DOS_WEBSITE_CHANGE_KEY,
-            new_value=changes_to_dos.new_website,
-            previous_value=changes_to_dos.current_website,
-            service_table_field_name="web",
-        )
-    return changes_to_dos

application/service_sync/service_sync.py

@@ -54,21 +54,15 @@ def lambda_handler(event: SQSEvent, context: LambdaContext) -> None:  # noqa: AR
             nhs_entity=nhs_entity,
             service_histories=service_histories,
         )
-
-        logger.warning(f"TEST LOG {changes_to_dos.nhs_entity.org_sub_type}")
-
-        logger.warning(f"TEST LOG22 {changes_to_dos}")
         # Update Service History with changes to be made
         service_histories = changes_to_dos.service_histories
         # Update DoS data
         update_dos_data(changes_to_dos=changes_to_dos, service_id=int(service_id), service_histories=service_histories)
         # Delete the message from the queue
         remove_sqs_message_from_queue(receipt_handle=record.receipt_handle)
-        logger.warning(f"final log {changes_to_dos}")
-
         # Log custom metrics
         logger.warning(
-            "Update Request Success TEST",
+            "Update Request Success",
             latency=(time_ns() // 1000000)
             - int(record.message_attributes.get("message_received", {}).get("stringValue")),
             environment=getenv("ENVIRONMENT"),