[7.1.0] Remove unnecessary cc_test coverage handling (#20641)

Both the `lcov_merger` and the `collect_cc_coverage` script are obtained
from "well-known" implicit attributes of Starlark rules and do not
require any further special handling.

Removing this special handling also fixes a fringe bug: When a `cc_test`
is wrapped in another test target that applies a transition to it, the
removed logic set the `LCOV_MERGER` variable to the untransitioned path
of the `lcov_merger`, which will then not be available in the test
sandbox since it is only added as a runfile, but the test action
references it via its exec path. This resulted in a "file not found"
error that broke coverage.

Closes #20349.

Commit
591d125

PiperOrigin-RevId: 589252093
Change-Id: I810f1b414dcfedcc8930a74390ef5c9bfd2cd030

Co-authored-by: Fabian Meumertzheim <[email protected]>
Co-authored-by: Ian (Hee) Cha <[email protected]>

src/main/starlark/builtins_bzl/common/cc/semantics.bzl

@@ -86,35 +86,7 @@ def _get_coverage_attrs():
     }
 
 def _get_coverage_env(ctx):
-    runfiles = ctx.runfiles()
-    test_env = {}
-    if ctx.configuration.coverage_enabled:
-        # Bazel’s coverage runner
-        # (https://github.com/bazelbuild/bazel/blob/3.0.0/tools/test/collect_coverage.sh)
-        # needs a binary called “lcov_merge.”  Its location is passed in the
-        # LCOV_MERGER environmental variable.  For builtin rules, this variable
-        # is set automatically based on a magic “$lcov_merger” or
-        # “:lcov_merger” attribute, but it’s not possible to create such
-        # attributes in Starlark.  Therefore we specify the variable ourselves.
-        # Note that the coverage runner runs in the runfiles root instead of
-        # the execution root, therefore we use “path” instead of “short_path.”
-        test_env["LCOV_MERGER"] = ctx.executable._lcov_merger.path
-
-        # C/C++ coverage instrumentation needs another binary that wraps gcov;
-        # see
-        # https://github.com/bazelbuild/bazel/blob/5.0.0/tools/test/collect_coverage.sh#L199.
-        # This is normally set from a hidden “$collect_cc_coverage” attribute;
-        # see
-        # https://github.com/bazelbuild/bazel/blob/5.0.0/src/main/java/com/google/devtools/build/lib/analysis/test/TestActionBuilder.java#L253-L258.
-        test_env["CC_CODE_COVERAGE_SCRIPT"] = ctx.executable._collect_cc_coverage.path
-
-        # The test runfiles need all applicable runfiles for the tools above.
-        runfiles = runfiles.merge_all([
-            ctx.attr._lcov_merger[DefaultInfo].default_runfiles,
-            ctx.attr._collect_cc_coverage[DefaultInfo].default_runfiles,
-        ])
-
-    return runfiles, test_env
+    return ctx.runfiles(), {}
 
 def _get_cc_runtimes(ctx, is_library):
     if is_library: