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());
         }
     }

Reply via email to