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

   ## Summary
   
   _Claude Code on behalf of gnodet_
   
   Fix three flaky camel-core tests identified by Develocity:
   
   - **TwoSchedulerTest** (4 flaky/70 builds = 6%): Replace 
`assertMockEndpointsSatisfied()` with `MockEndpoint.assertIsSatisfied(context, 
30, TimeUnit.SECONDS)`. Also fix `assertSame(tn1, tn2)` → `assertEquals(tn1, 
tn2)` since `assertSame` checks reference identity, not value equality, for 
String objects.
   - **DistributedTimeoutTest** (4 flaky/70 builds = 6%): Remove Awaitility 
wrapping around `MockEndpoint.assertIsSatisfied()` (anti-pattern per project 
guidelines — MockEndpoint already waits internally via CountDownLatch). Use 
direct `MockEndpoint.assertIsSatisfied(context, 30, TimeUnit.SECONDS)` for both 
contexts. Increase Awaitility `atMost` from 2s to 5s for the aggregation 
timeout wait.
   - **ThrottlingExceptionRoutePolicyOpenViaConfigTest** (11 flaky/275 builds = 
4%): Replace three `assertMockEndpointsSatisfied()` calls with 
`MockEndpoint.assertIsSatisfied(context, 30, TimeUnit.SECONDS)`.
   
   ## Test plan
   
   - [x] All three tests pass locally after changes
   - [ ] 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