Merge branch 'fix/flaky-jest-spec-links-layer' into 'master'

Fix flaky jest spec in links_layer_spec.js

See merge request gitlab-org/gitlab!61178

app/assets/javascripts/pipelines/components/graph_shared/api.js

@@ -2,6 +2,11 @@ import axios from '~/lib/utils/axios_utils';
 import { reportToSentry } from '../../utils';
 
 export const reportPerformance = (path, stats) => {
+  // FIXME: https://gitlab.com/gitlab-org/gitlab/-/issues/330245
+  if (!path) {
+    return;
+  }
+
   axios.post(path, stats).catch((err) => {
     reportToSentry('links_inner_perf', `error: ${err}`);
   });

spec/frontend/pipelines/graph_shared/links_layer_spec.js

@@ -96,23 +96,19 @@ describe('links layer component', () => {
   });
 
   describe('performance metrics', () => {
+    const metricsPath = '/root/project/-/ci/prometheus_metrics/histograms.json';
     let markAndMeasure;
     let reportToSentry;
     let reportPerformance;
     let mock;
 
     beforeEach(() => {
-      mock = new MockAdapter(axios);
       jest.spyOn(window, 'requestAnimationFrame').mockImplementation((cb) => cb());
       markAndMeasure = jest.spyOn(perfUtils, 'performanceMarkAndMeasure');
       reportToSentry = jest.spyOn(sentryUtils, 'reportToSentry');
       reportPerformance = jest.spyOn(Api, 'reportPerformance');
     });
 
-    afterEach(() => {
-      mock.restore();
-    });
-
     describe('with no metrics config object', () => {
       beforeEach(() => {
         createComponent();
@@ -164,7 +160,6 @@ describe('links layer component', () => {
     });
 
     describe('with metrics path and collect set to true', () => {
-      const metricsPath = '/root/project/-/ci/prometheus_metrics/histograms.json';
       const duration = 875;
       const numLinks = 7;
       const totalGroups = 8;
@@ -204,6 +199,9 @@ describe('links layer component', () => {
 
       describe('with duration and no error', () => {
         beforeEach(() => {
+          mock = new MockAdapter(axios);
+          mock.onPost(metricsPath).reply(200, {});
+
           jest.spyOn(window.performance, 'getEntriesByName').mockImplementation(() => {
             return [{ duration }];
           });
@@ -218,6 +216,10 @@ describe('links layer component', () => {
           });
         });
 
+        afterEach(() => {
+          mock.restore();
+        });
+
         it('it calls reportPerformance with expected arguments', () => {
           expect(markAndMeasure).toHaveBeenCalled();
           expect(reportPerformance).toHaveBeenCalled();