fix: update error text to be consistent

Author: aarthy-dk

testgen/ui/views/connections.py

@@ -181,7 +181,7 @@ def on_setup_table_group_clicked(*_args) -> None:
                 connection.save()
                 message = "Changes have been saved successfully."
             except Exception as error:
-                message = "Error creating connection"
+                message = "Something went wrong while creating the connection."
                 success = False
                 LOG.exception(message)
 
@@ -269,7 +269,7 @@ def test_connection(self, connection: Connection) -> "ConnectionStatus":
                 details = error.args[0]
             return ConnectionStatus(message="Error attempting the connection.", details=details, successful=False)
         except Exception as error:
-            details = "Try again"
+            details = "Something went wrong while testing the connection."
             if connection.connect_by_key and not connection.private_key:
                 details = "The private key is missing."
             LOG.exception("Error testing database connection")
@@ -472,7 +472,7 @@ def on_close_clicked(_params: dict) -> None:
                         LOG.info("Table group %s created", table_group.id)
                         safe_rerun()
                 except Exception as error:
-                    message = "Error creating table group"
+                    message = "Something went wrong while creating the table group."
                     success = False
                     LOG.exception(message)
 

testgen/ui/views/dialogs/import_metadata_dialog.py

@@ -428,7 +428,7 @@ def on_file_cleared(_payload: dict) -> None:
             LOG.exception("Metadata import failed")
             result = {
                 "success": False,
-                "message": "Import failed due to an unexpected error. Please try again.",
+                "message": "Something went wrong while importing the metadata.",
             }
 
         st.session_state.pop(PREVIEW_SESSION_KEY, None)

testgen/ui/views/hygiene_issues.py

@@ -484,7 +484,7 @@ def source_data_dialog(selected_row):
     elif bad_data_status == "ERR":
         st.error(bad_data_msg)
     elif df_bad is None:
-        st.error("An unknown error was encountered.")
+        st.error("Something went wrong while loading the data.")
     else:
         if bad_data_msg:
             st.info(bad_data_msg)

testgen/ui/views/monitors_dashboard.py

@@ -625,7 +625,7 @@ def on_delete_confirmed(*_args) -> None:
             LOG.exception("Failed to delete monitor suite")
             set_result({
                 "success": False,
-                "message": "Unable to delete monitors for the table group, try again.",
+                "message": "Something went wrong while deleting the monitors.",
             })
             safe_rerun(scope="fragment")
 

testgen/ui/views/profiling_runs.py

@@ -229,6 +229,6 @@ def on_delete_confirmed(*_args) -> None:
             LOG.exception("Failed to delete profiling runs")
             set_result({
                 "success": False,
-                "message": "Unable to delete the selected profiling runs, try again.",
+                "message": "Something went wrong while deleting the profiling runs.",
             })
             safe_rerun(scope="fragment")

testgen/ui/views/score_details.py

@@ -243,7 +243,7 @@ def recalculate_score_history(definition_id: str) -> None:
         st.toast("Scorecard trend recalculated", icon=":material/task_alt:")
     except:
         LOG.exception(f"Failure recalculating history for scorecard id={definition_id}")
-        st.toast("Recalculating the trend failed. Try again", icon=":material/error:")
+        st.toast("Something went wrong while recalculating the trend.", icon=":material/error:")
 
 
 class ScoreDropNotificationSettingsDialog(NotificationSettingsDialogBase):

testgen/ui/views/table_groups.py

@@ -341,6 +341,7 @@ def on_close_clicked(_params: dict) -> None:
                         safe_rerun()
 
                 except IntegrityError:
+                    get_current_session().rollback()
                     success = False
                     message = "A Table Group with the same name already exists."
             else:

testgen/ui/views/test_results.py

@@ -890,7 +890,7 @@ def source_data_dialog(selected_row):
     elif bad_data_status == "ERR":
         st.error(bad_data_msg)
     elif df_bad is None:
-        st.error("An unknown error was encountered.")
+        st.error("Something went wrong while loading the data.")
     else:
         if bad_data_msg:
             st.info(bad_data_msg)

testgen/ui/views/test_runs.py

@@ -246,5 +246,5 @@ def on_delete_confirmed(*_args) -> None:
             safe_rerun()
         except Exception:
             LOG.exception("Failed to delete test run")
-            result = {"success": False, "message": "Unable to delete the test run, try again."}
+            result = {"success": False, "message": "Something went wrong while deleting the test run."}
             safe_rerun(scope="fragment")