Merge branch 'aarthy/fixes' into 'enterprise'

Bug fixes on Monitors

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

Author: ci bot

testgen/commands/run_profiling.py

@@ -250,6 +250,7 @@ def update_frequency_progress(progress: ThreadedProgress) -> None:
                 LOG.info("Updating profiling results with frequency analysis and deleting staging")
                 execute_db_queries(sql_generator.update_frequency_analysis_results())
     except Exception as e:
+        LOG.exception("Error running frequency analysis")
         profiling_run.set_progress("freq_analysis", "Warning", error=f"Error encountered. {get_exception_message(e)}")
     else:
         if error_data:
@@ -290,6 +291,7 @@ def _run_hygiene_issue_detection(sql_generator: ProfilingSQL) -> None:
             ]
         )
     except Exception as e:
+        LOG.exception("Error detecting hygiene issues")
         profiling_run.set_progress("hygiene_issues", "Warning", error=f"Error encountered. {get_exception_message(e)}")
     else:
         profiling_run.set_progress("hygiene_issues", "Completed")

testgen/ui/components/frontend/js/components/table.js

@@ -201,7 +201,7 @@ const Table = (options, rows) => {
                             {class: 'tg-table-empty-state-body'},
                             tr(
                                 td(
-                                    {colspan: dataColumns.length},
+                                    {colspan: dataColumns.val.length},
                                     options.emptyState,
                                 ),
                             ),

testgen/ui/components/frontend/js/pages/monitors_dashboard.js

@@ -295,7 +295,7 @@ const MonitorsDashboard = (/** @type Properties */ props) => {
                         onTagClick: (type) => {
                             const current = anomalyTypeFilterValue.val;
                             const newFilter = current.length === 1 && current[0] === type ? null : type;
-                            emitEvent('SetParamValues', { payload: { anomaly_type_filter: newFilter } });
+                            emitEvent('SetParamValues', { payload: { anomaly_type_filter: newFilter, current_page: 0 } });
                         },
                         activeTypes: anomalyTypeFilterValue,
                     })
@@ -347,7 +347,7 @@ const MonitorsDashboard = (/** @type Properties */ props) => {
                             icon: 'search',
                             testId: 'search-tables',
                             value: tableNameFilterValue,
-                            onChange: (value, state) => emitEvent('SetParamValues', {payload: {table_name_filter: value}}),
+                            onChange: (value, state) => emitEvent('SetParamValues', {payload: {table_name_filter: value, current_page: 0}}),
                         }),
                         Select({
                             label: 'Anomaly type',
@@ -361,7 +361,7 @@ const MonitorsDashboard = (/** @type Properties */ props) => {
                             multiSelect: true,
                             width: 200,
                             onChange: (values) => emitEvent('SetParamValues', {
-                                payload: { anomaly_type_filter: values.length ? values.join(',') : null },
+                                payload: { anomaly_type_filter: values.length ? values.join(',') : null, current_page: 0 },
                             }),
                         }),
                         span({class: 'fx-flex'}, ''),

testgen/ui/views/monitors_dashboard.py

@@ -101,7 +101,7 @@ def render(
                         JobSchedule.kwargs["test_suite_id"].astext == str(monitor_suite_id),
                     )
 
-                    anomaly_type_filter = [t for t in anomaly_type_filter.split(",") if t in ANOMALY_TYPE_FILTERS] if anomaly_type_filter else []
+                    anomaly_type_filter = [t for t in anomaly_type_filter.split(",") if t in ANOMALY_TYPE_FILTERS] if anomaly_type_filter else None
                     if sort_field and sort_field not in ALLOWED_SORT_FIELDS:
                         sort_field = None
 
@@ -148,7 +148,7 @@ def render(
                 "filters": {
                     "table_group_id": table_group_id,
                     "table_name_filter": table_name_filter,
-                    "anomaly_type_filter": list(anomaly_type_filter),
+                    "anomaly_type_filter": list(anomaly_type_filter) if anomaly_type_filter else None,
                 },
                 "sort": {
                     "sort_field": sort_field,