Can't make DefaultPooledObjectInfo.getPooledObject() public here.

Author: garydgregory

src/main/java/org/apache/commons/pool2/impl/DefaultPooledObjectInfo.java

@@ -93,7 +93,7 @@ public String getLastReturnTimeFormatted() {
      * @return the pooled object for debugging, use at your own risk, changing the state of this object may have unintended consequences for the pool.
      * @since 2.14.0
      */
-    public PooledObject<?> getPooledObject() {
+    PooledObject<?> getPooledObject() {
         return pooledObject;
     }
 

src/test/java/org/apache/commons/pool2/impl/TestBaseGenericObjectPool.java

@@ -19,6 +19,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 import java.lang.management.ManagementFactory;
@@ -219,8 +220,7 @@ void testEvictionTimerMultiplePools() throws InterruptedException {
     }
 
     /**
-     * POOL-393
-     * Tests JMX registration does not add too much latency to pool creation.
+     * POOL-393: Tests JMX registration does not add too much latency to pool creation.
      */
     @SuppressWarnings("resource") // pools closed in finally block
     @Test
@@ -232,7 +232,9 @@ void testJMXRegistrationLatency() {
         try {
             // final long startTime = System.currentTimeMillis();
             for (int i = 0; i < numPools; i++) {
-                pools.add(new GenericObjectPool<>(new WaiterFactory<>(0, 0, 0, 0, 0, 0), new GenericObjectPoolConfig<>()));
+                final GenericObjectPool<Waiter> gop = new GenericObjectPool<>(new WaiterFactory<>(0, 0, 0, 0, 0, 0), new GenericObjectPoolConfig<>());
+                assertNotNull(gop.getJmxName());
+                pools.add(gop);
             }
             // System.out.println("Duration: " + (System.currentTimeMillis() - startTime));
             final ObjectName oname = pools.get(numPools - 1).getJmxName();