POOL-425 Add failing tests to reproduce POOL-426

raju.gupta · raju.gupta · commit 3f93386ff7dd · 2025-12-25T21:10:32.000Z
diff --git a/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPool.java b/src/test/java/org/apache/commons/pool2/impl/TestGenericObjectPool.java
@@ -1026,8 +1026,6 @@ void testAddObjectConcurrentCallsRespectsMaxIdle() throws Exception {
                 executorService.shutdownNow(); // Ensure cleanup
             }
 
-            assertTrue(executorService.awaitTermination(Long.MAX_VALUE, TimeUnit.NANOSECONDS));
-
             assertTrue(pool.getNumIdle() <= maxIdleLimit,
                 "Concurrent addObject() calls should not exceed maxIdle limit of " + maxIdleLimit +
                     ", but found " + pool.getNumIdle() + " idle objects");

Original file line number	Diff line number	Diff line change
`@@ -1026,8 +1026,6 @@ void testAddObjectConcurrentCallsRespectsMaxIdle() throws Exception {`
`1026`	`1026`	`executorService.shutdownNow(); // Ensure cleanup`
`1027`	`1027`	`}`
`1028`	`1028`
`1029`		`- assertTrue(executorService.awaitTermination(Long.MAX_VALUE, TimeUnit.NANOSECONDS));`
`1030`		`-`
`1031`	`1029`	`assertTrue(pool.getNumIdle() <= maxIdleLimit,`
`1032`	`1030`	`"Concurrent addObject() calls should not exceed maxIdle limit of " + maxIdleLimit +`
`1033`	`1031`	`", but found " + pool.getNumIdle() + " idle objects");`