This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-vfs.git
The following commit(s) were added to refs/heads/master by this push:
new ec99e67 Add file name to JUnit assert message.
ec99e67 is described below
commit ec99e67e75320d13e764943d15ff2cf152886771
Author: Gary Gregory <[email protected]>
AuthorDate: Wed Jan 30 14:15:30 2019 -0500
Add file name to JUnit assert message.
---
.../org/apache/commons/vfs2/test/PermissionsTests.java | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git
a/commons-vfs2/src/test/java/org/apache/commons/vfs2/test/PermissionsTests.java
b/commons-vfs2/src/test/java/org/apache/commons/vfs2/test/PermissionsTests.java
index 6ca2a09..910b574 100644
---
a/commons-vfs2/src/test/java/org/apache/commons/vfs2/test/PermissionsTests.java
+++
b/commons-vfs2/src/test/java/org/apache/commons/vfs2/test/PermissionsTests.java
@@ -56,15 +56,15 @@ public class PermissionsTests extends
AbstractProviderTestCase {
} else {
// Set the executable flag for owner
- Assert.assertTrue("Setting executable permission failed",
file.setExecutable(true, true));
+ Assert.assertTrue("Setting executable permission failed: " + file,
file.setExecutable(true, true));
Assert.assertTrue("File expected to be executable: " + file,
file.isExecutable());
// Set the executable flag for all
- Assert.assertTrue("Setting executable permission failed",
file.setExecutable(true, false));
+ Assert.assertTrue("Setting executable permission failed: " + file,
file.setExecutable(true, false));
Assert.assertTrue("File expected to be executable: " + file,
file.isExecutable());
// Clear the executable flag
- Assert.assertTrue("Setting executable permission failed",
file.setExecutable(false, true));
+ Assert.assertTrue("Setting executable permission failed: " + file,
file.setExecutable(false, true));
Assert.assertFalse("File expected to be not executable: " + file,
file.isExecutable());
}
}
@@ -76,15 +76,15 @@ public class PermissionsTests extends
AbstractProviderTestCase {
final FileObject file = createTestFile();
// Set the write permission for owner
- Assert.assertTrue("Setting write permission failed",
file.setWritable(true, true));
+ Assert.assertTrue("Setting write permission failed: " + file,
file.setWritable(true, true));
Assert.assertTrue("File expected to be writable: " + file,
file.isWriteable());
// Set the write permission for all
- Assert.assertTrue("Setting write permission failed",
file.setWritable(true, false));
+ Assert.assertTrue("Setting write permission failed: " + file,
file.setWritable(true, false));
Assert.assertTrue("File expected to be writable: " + file,
file.isWriteable());
// Clear the write permission
- Assert.assertTrue("Setting write permission failed",
file.setWritable(false, true));
+ Assert.assertTrue("Setting write permission failed: " + file,
file.setWritable(false, true));
Assert.assertFalse("File expected to be not writable: " + file,
file.isWriteable());
}
@@ -99,15 +99,15 @@ public class PermissionsTests extends
AbstractProviderTestCase {
Assert.assertTrue("File expected to be readable: " + file,
file.isReadable());
} else {
// Set the readable permission for owner
- Assert.assertTrue("Setting read permission failed",
file.setReadable(true, true));
+ Assert.assertTrue("Setting read permission failed: " + file,
file.setReadable(true, true));
Assert.assertTrue("File expected to be readable: " + file,
file.isReadable());
// Set the readable permission for all
- Assert.assertTrue("Setting read permission failed",
file.setReadable(true, false));
+ Assert.assertTrue("Setting read permission failed: " + file,
file.setReadable(true, false));
Assert.assertTrue("File expected to be readable: " + file,
file.isReadable());
// Clear the readable permission
- Assert.assertTrue("Setting read permission failed",
file.setReadable(false, true));
+ Assert.assertTrue("Setting read permission failed: " + file,
file.setReadable(false, true));
Assert.assertFalse("File expected to be not readable: " + file,
file.isReadable());
}
}