Remove test methods from ArtifactFactory and Root that violate root invariants.

In this case, the invariant violated is creating derived roots at the exec root.

PiperOrigin-RevId: 181994080

2 files changed, 0 insertions, 27 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/ArtifactFactory.java b/src/main/java/com/google/devtools/build/lib/actions/ArtifactFactory.java
index 569d236b18..37bd6f48b6 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/ArtifactFactory.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/ArtifactFactory.java
@@ -157,24 +157,6 @@ public class ArtifactFactory implements ArtifactResolver {
     return getSourceArtifact(execPath, root, ArtifactOwner.NULL_OWNER);
   }
 
-  /**
-   * Only for use by BinTools! Returns an artifact for a tool at the given path
-   * fragment, relative to the exec root, creating it if not found. This method
-   * only works for normalized, relative paths.
-   */
-  public Artifact getDerivedArtifact(PathFragment execPath, Path execRoot) {
-    Preconditions.checkArgument(!execPath.isAbsolute(), execPath);
-    Preconditions.checkArgument(execPath.isNormalized(), execPath);
-    // TODO(bazel-team): Check that either BinTools do not change over the life of the Blaze server,
-    // or require that a legitimate ArtifactOwner be passed in here to allow for ownership.
-    return getArtifact(
-        execRoot.getRelative(execPath),
-        Root.execRootAsDerivedRoot(execRoot),
-        execPath,
-        ArtifactOwner.NULL_OWNER,
-        null);
-  }
-
   private void validatePath(PathFragment rootRelativePath, Root root) {
     Preconditions.checkArgument(!root.isSourceRoot());
     Preconditions.checkArgument(!rootRelativePath.isAbsolute(), rootRelativePath);
diff --git a/src/main/java/com/google/devtools/build/lib/actions/Root.java b/src/main/java/com/google/devtools/build/lib/actions/Root.java
index e6451c9e41..5718478327 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/Root.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/Root.java
@@ -102,15 +102,6 @@ public final class Root implements Comparable<Root>, Serializable, SkylarkValue
     return new Root(execRoot, root, true);
   }
 
-  /**
-   * Returns the exec root as a derived root. The exec root should never be treated as a derived
-   * root, but this is currently allowed. Do not add any further uses besides the ones that already
-   * exist!
-   */
-  static Root execRootAsDerivedRoot(Path execRoot) {
-    return new Root(execRoot, execRoot);
-  }
-
   @Nullable private final Path execRoot;
   private final Path path;
   private final boolean isMiddlemanRoot;