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, 4 insertions, 21 deletions

diff --git a/src/test/java/com/google/devtools/build/lib/actions/ActionLookupValueTest.java b/src/test/java/com/google/devtools/build/lib/actions/ActionLookupValueTest.java
index 3bcf87b68b..5ae6c377e5 100644
--- a/src/test/java/com/google/devtools/build/lib/actions/ActionLookupValueTest.java
+++ b/src/test/java/com/google/devtools/build/lib/actions/ActionLookupValueTest.java
@@ -21,6 +21,7 @@ import static org.mockito.Mockito.when;
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.ImmutableSet;
 import com.google.devtools.build.lib.vfs.FileSystem;
+import com.google.devtools.build.lib.vfs.Path;
 import com.google.devtools.build.lib.vfs.PathFragment;
 import com.google.devtools.build.lib.vfs.inmemoryfs.InMemoryFileSystem;
 import org.junit.Before;
@@ -55,7 +56,9 @@ public class ActionLookupValueTest {
 
   @Test
   public void testActionNotPresentAfterEvaluation() {
-    Root root = Root.asDerivedRoot(fs.getRootDirectory());
+    Path execRoot = fs.getPath("/execroot");
+    Path outputRootPath = execRoot.getRelative("blaze-out");
+    Root root = Root.asDerivedRoot(execRoot, outputRootPath);
     Action normalAction = mock(Action.class);
     Artifact normalArtifact = new Artifact(PathFragment.create("normal"), root);
     when(normalAction.getOutputs()).thenReturn(ImmutableSet.of(normalArtifact));
diff --git a/src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java b/src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java
index a3b6f6c1e2..5c6005d1cb 100644
--- a/src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java
+++ b/src/test/java/com/google/devtools/build/lib/actions/ArtifactFactoryTest.java
@@ -210,26 +210,6 @@ public class ArtifactFactoryTest {
     }
   }
 
-  @Test
-  public void testGetDerivedArtifact() throws Exception {
-    PathFragment toolPath = PathFragment.create("_bin/tool");
-    Artifact artifact = artifactFactory.getDerivedArtifact(toolPath, execRoot);
-    assertThat(artifact.getExecPath()).isEqualTo(toolPath);
-    assertThat(artifact.getRoot()).isEqualTo(Root.asDerivedRoot(execRoot));
-    assertThat(artifact.getPath()).isEqualTo(execRoot.getRelative(toolPath));
-    assertThat(artifact.getOwner()).isNull();
-  }
-
-  @Test
-  public void testGetDerivedArtifactFailsForAbsolutePath() throws Exception {
-    try {
-      artifactFactory.getDerivedArtifact(PathFragment.create("/_bin/b"), execRoot);
-      fail();
-    } catch (IllegalArgumentException e) {
-      // Expected exception
-    }
-  }
-
   private static class MockPackageRootResolver implements PackageRootResolver {
     private Map<PathFragment, Root> packageRoots = Maps.newHashMap();