Include coverage reports from child pipeline

What does this MR do and why?

Include coverage reports from child pipelines when creating pipeline coverage report.

Part of #363301 (closed).

Screenshots or screen recordings

In this sample MR, alberts-gitlab/test-ruby-cobertura!2, there is a parent and child pipeline set up to generate coverage. The parent pipeline generates coverage for demo.rb. The child pipeline generates coverage for demo_child.rb.

Before - coverage from child pipeline is not included:

After - coverage from child pipeline is included:

Example project and MR: alberts-gitlab/test-ruby-cobertura!2

How to set up and validate locally

1. Clone project https://gitlab.com/alberts-gitlab/test-ruby-cobertura locally
2. Pull branch https://gitlab.com/alberts-gitlab/test-ruby-cobertura/-/tree/test-coverage-child-pipeline containing CI and test changes to generate coverage from child pipeline.
3. Create MR in GDK and wait for pipeline to succeed.
4. Verify that the demo_child.rb coverage generated in the child pipeline is not shown in the MR diff.
5. Enable feature flag ci_child_pipeline_coverage_reports.

   Feature.enable(:ci_child_pipeline_coverage_reports)

6. Run a new pipeline on the branch and wait for it to succeed.
7. Verify that the demo_child.rb coverage is now shown in the MR diff.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

• I have evaluated the MR acceptance checklist for this MR.

qa/qa/specs/features/api/4_verify/api_variable_inheritance_with_forward_pipeline_variables_spec.rb

@@ -29,20 +29,14 @@ module QA
           start_pipeline_via_api_with_variable(i + 1)
           wait_for_pipelines
 
-          # Is inheritable when true
-          expect(child1_pipeline).to have_variable(key: key, value: value),
-                                     "Expected to find `{key: 'TEST_VAR', value: 'This is great!'}`" \
-            " but got #{child1_pipeline.pipeline_variables}"
+          # When forward:pipeline_variables is true
+          expect_downstream_pipeline_to_inherit_variable
 
-          # Is not inheritable when false
-          expect(child2_pipeline).not_to have_variable(key: key, value: value),
-                                         "Did not expect to find `{key: 'TEST_VAR', value: 'This is great!'}`" \
-            " but got #{child2_pipeline.pipeline_variables}"
+          # When forward:pipeline_variables is false
+          expect_downstream_pipeline_not_to_inherit_variable(upstream_project, 'child2_trigger')
 
-          # Is not inheritable by default
-          expect(downstream1_pipeline).not_to have_variable(key: key, value: value),
-                                              "Did not expect to find `{key: 'TEST_VAR', value: 'This is great!'}`" \
-            " but got #{downstream1_pipeline.pipeline_variables}"
+          # When forward:pipeline_variables is default
+          expect_downstream_pipeline_not_to_inherit_variable(downstream1_project, 'downstream1_trigger')
         end
       end
 
@@ -84,6 +78,20 @@ def upstream_ci_file
           YAML
         }
       end
+
+      def expect_downstream_pipeline_to_inherit_variable
+        pipeline = downstream_pipeline(upstream_project, 'child1_trigger')
+        expect(pipeline).to have_variable(key: key, value: value),
+                            "Expected to find `{key: 'TEST_VAR', value: 'This is great!'}`" \
+            " but got #{pipeline.pipeline_variables}"
+      end
+
+      def expect_downstream_pipeline_not_to_inherit_variable(project, bridge_name)
+        pipeline = downstream_pipeline(project, bridge_name)
+        expect(pipeline).not_to have_variable(key: key, value: value),
+                                "Did not expect to find `{key: 'TEST_VAR', value: 'This is great!'}`" \
+            " but got #{pipeline.pipeline_variables}"
+      end
     end
   end
 end