Update test cases and comment

Author: wangmengyan95

Parse/src/main/java/com/parse/ParseFile.java

@@ -262,7 +262,6 @@ public boolean isDirty() {
    * Whether the file has available data.
    */
   public boolean isDataAvailable() {
-    // TODO(mengyan): Revise when we have proper stage strategy for file pointer
     return data != null || getFileController().isDataAvailable(state);
   }
 

Parse/src/test/java/com/parse/ParseFileTest.java

@@ -58,7 +58,8 @@ public void testConstructor() throws Exception {
     String contentType = "content_type";
     File file = temporaryFolder.newFile(name);
 
-    // TODO(mengyan): Test data and file pointer in ParseFile when we have proper stage strategy
+    // TODO(mengyan): After we have proper staging strategy, we should verify the staging file's
+    // content is the same with the original file.
 
     ParseFile parseFile = new ParseFile(name, data, contentType);
     assertEquals("name", parseFile.getName());
@@ -288,6 +289,22 @@ public void testGetDataAsyncSuccess() throws Exception {
     assertEquals(url, stateCaptor.getValue().url());
     // Verify the data we get is correct
     assertArrayEquals(content.getBytes(), data);
+
+    // Make sure we always get the data from network
+    byte[] dataAgain = ParseTaskUtils.wait(parseFile.getDataInBackground());
+
+    // Verify controller get the correct data
+    ArgumentCaptor<ParseFile.State> stateCaptorAgain =
+        ArgumentCaptor.forClass(ParseFile.State.class);
+    verify(controller, times(2)).fetchAsync(
+        stateCaptorAgain.capture(),
+        anyString(),
+        any(ProgressCallback.class),
+        Matchers.<Task<Void>>any()
+    );
+    assertEquals(url, stateCaptorAgain.getValue().url());
+    // Verify the data we get is correct
+    assertArrayEquals(content.getBytes(), dataAgain);
   }
 
   @Test
@@ -322,6 +339,22 @@ public void testGetDataStreamAsyncSuccess() throws Exception {
     assertEquals(url, stateCaptor.getValue().url());
     // Verify the data we get is correct
     assertArrayEquals(content.getBytes(), ParseIOUtils.toByteArray(dataStream));
+
+    // Make sure we always get the data from network
+    InputStream dataStreamAgain = ParseTaskUtils.wait(parseFile.getDataStreamInBackground());
+
+    // Verify controller get the correct data
+    ArgumentCaptor<ParseFile.State> stateCaptorAgain =
+        ArgumentCaptor.forClass(ParseFile.State.class);
+    verify(controller, times(2)).fetchAsync(
+        stateCaptorAgain.capture(),
+        anyString(),
+        any(ProgressCallback.class),
+        Matchers.<Task<Void>>any()
+    );
+    assertEquals(url, stateCaptorAgain.getValue().url());
+    // Verify the data we get is correct
+    assertArrayEquals(content.getBytes(), ParseIOUtils.toByteArray(dataStreamAgain));
   }
 
   @Test
@@ -356,6 +389,22 @@ public void testGetFileAsyncSuccess() throws Exception {
     assertEquals(url, stateCaptor.getValue().url());
     // Verify the data we get is correct
     assertArrayEquals(content.getBytes(), ParseFileUtils.readFileToByteArray(fetchedFile));
+
+    // Make sure we always get the data from network
+    File fetchedFileAgain = ParseTaskUtils.wait(parseFile.getFileInBackground());
+
+    // Verify controller get the correct data
+    ArgumentCaptor<ParseFile.State> stateCaptorAgain =
+        ArgumentCaptor.forClass(ParseFile.State.class);
+    verify(controller, times(2)).fetchAsync(
+        stateCaptorAgain.capture(),
+        anyString(),
+        any(ProgressCallback.class),
+        Matchers.<Task<Void>>any()
+    );
+    assertEquals(url, stateCaptorAgain.getValue().url());
+    // Verify the data we get is correct
+    assertArrayEquals(content.getBytes(), ParseFileUtils.readFileToByteArray(fetchedFileAgain));
   }
 
   //endregion