diff options
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/remote/ByteStreamBuildEventArtifactUploaderTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/remote/ByteStreamBuildEventArtifactUploaderTest.java | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/remote/ByteStreamBuildEventArtifactUploaderTest.java b/src/test/java/com/google/devtools/build/lib/remote/ByteStreamBuildEventArtifactUploaderTest.java index 75b46c461e..f45e3bca96 100644 --- a/src/test/java/com/google/devtools/build/lib/remote/ByteStreamBuildEventArtifactUploaderTest.java +++ b/src/test/java/com/google/devtools/build/lib/remote/ByteStreamBuildEventArtifactUploaderTest.java @@ -15,6 +15,7 @@ package com.google.devtools.build.lib.remote; import static com.google.common.truth.Truth.assertThat; import static org.junit.Assert.fail; +import static org.mockito.Mockito.mock; import com.google.bytestream.ByteStreamProto.WriteRequest; import com.google.bytestream.ByteStreamProto.WriteResponse; @@ -162,6 +163,21 @@ public class ByteStreamBuildEventArtifactUploaderTest { } @Test + public void testUploadDirectoryDoesNotCrash() throws Exception { + Path dir = fs.getPath("/dir"); + dir.createDirectoryAndParents(); + Map<Path, LocalFile> filesToUpload = new HashMap<>(); + filesToUpload.put(dir, new LocalFile(dir, LocalFileType.OUTPUT)); + ByteStreamUploader uploader = mock(ByteStreamUploader.class); + ByteStreamBuildEventArtifactUploader artifactUploader = + new ByteStreamBuildEventArtifactUploader( + uploader, "localhost", withEmptyMetadata, "instance"); + PathConverter pathConverter = artifactUploader.upload(filesToUpload).get(); + assertThat(pathConverter.apply(dir)).isNull(); + artifactUploader.shutdown(); + } + + @Test public void someUploadsFail() throws Exception { // Test that if one of multiple file uploads fails, the upload future fails and that the // error is propagated correctly. |