Expose metadata_files parameter in coverage_common.instrumented_files_info

Allows all rules to provide additional metadata files to the
InstrumentedFilesInfo provider.

It was always a little odd that the metadata_files attribute on the provider
was exposed, but there was no way of populating it from Starlark.

The other parameters are still considered too tied up with internal details at
the moment to be exposed at this time.

RELNOTES: coverage_common.instrumented_files_info now has a metadata_files argument
PiperOrigin-RevId: 545391150
Change-Id: I3febdaf23c82d60c32b241a3935da27fd086fb7a

Author: c-mita

src/main/java/com/google/devtools/build/lib/analysis/test/CoverageCommon.java

@@ -100,10 +100,9 @@ public InstrumentedFilesInfoApi instrumentedFilesInfo(
       // Should have been verified by Starlark before this function is called
       throw new IllegalStateException();
     }
-    if (!supportFilesBuilder.isEmpty() || !environmentPairs.isEmpty()) {
-      BuiltinRestriction.throwIfNotBuiltinUsage(thread);
-    }
-    if (!metadataFiles.isEmpty() || !reportedToActualSources.isEmpty()) {
+    if (!supportFilesBuilder.isEmpty()
+        || !reportedToActualSources.isEmpty()
+        || !environmentPairs.isEmpty()) {
       BuiltinRestriction.throwIfNotBuiltinUsage(thread);
     }
     return createInstrumentedFilesInfo(

src/main/java/com/google/devtools/build/lib/starlarkbuildapi/test/CoverageCommonApi.java

@@ -99,9 +99,11 @@ public interface CoverageCommonApi<
             defaultValue = "None"),
         @Param(
             name = "metadata_files",
+            doc =
+                "Additional files required to generate coverage LCOV files after code execution."
+                    + " e.g. .gcno files for C++.",
             named = true,
             positional = false,
-            documented = false,
             defaultValue = "[]",
             allowedTypes = {
               @ParamType(type = Sequence.class, generic1 = FileApi.class),

src/test/java/com/google/devtools/build/lib/starlark/StarlarkIntegrationTest.java

@@ -948,6 +948,8 @@ public void testInstrumentedFilesInfo_coverageEnabled() throws Exception {
         "load('//myinfo:myinfo.bzl', 'MyInfo')",
         "",
         "def custom_rule_impl(ctx):",
+        "    metadata = ctx.actions.declare_file(ctx.label.name + '.metadata')",
+        "    ctx.actions.write(metadata, '')",
         "    return [",
         "        coverage_common.instrumented_files_info(",
         "            ctx,",
@@ -967,6 +969,7 @@ public void testInstrumentedFilesInfo_coverageEnabled() throws Exception {
         // Missing attrs are ignored
         "                'missing_dep_attr',",
         "            ],",
+        "            metadata_files = [metadata],",
         "        ),",
         "    ]",
         "",
@@ -1033,7 +1036,7 @@ public void testInstrumentedFilesInfo_coverageEnabled() throws Exception {
     assertThat(
             ActionsTestUtil.baseArtifactNames(
                 ((Depset) myInfo.getValue("metadata_files")).getSet(Artifact.class)))
-        .containsExactly("label_dep.gcno", "label_list_dep.gcno", "dict_dep.gcno");
+        .containsExactly("label_dep.gcno", "label_list_dep.gcno", "dict_dep.gcno", "cr.metadata");
     ConfiguredTarget customRule = getConfiguredTarget("//test/starlark:cr");
     assertThat(
             ActionsTestUtil.baseArtifactNames(