[JAVA-8147] Update and cleanup code for debugging reactive streams

spring-reactive/src/test/java/com/baeldung/reactive/debugging/consumer/ConsumerFooServiceIntegrationTest.java

@@ -45,19 +45,19 @@ public class ConsumerFooServiceIntegrationTest {
         Collection<String> allSuppressedEntries = ListAppender.getEvents()
                 .stream()
                 .map(ILoggingEvent::getThrowableProxy)
-                .flatMap(t -> {
-                    return Optional.ofNullable(t)
-                            .map(IThrowableProxy::getSuppressed)
-                            .map(Arrays::stream)
-                            .orElse(Stream.empty());
-                })
+                .flatMap(t -> Optional.ofNullable(t)
+                        .map(IThrowableProxy::getSuppressed)
+                        .map(Arrays::stream)
+                        .orElse(Stream.empty()))
                 .map(IThrowableProxy::getClassName)
                 .collect(Collectors.toList());
-        assertThat(allLoggedEntries).anyMatch(entry -> entry.contains("The following error happened on processFoo method!"))
-                .anyMatch(entry -> entry.contains("| onSubscribe"))
-                .anyMatch(entry -> entry.contains("| cancel()"));
+
+        assertThat(allLoggedEntries)
+          .anyMatch(entry -> entry.contains("The following error happened on processFoo method!"))
+          .anyMatch(entry -> entry.contains("| onSubscribe"))
+          .anyMatch(entry -> entry.contains("| cancel()"));
 
         assertThat(allSuppressedEntries)
-                .anyMatch(entry -> entry.contains("reactor.core.publisher.FluxOnAssembly$OnAssemblyException"));
+          .anyMatch(entry -> entry.contains("reactor.core.publisher.FluxOnAssembly$OnAssemblyException"));
     }
 }

spring-reactive/src/test/java/com/baeldung/reactive/debugging/consumer/ConsumerFooServiceLiveTest.java

@@ -1,6 +1,5 @@
 package com.baeldung.reactive.debugging.consumer;
 
-import com.baeldung.reactive.debugging.consumer.service.FooService;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 import org.springframework.test.web.reactive.server.WebTestClient;
@@ -13,8 +12,6 @@ import org.springframework.test.web.reactive.server.WebTestClient.ResponseSpec;
  */
 public class ConsumerFooServiceLiveTest {
 
-    FooService service = new FooService();
-
     private static final String BASE_URL = "http://localhost:8082";
     private static final String DEBUG_HOOK_ON = BASE_URL + "/debug-hook-on";
     private static final String DEBUG_HOOK_OFF = BASE_URL + "/debug-hook-off";

spring-reactive/src/test/java/com/baeldung/reactive/debugging/consumer/utils/ListAppender.java

@@ -8,18 +8,18 @@ import java.util.List;
 
 public class ListAppender extends AppenderBase<ILoggingEvent> {
 
-    static private List<ILoggingEvent> events = new ArrayList<>();
+    private static final List<ILoggingEvent> EVENTS = new ArrayList<>();
 
     @Override
     protected void append(ILoggingEvent eventObject) {
-        events.add(eventObject);
+        EVENTS.add(eventObject);
     }
 
     public static List<ILoggingEvent> getEvents() {
-        return events;
+        return EVENTS;
     }
 
     public static void clearEventList() {
-        events.clear();
+        EVENTS.clear();
     }
 }