[ML] Fixing jest tests with unhandled promise rejections (#112804)

elastic · Sep 22, 2021 · 1456257 · 1456257
1 parent 84e42e4
commit 1456257
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/x-pack/plugins/ml/public/application/components/validate_job/validate_job_view.test.js b/x-pack/plugins/ml/public/application/components/validate_job/validate_job_view.test.js
@@ -41,7 +41,7 @@ function prepareTest(messages) {
   };
   const kibana = {
     services: {
-      notifications: { toasts: { addDanger: jest.fn() } },
+      notifications: { toasts: { addDanger: jest.fn(), addError: jest.fn() } },
     },
   };
 

diff --git a/x-pack/plugins/ml/public/application/explorer/explorer_charts/explorer_charts_container.js b/x-pack/plugins/ml/public/application/explorer/explorer_charts/explorer_charts_container.js
@@ -79,8 +79,12 @@ function ExplorerChartContainer({
     let isCancelled = false;
     const generateLink = async () => {
       if (!isCancelled && series.functionDescription !== ML_JOB_AGGREGATION.LAT_LONG) {
-        const singleMetricViewerLink = await getExploreSeriesLink(mlLocator, series, timefilter);
-        setExplorerSeriesLink(singleMetricViewerLink);
+        try {
+          const singleMetricViewerLink = await getExploreSeriesLink(mlLocator, series, timefilter);
+          setExplorerSeriesLink(singleMetricViewerLink);
+        } catch (error) {
+          setExplorerSeriesLink('');
+        }
       }
     };
     generateLink();

diff --git a/x-pack/plugins/ml/server/models/bucket_span_estimator/bucket_span_estimator.test.ts b/x-pack/plugins/ml/server/models/bucket_span_estimator/bucket_span_estimator.test.ts
@@ -12,6 +12,10 @@ import { BucketSpanEstimatorData } from '../../../common/types/job_service';
 
 import { estimateBucketSpanFactory } from './bucket_span_estimator';
 
+jest.mock('../../lib/log', () => ({
+  mlLog: { warn: jest.fn() },
+}));
+
 const callAs = {
   search: () => Promise.resolve({ body: {} }),
   cluster: { getSettings: () => Promise.resolve({ body: {} }) },