GlassFishLogManagerLifeCycleTest stabilized

- all logs are asynchronous here, so we have to respect that we don't have
  control over those unsynchronized parallel executions. Same as we don't
  have that control when it is running in multithreaded appserver.
- as an alternative we use the Thread.sleep, however even this might not be
  enough on some systems (virtual systems with cpu sharing limits, ie.).

Signed-off-by: David Matějček <david.matejcek@omnifish.ee>

nucleus/glassfish-jul-extension/src/test/java/org/glassfish/main/jul/GlassFishLogManagerLifeCycleTest.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2022 Eclipse Foundation and/or its affiliates. All rights reserved.
+ * Copyright (c) 2022, 2023 Eclipse Foundation and/or its affiliates. All rights reserved.
  *
  * This program and the accompanying materials are made available under the
  * terms of the Eclipse Public License v. 2.0, which is available at
@@ -126,14 +126,14 @@ public void startReconfiguration() throws Exception {
         assertEquals(GlassFishLoggingStatus.CONFIGURING, MANAGER.getLoggingStatus(),
             "status after startReconfigurationAndBlock test");
 
-        doLog(Level.FINE, "Log before reconfiguration");
-        doLog(Level.FINEST, "This log should be dropped, logger's level is FINE");
+        doLog(Level.FINE, "Log before reconfiguration", 0);
+        doLog(Level.FINEST, "This log should be dropped, logger's level is FINE", 0);
         process = runAsync(() -> MANAGER.reconfigure(originalCfg, ACTION_RECONFIG, ACTION_FLUSH));
         Thread.sleep(10L);
         assertAll(
             () -> assertEquals(GlassFishLoggingStatus.CONFIGURING, MANAGER.getLoggingStatus()),
             () -> assertNull(COLLECTOR.pop(), "COLLECTOR should be empty after reconfiguration started"));
-        doLog(Level.INFO, "Log after reconfiguration started");
+        doLog(Level.INFO, "Log after reconfiguration started", 0);
         assertNull(COLLECTOR.pop(), "COLLECTOR should be empty after reconfiguration started even if we log again");
     }
 
@@ -154,7 +154,7 @@ public void finishReconfigurationAndStartFlushing() throws Exception {
             () -> assertNull(COLLECTOR.pop(), "COLLECTOR should be empty after reconfiguration finished")
         );
 
-        doLog(Level.SEVERE, "Log while flushing is still executed");
+        doLog(Level.SEVERE, "Log while flushing is still executed", 5);
         assertNull(COLLECTOR.pop(), "COLLECTOR should be empty after reconfiguration finished even if we log again");
     }
 
@@ -170,7 +170,7 @@ public void finishReconfigurationAndStartFlushing() throws Exception {
     @Timeout(10)
     public void finishFlushing() throws Exception {
         ACTION_FLUSH.unblock();
-        doLog(Level.INFO, "Log after flushing finished");
+        doLog(Level.INFO, "Log after flushing finished", 5);
         final List<GlassFishLogRecord> logRecords = COLLECTOR.getAll();
         assertAll(
             () -> assertEquals(GlassFishLoggingStatus.FULL_SERVICE, MANAGER.getLoggingStatus(),
@@ -205,8 +205,8 @@ public void reconfigureWithoutResolvingLevelsWithincompleteconfiguration() throw
         setResolveLevelWithIncompleteConfiguration(false);
         assertEquals(GlassFishLoggingStatus.CONFIGURING, MANAGER.getLoggingStatus(), "after reconfigureToBlockingHandler");
 
-        doLog(Level.FINEST, "message0");
-        doLog(Level.INFO, "message1");
+        doLog(Level.FINEST, "message0", 1);
+        doLog(Level.INFO, "message1", 2);
         MANAGER.reconfigure(originalCfg, () -> LOG.setLevel(Level.FINEST), null);
         assertFalse(isResolveLevelWithIncompleteConfiguration(), "isResolveLevelWithIncompleteConfiguration");
 
@@ -238,8 +238,8 @@ public void reconfigureWithResolvingLevelsWithincompleteconfiguration() throws E
         setResolveLevelWithIncompleteConfiguration(true);
         assertEquals(GlassFishLoggingStatus.CONFIGURING, MANAGER.getLoggingStatus(), "after reconfigureToBlockingHandler");
 
-        doLog(Level.FINEST, "message0");
-        doLog(Level.INFO, "message1");
+        doLog(Level.FINEST, "message0", 1);
+        doLog(Level.INFO, "message1", 2);
         MANAGER.reconfigure(originalCfg, () -> LOG.setLevel(Level.FINEST), null);
         assertTrue(isResolveLevelWithIncompleteConfiguration(), "isResolveLevelWithIncompleteConfiguration");
 
@@ -269,9 +269,11 @@ private void reconfigureToBlockingHandler() {
      *
      * @throws Exception
      */
-    private void doLog(final Level level, final String message) throws Exception {
+    private void doLog(final Level level, final String message, final int sleepMillis) throws Exception {
         new Thread(() -> LOG.log(level, message)).start();
-        Thread.sleep(10L);
+        if (sleepMillis > 0) {
+            Thread.sleep(sleepMillis);
+        }
     }
 
     private static final class BlockingAction implements Action {