address brian's comments

snuspl · May 19, 2015 · f7c1649 · f7c1649
1 parent b0e923a
commit f7c1649
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/reef-inmemory/src/test/java/org/apache/reef/inmemory/client/SurfFSDirectoryITCase.java b/reef-inmemory/src/test/java/org/apache/reef/inmemory/client/SurfFSDirectoryITCase.java
@@ -225,7 +225,7 @@ public void testWorkingDirectory() throws IOException {
     assertEquals(ITUtils.getTestDir() + "/" + file, fileStatuses[0].getPath().toUri().getPath());
 
     // Clean up to avoid conflicts with other tests
-    assertTrue(surfFs.delete(file, true));
+    assertTrue("delete() must return true, as the file should be found", surfFs.delete(file, true));
   }
 
   private void testGetFileStatusOnDir(final String path) throws IOException {

diff --git a/reef-inmemory/src/test/java/org/apache/reef/inmemory/client/SurfFSFileStatusITCase.java b/reef-inmemory/src/test/java/org/apache/reef/inmemory/client/SurfFSFileStatusITCase.java
@@ -102,7 +102,7 @@ private void testListStatusOfDir(final Path dirPath) throws IOException {
         tested = true;
       }
     }
-    assertTrue(tested);
+    assertTrue("A filestatus whose path is ABSPATH must be tested", tested);
   }
 
   private void testListStatusOfFile(final Path filePath) throws IOException {
@@ -119,7 +119,7 @@ private void testListStatusOfFile(final Path filePath) throws IOException {
         tested = true;
       }
     }
-    assertTrue(tested);
+    assertTrue("A filestatus whose path is ABSPATH must be tested", tested);
   }
 
   private void testFileStatusOfTestFile(final FileStatus fileStatus) throws IOException {