Merge pull request #42 from mhalbritter/mh/logback-slf4j

Make sure that org.slf4j.impl.StaticLoggerBinder is available

Author: dnestoro

metadata/ch.qos.logback/logback-classic/1.2.11/reflect-config.json

@@ -292,5 +292,8 @@
     },
     "name": "ch.qos.logback.core.pattern.color.YellowCompositeConverter",
     "allPublicConstructors": true
+  },
+  {
+    "name": "org.slf4j.impl.StaticLoggerBinder"
   }
 ]

tests/src/ch.qos.logback/logback-classic/1.2.11/src/test/java/org/graalvm/logback/LogbackTests.java

@@ -19,6 +19,7 @@
 import ch.qos.logback.core.ConsoleAppender;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
@@ -55,6 +56,21 @@ void shouldRegisterConverter(String converterName, String pattern) {
     cleanUp(encoder, consoleAppender, logger);
   }
 
+  @Test
+  void slf4jCanBeDetected() {
+    boolean slf4jAvailable = isClassPresent("org.slf4j.Logger") && (isClassPresent("org.slf4j.impl.StaticLoggerBinder") || isClassPresent("org.slf4j.spi.SLF4JServiceProvider"));
+    assertThat(slf4jAvailable).isTrue();
+  }
+
+  private boolean isClassPresent(String className) {
+    try {
+      Class.forName(className);
+      return true;
+    } catch (ClassNotFoundException ex) {
+      return false;
+    }
+  }
+
   private static Stream<Arguments> converterSource() {
     return PatternLayout.DEFAULT_CONVERTER_MAP.entrySet().stream()
         .map(entry -> Arguments.of(entry.getValue(), entry.getKey()));