Merge branch '254308-fix-query-params-for-test-coverage-report' into 'master'

Fix the project ID query param

See merge request gitlab-org/gitlab!42880

ee/app/assets/javascripts/analytics/repository_analytics/components/group_repository_analytics.vue

@@ -79,7 +79,7 @@ export default {
 
       // not including a project_ids param is the same as selecting all the projects
       if (!this.selectAllProjects) {
-        queryParams.set('project_ids', this.selectedProjectIdsParam);
+        this.selectedProjectIds.forEach(id => queryParams.append('project_ids[]', id));
       }
 
       return `${this.groupAnalyticsCoverageReportsPath}&${queryParams.toString()}`;
@@ -105,7 +105,7 @@ export default {
         project.name.toLowerCase().includes(this.projectSearchTerm.toLowerCase()),
       );
     },
-    selectedProjectIdsParam() {
+    selectedProjectIds() {
       return this.groupProjects.filter(project => project.isSelected).map(project => project.id);
     },
   },

ee/app/assets/javascripts/analytics/repository_analytics/graphql/queries/get_group_projects.query.graphql

 query getGroupProjects($groupFullPath: ID!) {
   group(fullPath: $groupFullPath) {
-    projects(includeSubgroups: true) {
+    projects {
       nodes {
         name
         id

ee/changelogs/unreleased/254308-fix-query-params-for-test-coverage-report.yml

+---
+title: Fix project_ids query param for group test coverage report
+merge_request: 42880
+author:
+type: fixed

ee/spec/frontend/analytics/repository_analytics/components/group_repository_analytics_spec.js

@@ -89,7 +89,8 @@ describe('Group repository analytics app', () => {
         });
 
         it('renders primary action as a link with two project IDs as parameters', () => {
-          const expectedPath = `${groupAnalyticsCoverageReportsPathWithDates}&project_ids=${groupProjectsData[0].id}%2C${groupProjectsData[1].id}`;
+          const projectIdsQueryParam = `project_ids%5B%5D=${groupProjectsData[0].id}&project_ids%5B%5D=${groupProjectsData[1].id}`;
+          const expectedPath = `${groupAnalyticsCoverageReportsPathWithDates}&${projectIdsQueryParam}`;
 
           expect(findCodeCoverageDownloadButton().attributes('href')).toBe(expectedPath);
         });
@@ -101,7 +102,8 @@ describe('Group repository analytics app', () => {
         });
 
         it('renders primary action as a link with one project ID as a parameter', () => {
-          const expectedPath = `${groupAnalyticsCoverageReportsPathWithDates}&project_ids=${groupProjectsData[0].id}`;
+          const projectIdsQueryParam = `project_ids%5B%5D=${groupProjectsData[0].id}`;
+          const expectedPath = `${groupAnalyticsCoverageReportsPathWithDates}&${projectIdsQueryParam}`;
 
           expect(findCodeCoverageDownloadButton().attributes('href')).toBe(expectedPath);
         });