gnodet opened a new pull request, #24628:
URL: https://github.com/apache/camel/pull/24628
## Summary
Fix flaky `SjmsConnectionRecoveryTest` by addressing async consumer startup
race conditions.
- **Phases 1 & 5**: Wrap send+assert cycles with
`Awaitility.untilAsserted()` (30s timeout) to
retry when the SJMS consumer hasn't subscribed yet
(`asyncStartListener=true` means background
startup). The retry covers both failure modes: send throwing
`JMSException` and message being
silently lost.
- **Phase 3 (connection recovery)**: Replace disguised `Thread.sleep`
(`await().pollDelay(4 * interval).until(() -> true)`)
with condition-based waits: first wait for a new connection to be created,
then verify the
connection count stabilizes (no extra connections).
**Develocity data**: 23 failures, 8 flaky out of 271 runs.
**Test verified locally**: 1 test, 0 failures.
_Claude Code on behalf of @gnodet_
--
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]
