Read InputStream content fully

src/main/java/com/uber/jenkins/phabricator/RemoteFileFetcher.java

@@ -24,6 +24,7 @@
 import com.uber.jenkins.phabricator.utils.Logger;
 import hudson.FilePath;
 
+import org.apache.commons.io.IOUtils;
 import java.io.IOException;
 
 import static java.lang.Integer.parseInt;
@@ -75,7 +76,7 @@ public String getRemoteFile() throws InterruptedException, IOException {
             maxLength = (int)source.length();
         }
         byte[] buffer = new byte[maxLength];
-        source.read().read(buffer, 0, maxLength);
+        IOUtils.read(source.read(), buffer);
 
         return new String(buffer);
     }

src/test/java/com/uber/jenkins/phabricator/RemoteFileFetcherTest.java

@@ -36,6 +36,7 @@
 
 import java.io.IOException;
 import java.util.concurrent.ExecutionException;
+import org.apache.commons.lang.StringUtils;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNull;
@@ -76,15 +77,20 @@ public void testNoCommentConfigured() throws Exception {
 
     @Test
     public void testSingleFile() throws Exception {
-        testWithContent("hello, world");
+        testWithContent("hello, world", "1000");
     }
 
     @Test
     public void testUTF8File() throws Exception {
-        testWithContent("こんにちは世界");
+        testWithContent("こんにちは世界", "1000");
     }
 
-    private void testWithContent(String content) throws Exception {
+    @Test
+    public void testLargeFile() throws Exception {
+        testWithContent(StringUtils.repeat("a", 10000), "10000");
+    }
+
+    private void testWithContent(String content, String len) throws Exception {
         final String fileName = "just-a-test.txt";
         project.getBuildersList().add(echoBuilder(fileName, content));
         FreeStyleBuild build = getBuild();
@@ -93,7 +99,7 @@ private void testWithContent(String content) throws Exception {
                 build.getWorkspace(),
                 logger,
                 fileName,
-                "1000"
+                len
         );
 
         assertEquals(content, fetcher.getRemoteFile());
@@ -102,7 +108,7 @@ private void testWithContent(String content) throws Exception {
                 build.getWorkspace(),
                 logger,
                 "*.txt",
-                "1000"
+                len
         );
 
         assertEquals(content, fetcher.getRemoteFile());