gnodet commented on PR #26845: URL: https://github.com/apache/camel/pull/26845#issuecomment-5854788128
Thanks @davsclaus — both test points fixed in d711425: 1. **Lazy pool init**: `evict_notStartedPool_handlesNullMXBeanGracefully` now uses the no-arg `HikariDataSource()` constructor + setters, and `assertNull(ds.getHikariPoolMXBean())` confirms the pool is truly not started. 2. **Stronger assertions**: `evict_startedPool_evictsIdleConnections` now verifies eviction actually happened — it grabs the physical connection before eviction, calls `evictDataSourceConnections()`, gets a new connection, and asserts `assertNotSame(physicalBefore, physicalAfter)` to prove the pool replaced the evicted connection with a fresh one. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
