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

   ## Summary
   
   _Claude Code on behalf of Guillaume Nodet_
   
   Fix flaky `RecipientListParallelStreamingTest#testRecipientListParallel` 
that fails on both JDK 17 and JDK 25.
   
   - **Split** the single test method into two isolated methods 
(`testRecipientListParallel` and `testRecipientListParallelStreaming`) to 
eliminate mock state contamination via `reset()`
   - **Increase delay gaps** from 100ms/500ms to 500ms/2000ms so the completion 
ordering for the streaming case is deterministic even on busy CI systems 
(matching the pattern used in `MulticastParallelStreamingTest`)
   - **Replace** `assertMockEndpointsSatisfied()` with Awaitility 
`await().untilAsserted()` per project testing guidelines
   
   ### Root cause
   
   The streaming test relies on `direct:b` (previously 500ms delay) being the 
last recipient to complete so that `UseLatestAggregationStrategy` picks body 
`"b"`. With only a 400ms gap over `direct:a` (100ms delay), CI thread 
scheduling jitter could collapse the gap and produce the wrong aggregation 
result.
   
   ## Test plan
   
   - [x] `RecipientListParallelStreamingTest` passes reliably (verified 5 
consecutive runs)
   - [ ] CI build passes
   
   🤖 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