issue-3561: Add tests for log behavior after removing unnecessary con…

…ditional statements

reactor-netty-core/src/test/java/reactor/netty/resources/DefaultPooledConnectionProviderTest.java

@@ -39,8 +39,10 @@
 import java.util.function.Function;
 import java.util.function.Supplier;
 
+import io.netty.channel.Channel;
 import io.netty.channel.ChannelOption;
 import io.netty.channel.EventLoopGroup;
+import io.netty.channel.embedded.EmbeddedChannel;
 import io.netty.channel.nio.NioEventLoopGroup;
 import io.netty.handler.logging.LoggingHandler;
 import io.netty.resolver.AddressResolver;
@@ -82,6 +84,11 @@
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.awaitility.Awaitility.await;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+import static reactor.netty.resources.PooledConnectionProvider.logPoolState;
+import static reactor.util.Loggers.resetLoggerFactory;
+import static reactor.util.Loggers.useVerboseConsoleLoggers;
 
 class DefaultPooledConnectionProviderTest {
 
@@ -570,6 +577,23 @@ void testIssue3316() throws ExecutionException, InterruptedException {
 		}
 	}
 
+	@Test
+	void testLogFormat() {
+		Channel channel = new EmbeddedChannel();
+		useVerboseConsoleLoggers();
+		try {
+			InstrumentedPool<? extends Connection> pool = mock(InstrumentedPool.class);
+			InstrumentedPool.PoolMetrics metrics = mock(InstrumentedPool.PoolMetrics.class);
+			when(pool.metrics()).thenReturn(metrics);
+			logPoolState(channel, pool, "log message");
+			logPoolState(channel, pool, "log message", new Exception("test log format"));
+		}
+		finally {
+			resetLoggerFactory();
+			Connection.from(channel).dispose();
+		}
+	}
+
 	static final class PoolImpl extends AtomicInteger implements InstrumentedPool<PooledConnection> {
 
 		@Override