Fix log assertions

This should not prevent Quarkus from shutting down
quarkusio · Sep 6, 2021 · 31138b0 · 31138b0
1 parent 9eb3e8b
commit 31138b0
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/test-framework/junit5-internal/src/main/java/io/quarkus/test/QuarkusUnitTest.java b/test-framework/junit5-internal/src/main/java/io/quarkus/test/QuarkusUnitTest.java
@@ -572,8 +572,9 @@ private Throwable unwrapException(Throwable cause) {
     public void afterAll(ExtensionContext extensionContext) throws Exception {
         actualTestClass = null;
         actualTestInstance = null;
+        List<LogRecord> records = null;
         if (assertLogRecords != null) {
-            assertLogRecords.accept(inMemoryLogHandler.records);
+            records = new ArrayList<>(inMemoryLogHandler.records);
         }
         rootLogger.setHandlers(originalHandlers);
         inMemoryLogHandler.clearRecords();
@@ -606,9 +607,12 @@ public void afterAll(ExtensionContext extensionContext) throws Exception {
             if (afterAllCustomizer != null) {
                 afterAllCustomizer.run();
             }
+            ClearCache.clearAnnotationCache();
+            GroovyCacheCleaner.clearGroovyCache();
+        }
+        if (records != null) {
+            assertLogRecords.accept(records);
         }
-        ClearCache.clearAnnotationCache();
-        GroovyCacheCleaner.clearGroovyCache();
     }
 
     @Override