commit f3cf98faa44f3b647956868422a11074aff0e9ee
author Laurent Le Brun <laurentlb@google.com> Fri Jun 17 13:36:24 2016 +0000
committer Philipp Wollermann <philwo@google.com> Mon Jun 20 09:34:06 2016 +0000
tree 93a361a853b3ebccd0af7074c993c29618ddf0e4
parent 48f2470b3cc60c11e46a95872a409341c4224096

Rollback of commit bdfd58a8ca2ed5735d6aaa5b238fb0f689515724.

--
MOS_MIGRATED_REVID=125160288

src/test/java/com/google/devtools/build/lib/actions/ArtifactTest.java

diff --git a/src/test/java/com/google/devtools/build/lib/actions/ArtifactTest.java b/src/test/java/com/google/devtools/build/lib/actions/ArtifactTest.java
index 676eab4..e43ddd1 100644
--- a/src/test/java/com/google/devtools/build/lib/actions/ArtifactTest.java
+++ b/src/test/java/com/google/devtools/build/lib/actions/ArtifactTest.java
@@ -57,12 +57,13 @@
   }
 
   @Test
-  public void testConstruction_UplevelRootDir() throws IOException {
+  public void testConstruction_badRootDir() throws IOException {
     Path f1 = scratch.file("/exec/dir/file.ext");
     Path bogusDir = scratch.file("/exec/dir/bogus");
-    Artifact artifact = new Artifact(f1, Root.asDerivedRoot(bogusDir), f1.relativeTo(execDir));
-    assertThat(artifact.getExecPath()).isEqualTo(new PathFragment("dir/file.ext"));
-    assertThat(artifact.getRootRelativePath()).isEqualTo(new PathFragment("../file.ext"));
+    try {
+      new Artifact(f1, Root.asDerivedRoot(bogusDir), f1.relativeTo(execDir));
+      fail("Expected IllegalArgumentException constructing artifact with a bad root dir");
+    } catch (IllegalArgumentException expected) {}
   }
 
   @Test