Add more tests

Author: fmeum

src/test/java/com/google/devtools/build/lib/remote/RemoteActionFileSystemTest.java

@@ -1046,10 +1046,7 @@ public void createSymbolicLink_localFileArtifact() throws Exception {
     // assert
     assertThat(symlinkActionFs.getFileSystem()).isSameInstanceAs(actionFs);
     assertThat(symlinkActionFs.readSymbolicLink()).isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
+    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).exists(Symlinks.NOFOLLOW)).isFalse();
   }
 
   @Test
@@ -1070,11 +1067,8 @@ public void createSymbolicLink_remoteFileArtifact() throws Exception {
     // assert
     assertThat(symlinkActionFs.getFileSystem()).isSameInstanceAs(actionFs);
     assertThat(symlinkActionFs.readSymbolicLink()).isEqualTo(targetPath);
-    assertThat(outputArtifact.getPath().readSymbolicLink()).isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
+    assertThat(outputArtifact.getPath().exists(Symlinks.NOFOLLOW)).isFalse();
+    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).exists(Symlinks.NOFOLLOW)).isFalse();
   }
 
   @Test
@@ -1098,10 +1092,7 @@ public void createSymbolicLink_localTreeArtifact() throws Exception {
     // assert
     assertThat(symlinkActionFs.getFileSystem()).isSameInstanceAs(actionFs);
     assertThat(symlinkActionFs.readSymbolicLink()).isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
+    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).exists(Symlinks.NOFOLLOW)).isFalse();
   }
 
   @Test
@@ -1125,10 +1116,7 @@ public void createSymbolicLink_remoteTreeArtifact() throws Exception {
     // assert
     assertThat(symlinkActionFs.getFileSystem()).isSameInstanceAs(actionFs);
     assertThat(symlinkActionFs.readSymbolicLink()).isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
+    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).exists(Symlinks.NOFOLLOW)).isFalse();
   }
 
   @Test
@@ -1149,10 +1137,7 @@ public void createSymbolicLink_unresolvedSymlink() throws Exception {
     // assert
     assertThat(symlinkActionFs.getFileSystem()).isSameInstanceAs(actionFs);
     assertThat(symlinkActionFs.readSymbolicLink()).isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
-    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).readSymbolicLink())
-        .isEqualTo(targetPath);
+    assertThat(getLocalFileSystem(actionFs).getPath(linkPath).exists(Symlinks.NOFOLLOW)).isFalse();
   }
 
   @Test