Impove integration tests for commons-logging, jdk-platform-logging an…

…d slf4j

commons-logging/src/test/java/io/github/vitalijr2/logging/mock/commons/MockLoggerFactorySlowTest.java

@@ -1,5 +1,7 @@
 package io.github.vitalijr2.logging.mock.commons;
 
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.collection.IsIterableContainingInOrder.contains;
 import static org.junit.jupiter.api.Assertions.fail;
 import static org.mockito.Mockito.clearInvocations;
 import static org.mockito.Mockito.times;
@@ -77,14 +79,14 @@ void test(String level, String message, int traceCount, int debugCount, int info
   @Test
   void cleanAndResetMockLoggers() {
     // when
-    var loggerFactory = new MockLoggerFactory();
+    log.info("test message");
 
-    loggerFactory.getInstance("test").info("test message");
-
-    MockLoggerKeeper.getInstance().cleanAndReset();
+    var names = MockLoggerKeeper.getInstance().cleanAndReset();
 
     // then
-    verifyNoInteractions(loggerFactory.getInstance("test"));
+    assertThat(names, contains(MockLoggerFactorySlowTest.class.getName()));
+
+    verifyNoInteractions(log);
   }
 
 }

jdk-platform-logging/src/test/java/io/github/vitalijr2/logging/mock/platform/MockLoggerFinderSlowTest.java

@@ -1,5 +1,7 @@
 package io.github.vitalijr2.logging.mock.platform;
 
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.collection.IsIterableContainingInOrder.contains;
 import static org.mockito.Mockito.clearInvocations;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.verifyNoInteractions;
@@ -48,14 +50,14 @@ void test(Level level) {
   @Test
   void cleanAndResetMockLoggers() {
     // when
-    var loggerFinder = new MockLoggerFinder();
+    logger.log(Level.INFO, "test message");
 
-    loggerFinder.getLogger("test", null).log(Level.INFO, "test message");
-
-    MockLoggerKeeper.getInstance().cleanAndReset();
+    var names = MockLoggerKeeper.getInstance().cleanAndReset();
 
     // then
-    verifyNoInteractions(loggerFinder.getLogger("test", null));
+    assertThat(names, contains("test"));
+
+    verifyNoInteractions(logger);
   }
 
 }

slf4j/src/test/java/io/github/vitalijr2/logging/mock/slf4j/MockLoggerServiceProviderTest.java → slf4j/src/test/java/io/github/vitalijr2/logging/mock/slf4j/MockLoggerServiceProviderFastTest.java

@@ -9,7 +9,7 @@
 import org.junit.jupiter.api.Test;
 
 @Tag("fast")
-class MockLoggerServiceProviderTest {
+class MockLoggerServiceProviderFastTest {
 
   private MockLoggerServiceProvider provider;
 

slf4j/src/test/java/io/github/vitalijr2/logging/mock/slf4j/MockLoggerServiceProviderSlowTest.java

@@ -0,0 +1,32 @@
+package io.github.vitalijr2.logging.mock.slf4j;
+
+import static org.hamcrest.MatcherAssert.assertThat;
+import static org.hamcrest.collection.IsIterableContainingInOrder.contains;
+import static org.mockito.Mockito.verifyNoInteractions;
+
+import io.github.vitalijr2.logging.mock.MockLoggerKeeper;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Tag;
+import org.junit.jupiter.api.Test;
+
+@Tag("slow")
+class MockLoggerServiceProviderSlowTest {
+
+  @DisplayName("Clean and reset mock loggers")
+  @Test
+  void cleanAndResetMockLoggers() {
+    // given
+    var loggerProvider = new MockLoggerServiceProvider();
+
+    // when
+    loggerProvider.getLoggerFactory().getLogger("test").info("test message");
+
+    var names = MockLoggerKeeper.getInstance().cleanAndReset();
+
+    // then
+    assertThat(names, contains("test"));
+
+    verifyNoInteractions(loggerProvider.getLoggerFactory().getLogger("test"));
+  }
+
+}