diff --git a/feign-reactor-core/src/test/java/feign/reactor/LoggerTest.java b/feign-reactor-core/src/test/java/feign/reactor/LoggerTest.java index 79360dc..9270d62 100644 --- a/feign-reactor-core/src/test/java/feign/reactor/LoggerTest.java +++ b/feign-reactor-core/src/test/java/feign/reactor/LoggerTest.java @@ -33,10 +33,12 @@ import org.junit.Test; import org.mockito.ArgumentCaptor; import org.mockito.Mockito; +import reactor.core.publisher.Mono; import java.util.List; import static com.github.tomakehurst.wiremock.client.WireMock.*; import static com.github.tomakehurst.wiremock.core.WireMockConfiguration.wireMockConfig; import static org.assertj.core.api.Assertions.assertThat; +import static org.mockito.Mockito.never; import static org.mockito.Mockito.times; import static org.mockito.Mockito.when; @@ -74,9 +76,14 @@ public void shouldLog() throws Exception { .target(IcecreamServiceApi.class, "http://localhost:" + wireMockRule.port()); - client.makeOrder(order).block(); + Mono billMono = client.makeOrder(order); + // no logs before subscription ArgumentCaptor argumentCaptor = ArgumentCaptor.forClass(LogEvent.class); + Mockito.verify(appender, never()).append(argumentCaptor.capture()); + + billMono.block(); + Mockito.verify(appender, times(7)).append(argumentCaptor.capture()); List logEvents = argumentCaptor.getAllValues();