From f7c1649aba18af574c778f14d98ccf476c22004d Mon Sep 17 00:00:00 2001 From: John Yang Date: Tue, 19 May 2015 10:40:50 +0900 Subject: [PATCH] address brian's comments --- .../apache/reef/inmemory/client/SurfFSDirectoryITCase.java | 2 +- .../apache/reef/inmemory/client/SurfFSFileStatusITCase.java | 4 ++-- 2 files changed, 3 insertions(+), 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 index 391e711a0..2f3e10e68 100644 --- 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 index 7de5f47e5..45c07e0cd 100644 --- 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 {