LOG4J2-435 added comments Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/bc807a8a Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/bc807a8a Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/bc807a8a
Branch: refs/heads/master Commit: bc807a8a5f76dfb33d41cc96e1b12371836aa52b Parents: 8152f19 Author: rpopma <[email protected]> Authored: Fri Nov 27 00:30:00 2015 +0900 Committer: rpopma <[email protected]> Committed: Fri Nov 27 00:30:00 2015 +0900 ---------------------------------------------------------------------- .../core/appender/rolling/action/IfAccumulatedFileCountTest.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/bc807a8a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/action/IfAccumulatedFileCountTest.java ---------------------------------------------------------------------- diff --git a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/action/IfAccumulatedFileCountTest.java b/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/action/IfAccumulatedFileCountTest.java index 719f6f6..343aa09 100644 --- a/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/action/IfAccumulatedFileCountTest.java +++ b/log4j-core/src/test/java/org/apache/logging/log4j/core/appender/rolling/action/IfAccumulatedFileCountTest.java @@ -35,11 +35,13 @@ public class IfAccumulatedFileCountTest { @Test public void testAccept() { int[] counts = {3, 5, 9}; - for (int count: counts) { + for (int count : counts) { IfAccumulatedFileCount condition = IfAccumulatedFileCount.createFileCountCondition(count); for (int i = 0; i < count; i++) { assertFalse(condition.accept(null, null, null)); + // exact match: does not accept } + // accept when threshold is exceeded assertTrue(condition.accept(null, null, null)); assertTrue(condition.accept(null, null, null)); }
