Always fetch pipeline test report

Reverting a change where we prematurely
checked if a pipeline had a test report before
we fetched the test summary. This was not
reliable when a pipeline was still running, but
tests should still be able to be seen.

Changelog: fixed

app/assets/javascripts/pipelines/components/test_reports/test_reports.vue

@@ -15,11 +15,6 @@ export default {
     TestSummary,
     TestSummaryTable,
   },
-  inject: {
-    hasTestReport: {
-      default: false,
-    },
-  },
   computed: {
     ...mapState(['isLoading', 'selectedSuiteIndex', 'testReports']),
     ...mapGetters(['getSelectedSuite']),
@@ -32,9 +27,7 @@ export default {
     },
   },
   created() {
-    if (this.hasTestReport) {
-      this.fetchSummary();
-    }
+    this.fetchSummary();
   },
   methods: {
     ...mapActions([

changelogs/unreleased/329206-fix-test-report.yml

+---
+title: Fix bug where test report was not showing while pipeline was running
+merge_request: 60416
+author:
+type: fixed

spec/frontend/pipelines/test_reports/test_reports_spec.js

@@ -31,7 +31,7 @@ describe('Test reports app', () => {
     removeSelectedSuiteIndex: jest.fn(),
   };
 
-  const createComponent = ({ state = {}, hasTestReport = true } = {}) => {
+  const createComponent = ({ state = {} } = {}) => {
     store = new Vuex.Store({
       state: {
         isLoading: false,
@@ -47,9 +47,6 @@ describe('Test reports app', () => {
       shallowMount(TestReports, {
         store,
         localVue,
-        provide: {
-          hasTestReport,
-        },
       }),
     );
   };
@@ -64,12 +61,6 @@ describe('Test reports app', () => {
 
       expect(actionSpies.fetchSummary).toHaveBeenCalled();
     });
-
-    it('should not call fetchSummary when pipeline does not have test report', () => {
-      createComponent({ state: {}, hasTestReport: false });
-
-      expect(actionSpies.fetchSummary).not.toHaveBeenCalled();
-    });
   });
 
   describe('when loading', () => {