gnodet opened a new pull request, #24420:
URL: https://github.com/apache/camel/pull/24420

   ## Summary
   
   Fix flaky `RouteIdTransactedIT.testRouteIdFailed` (and `testRouteId` for 
consistency) by replacing `MockEndpoint.assertIsSatisfied(context)` with 
Awaitility's `untilAsserted()`.
   
   **Root cause:** The default `MockEndpoint` latch timeout is 10 seconds 
(`waitForCompleteLatch` defaults to 10s when `resultWaitTime=0`). In transacted 
JMS routes under CI load on JDK 17, the message processing (including 
transaction handling and exception routing via `onException`) can exceed this 
timeout, causing intermittent test failures.
   
   **Fix:** Wrap mock endpoint assertions with `Awaitility.await().atMost(30, 
TimeUnit.SECONDS).untilAsserted(...)` to give transacted routes a generous 
30-second window. This follows the same Awaitility pattern already used 
elsewhere in camel-jms tests (e.g., `JmsAutoStartupTest`, 
`JmsDeliveryDelayTest`).
   
   ## Test plan
   
   - [x] `RouteIdTransactedIT` passes (2 tests, 0 failures)
   - [ ] CI build passes
   
   _Claude Code on behalf of Guillaume Nodet_
   
   🤖 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]

Reply via email to