Changed usage of .lower().capitalize() to just .capitalize. Also updated the query string in get_prov_info when status='QA

is selected.

Author: DerekFurstPitt

src/app_neo4j_queries.py

@@ -954,14 +954,13 @@ def get_prov_info(neo4j_driver, param_dict):
             rui_info_where_clause = "WHERE NOT EXISTS {MATCH (ds)<-[*]-(ruiSample:Sample) WHERE NOT ruiSample.rui_location IS NULL AND NOT TRIM(ruiSample.rui_location) = ''} MATCH (ds)<-[*]-(ruiSample:Sample)"
     if 'dataset_status' in param_dict:
         if first_param:
-            print("true")
-            dataset_status_query_string = f" WHERE ds.status = '{param_dict['dataset_status'].lower().capitalize()}'"
+            dataset_status_query_string = f" WHERE ds.status = '{param_dict['dataset_status'].capitalize()}'"
             if param_dict['dataset_status'].lower() == "qa":
-                dataset_status_query_string = f" WHERE ds.status = '{param_dict['dataset_status'].upper()}'"
+                dataset_status_query_string = f" WHERE ds.status = 'QA'"
         else:
-            dataset_status_query_string = f" AND ds.status = '{param_dict['dataset_status'].lower().capitalize()}'"
+            dataset_status_query_string = f" AND ds.status = '{param_dict['dataset_status'].capitalize()}'"
             if param_dict['dataset_status'].lower() == "qa":
-                dataset_status_query_string = f" AND ds.status = '{param_dict['dataset_status'].upper()}'"
+                dataset_status_query_string = f" AND ds.status = 'QA'"
     query = (f"MATCH (ds:Dataset)<-[:ACTIVITY_OUTPUT]-(a)<-[:ACTIVITY_INPUT]-(firstSample:Sample)<-[*]-(donor:Donor)"
              f"{group_uuid_query_string}"
              f"{dataset_status_query_string}"
@@ -981,7 +980,6 @@ def get_prov_info(neo4j_driver, param_dict):
              f" ds.last_modified_user_email, ds.lab_dataset_id, ds.data_types, METASAMPLE, PROCESSED_DATASET")
     logger.debug("======get_prov_info() query======")
     logger.debug(query)
-    print(query)
     with neo4j_driver.session() as session:
         # Because we're returning multiple things, we use session.run rather than session.read_transaction
         result = session.run(query)