1 files changed, 21 insertions, 24 deletions

diff --git a/app/assets/javascripts/pipelines/stores/test_reports/actions.js b/app/assets/javascripts/pipelines/stores/test_reports/actions.js
index ccacb9f7e97..f10bbeec77c 100644
--- a/app/assets/javascripts/pipelines/stores/test_reports/actions.js
+++ b/app/assets/javascripts/pipelines/stores/test_reports/actions.js
@@ -1,46 +1,46 @@
 import axios from '~/lib/utils/axios_utils';
 import * as types from './mutation_types';
-import createFlash from '~/flash';
+import { deprecatedCreateFlash as createFlash } from '~/flash';
 import { s__ } from '~/locale';
 
 export const fetchSummary = ({ state, commit, dispatch }) => {
-  // If we do this without the build_report_summary feature flag enabled
-  // it causes a race condition for toggleLoading and ruins the loading
-  // state in the application
-  if (state.useBuildSummaryReport) {
-    dispatch('toggleLoading');
-  }
+  dispatch('toggleLoading');
 
   return axios
     .get(state.summaryEndpoint)
     .then(({ data }) => {
       commit(types.SET_SUMMARY, data);
-
-      if (!state.useBuildSummaryReport) {
-        // Set the tab counter badge to total_count
-        // This is temporary until we can server-side render that count number
-        // (see https://gitlab.com/gitlab-org/gitlab/-/issues/223134)
-        document.querySelector('.js-test-report-badge-counter').innerHTML = data.total_count || 0;
-      }
     })
     .catch(() => {
       createFlash(s__('TestReports|There was an error fetching the summary.'));
     })
     .finally(() => {
-      if (state.useBuildSummaryReport) {
-        dispatch('toggleLoading');
-      }
+      dispatch('toggleLoading');
     });
 };
 
-export const fetchFullReport = ({ state, commit, dispatch }) => {
+export const fetchTestSuite = ({ state, commit, dispatch }, index) => {
+  const { hasFullSuite } = state.testReports?.test_suites?.[index] || {};
+  // We don't need to fetch the suite if we have the information already
+  if (hasFullSuite) {
+    return Promise.resolve();
+  }
+
   dispatch('toggleLoading');
 
+  const { name = '', build_ids = [] } = state.testReports?.test_suites?.[index] || {};
+  // Replacing `/:suite_name.json` with the name of the suite. Including the extra characters
+  // to ensure that we replace exactly the template part of the URL string
+  const endpoint = state.suiteEndpoint?.replace(
+    '/:suite_name.json',
+    `/${encodeURIComponent(name)}.json`,
+  );
+
   return axios
-    .get(state.fullReportEndpoint)
-    .then(({ data }) => commit(types.SET_REPORTS, data))
+    .get(endpoint, { params: { build_ids } })
+    .then(({ data }) => commit(types.SET_SUITE, { suite: data, index }))
     .catch(() => {
-      createFlash(s__('TestReports|There was an error fetching the test reports.'));
+      createFlash(s__('TestReports|There was an error fetching the test suite.'));
     })
     .finally(() => {
       dispatch('toggleLoading');
@@ -52,6 +52,3 @@ export const setSelectedSuiteIndex = ({ commit }, data) =>
 export const removeSelectedSuiteIndex = ({ commit }) =>
   commit(types.SET_SELECTED_SUITE_INDEX, null);
 export const toggleLoading = ({ commit }) => commit(types.TOGGLE_LOADING);
-
-// prevent babel-plugin-rewire from generating an invalid default during karma tests
-export default () => {};