This is an automated email from the ASF dual-hosted git repository.
mattsicker pushed a commit to branch mean-bean-machine
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git
The following commit(s) were added to refs/heads/mean-bean-machine by this push:
new 546ab29 Make test more explicit
546ab29 is described below
commit 546ab296129533739767b021faf49f52ac3c73ae
Author: Matt Sicker <bo...@gmail.com>
AuthorDate: Sun Mar 22 12:48:24 2020 -0500
Make test more explicit
This interweaves the state of the id generator in between invocations to make explicit the laziness in Provider<T> versus the behavior of different scopes.
Signed-off-by: Matt Sicker <bo...@gmail.com>
---
.../log4j/plugins/defaults/bean/DefaultBeanManagerTest.java | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/log4j-plugins/src/test/java/org/apache/logging/log4j/plugins/defaults/bean/DefaultBeanManagerTest.java b/log4j-plugins/src/test/java/org/apache/logging/log4j/plugins/defaults/bean/DefaultBeanManagerTest.java
index 78b42ce..0d30f17 100644
--- a/log4j-plugins/src/test/java/org/apache/logging/log4j/plugins/defaults/bean/DefaultBeanManagerTest.java
+++ b/log4j-plugins/src/test/java/org/apache/logging/log4j/plugins/defaults/bean/DefaultBeanManagerTest.java
@@ -331,12 +331,19 @@ public class DefaultBeanManagerTest {
final Provider<DeferredDependent> dependentProvider) {
assertEquals(0, generator.getCurrent());
assertEquals(1, singletonProvider.get().id);
+ assertEquals(1, generator.getCurrent());
assertEquals(1, singletonProvider.get().id);
+ assertEquals(1, generator.getCurrent());
assertEquals(2, dependentProvider.get().id);
+ assertEquals(2, generator.getCurrent());
assertEquals(1, singletonProvider.get().id);
+ assertEquals(2, generator.getCurrent());
assertEquals(3, dependentProvider.get().id);
+ assertEquals(3, generator.getCurrent());
assertEquals(1, singletonProvider.get().id);
+ assertEquals(3, generator.getCurrent());
assertEquals(4, dependentProvider.get().id);
+ assertEquals(4, generator.getCurrent());
}
// TODO: add tests for other supported injection scenarios