Merge branch 'fix-test' into 'enterprise'

fix(tests): bugs related to testing

See merge request dkinternal/testgen/dataops-testgen!217

Author: ci bot

testgen/commands/run_execute_cat_tests.py

@@ -71,16 +71,20 @@ def FinalizeTestRun(clsCATExecute: CCATExecutionSQL):
     ])
     end_time = datetime.now(UTC)
 
-    RunActionQueryList(("DKTG"), [
-        clsCATExecute.CalcPrevalenceTestResultsSQL(),
-        clsCATExecute.TestScoringRollupRunSQL(),
-        clsCATExecute.TestScoringRollupTableGroupSQL(),
-    ])
-    run_refresh_score_cards_results(
-        project_code=clsCATExecute.project_code,
-        add_history_entry=True,
-        refresh_date=date_service.parse_now(clsCATExecute.run_date),
-    )
+    try:
+        RunActionQueryList(("DKTG"), [
+            clsCATExecute.CalcPrevalenceTestResultsSQL(),
+            clsCATExecute.TestScoringRollupRunSQL(),
+            clsCATExecute.TestScoringRollupTableGroupSQL(),
+        ])
+        run_refresh_score_cards_results(
+            project_code=clsCATExecute.project_code,
+            add_history_entry=True,
+            refresh_date=date_service.parse_now(clsCATExecute.run_date),
+        )
+    except Exception:
+        LOG.exception("Error refreshing scores after test run")
+        pass
 
     MixpanelService().send_event(
         "run-tests",

testgen/template/validate_tests/ex_get_test_column_list_tg.sql

@@ -24,6 +24,7 @@
          WHERE test_suite_id = '{TEST_SUITE_ID}'
          AND COALESCE(test_active, 'Y') = 'Y'
          AND t.test_scope = 'referential'
+         AND t.test_type NOT LIKE 'Aggregate_%'
          UNION
          -- FROM: groupby_names (should be referential)
          SELECT cat_test_id,
@@ -60,6 +61,7 @@
          WHERE test_suite_id = '{TEST_SUITE_ID}'
          AND COALESCE(test_active, 'Y') = 'Y'
          AND t.test_scope = 'referential'
+         AND t.test_type NOT LIKE 'Aggregate_%'
          UNION
          -- FROM: match_groupby_names (referential)
          SELECT cat_test_id,

testgen/ui/views/test_results.py

@@ -381,6 +381,10 @@ def get_test_result_history(selected_row):
 
 
 def show_test_def_detail(str_test_def_id):
+    if not str_test_def_id:
+        st.warning("Test definition no longer exists.")
+        return
+    
     df = get_test_definition(str_test_def_id)
 
     specs = []
@@ -758,9 +762,10 @@ def source_data_dialog(selected_row):
 
 
 def view_edit_test(button_container, test_definition_id):
-    with button_container:
-        if st.button(":material/edit: Edit Test", help="Edit the Test Definition", use_container_width=True):
-            show_test_form_by_id(test_definition_id)
+    if test_definition_id:
+        with button_container:
+            if st.button(":material/edit: Edit Test", help="Edit the Test Definition", use_container_width=True):
+                show_test_form_by_id(test_definition_id)
 
 
 def get_report_file_data(update_progress, tr_data) -> FILE_DATA_TYPE: