Upgrade to bootique-logback 1.5.18 #72

.. adding a small delay to LogTester to increase the probability that the files are flushed

Author: andrus

RELEASE-NOTES.md

@@ -1,6 +1,7 @@
 ## 4.0-M1
 
 * #71 Remove deprecated "bootique-logback-sentry" module
+* #72 Upgrade to bootique-logback 1.5.18
 
 ## 3.0-M4
 

bootique-logback/src/test/java/io/bootique/logback/LogbackBQConfigIT.java

@@ -247,10 +247,10 @@ public void fileAppender_Rotate_By_Size() {
      * Checks file appender with rolling policy "size" (SizeAndTimeBasedRollingPolicy)
      * <p>
      * This test makes 4 attempts of printing 2 log rows each second.
-     * Logback configuration defines rollover by seconds with help of file name pattern;
+     * Logback configuration defines rollover by seconds with the help of file name pattern;
      * 5 seconds of total history and size of each file is 40 bytes.
      * <p>
-     * As result, 2 archived files + 1 current log-file are expected; 6 rows are expected in all log files
+     * As a result, 2 archived files + 1 current log-file are expected; 6 rows are expected in all log files
      */
     @Test
     public void fileAppender_Rotate_By_Size_And_History() {

bootique-logback/src/test/java/io/bootique/logback/unit/LogTester.java

@@ -73,21 +73,22 @@ public String run(String appConfig, String logFile, Consumer<Logger> logger) {
     public Map<String, String> run(String appConfig, Consumer<Logger> logger) {
         try {
             return doRun(appConfig, logger);
-        } catch (IOException e) {
+        } catch (Exception e) {
             throw new RuntimeException(e);
         }
     }
 
-    private Map<String, String> doRun(String appConfig, Consumer<Logger> logger) throws IOException {
+    private Map<String, String> doRun(String appConfig, Consumer<Logger> logger) throws IOException, InterruptedException {
 
         BQRuntime app = testFactory.app("-c", appConfig)
                 .module(LogbackModule.class)
                 .createRuntime();
 
         logger.accept(app.getInstance(Logger.class));
 
-        // must stop to ensure logs are flushed...
+        // must stop and wait to ensure logs are flushed...
         app.shutdown();
+        Thread.sleep(50);
 
         return logsByFile();
     }

pom.xml

@@ -44,7 +44,7 @@
     </modules>
 
     <properties>
-        <logback.version>1.5.5</logback.version>
+        <logback.version>1.5.18</logback.version>
         <logback.contrib.version>0.1.5</logback.contrib.version>
     </properties>