commit 0e71e901d0ccf59b558e9ef6e5f4f5cec5f5156c
author ajurkowski <ajurkowski@google.com> Fri Jun 11 10:58:16 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Jun 11 10:59:55 2021 -0700
tree 78e0f2ef3de547eb4704616cea00c014424499bf
parent 060d5966adf2c8035276cd1159601567bea57757

Preserve `PathFragment` passed to `ActionInputHelper::fromPath(PathFragment)`.

`ActionInputHelper::fromPath` allows to create an action input from an exec path or a string representing it. In case of `PathFragment`, we will get the path string of it and recreate a new `PathFragment` in `getExecPath`. Store the `PathFragment` instead. Change callers which already have a `PathFragment` to use the override accepting it.

PiperOrigin-RevId: 378907363

src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java

diff --git a/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java b/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java
index 5a51de8..15e1d58 100644
--- a/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java
+++ b/src/main/java/com/google/devtools/build/lib/exec/StandaloneTestStrategy.java
@@ -472,7 +472,7 @@
             .build(),
         /* tools= */ NestedSetBuilder.emptySet(Order.STABLE_ORDER),
         /* outputs= */ ImmutableSet.of(
-            ActionInputHelper.fromPath(action.getCoverageData().getExecPathString())),
+            ActionInputHelper.fromPath(action.getCoverageData().getExecPath())),
         SpawnAction.DEFAULT_RESOURCE_SET);
   }