gnodet opened a new pull request, #2970: URL: https://github.com/apache/cxf/pull/2970
## Summary - Replace hardcoded `Thread.sleep()` calls with Awaitility-based polling where possible, and increase remaining sleep durations to accommodate CI environments under load - Fix `testSessionsExceptionHandling` where the sleep (4s) was shorter than the minimum required time (5 failures x 1s retry = 5s) - Use Awaitility to poll `connectionsCreated` counter in connection/session fault tests, and poll for broker connectivity after broker restart ## Changes | Test | Before | After | |------|--------|-------| | `testDurableSubscriber` | `Thread.sleep(500)` | `Thread.sleep(2000)` | | `testMessageObserverExceptionHandling` | `Thread.sleep(500)` | `Thread.sleep(2000)` | | `testTemporaryQueueDeletionUponReset` | `atMost(1s)` | `atMost(5s)` | | `testConnectionFactoryExceptionHandling` | `Thread.sleep(4000)` | Awaitility polling `connectionsCreated` | | `testBrokerExceptionHandling` | `Thread.sleep(500)` + `Thread.sleep(2000)` | `Thread.sleep(2000)` + Awaitility broker connectivity poll + `Thread.sleep(5000)` | | `testSessionsExceptionHandling` | `Thread.sleep(4000)` (too short) | Awaitility polling `connectionsCreated >= 2` | ## Test plan - [x] `mvn -B test -pl rt/transports/jms -Dtest=JMSDestinationTest` passes (16 tests, 0 failures) - [x] `mvn -B validate -pl rt/transports/jms` passes (no checkstyle/PMD violations) 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
