You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@logging.apache.org by gg...@apache.org on 2017/06/21 00:42:06 UTC

logging-log4j2 git commit: [LOG4J2-1699] Configurable Log File Permissions with PosixFilePermission. Update condition for test exit with a JUnit Assume.

Repository: logging-log4j2
Updated Branches:
  refs/heads/master c16cfdb60 -> b1356fa16


[LOG4J2-1699] Configurable Log File Permissions with
PosixFilePermission. Update condition for test exit with a JUnit Assume.

Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/b1356fa1
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/b1356fa1
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/b1356fa1

Branch: refs/heads/master
Commit: b1356fa16e8ae19eba8bcd117212d489f041f5a7
Parents: c16cfdb
Author: Gary Gregory <gg...@apache.org>
Authored: Tue Jun 20 17:42:02 2017 -0700
Committer: Gary Gregory <gg...@apache.org>
Committed: Tue Jun 20 17:42:02 2017 -0700

----------------------------------------------------------------------
 .../apache/logging/log4j/core/appender/FilePermissionsTest.java  | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/b1356fa1/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/FilePermissionsTest.java
----------------------------------------------------------------------
diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/FilePermissionsTest.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/FilePermissionsTest.java
index b8b8d9d..9754240 100644
--- a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/FilePermissionsTest.java
+++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/FilePermissionsTest.java
@@ -91,9 +91,7 @@ public class FilePermissionsTest {
 
     @Test
     public void testFilePermissions() throws Exception {
-        if (!FileSystems.getDefault().supportedFileAttributeViews().contains("posix")) {
-            return;
-        }
+        Assume.assumeTrue(FileSystems.getDefault().supportedFileAttributeViews().contains("posix"));
         final Layout<String> layout = PatternLayout.newBuilder().withPattern(PatternLayout.SIMPLE_CONVERSION_PATTERN)
                 .build();
         // @formatter:off