gnodet opened a new pull request, #24496:
URL: https://github.com/apache/camel/pull/24496
## Summary
_Claude Code on behalf of Guillaume Nodet_
- Adds guidance to the "Asynchronous Testing" section of AGENTS.md to prefer
`MockEndpoint`'s
built-in timed assertions (`assertIsSatisfied(context, timeout, unit)`)
over wrapping with
Awaitility when the wait condition is "mock expectations are met"
- Documents when Awaitility is still the right choice (mid-test
synchronization on conditions
MockEndpoint cannot express, e.g. `mock.getReceivedCounter() >= N`)
- Updates the rules to reference both mechanisms
This came up during review of #24304, where `Thread.sleep()` was being
replaced with
`await().untilAsserted(() -> MockEndpoint.assertIsSatisfied(...))` — which
is redundant because
MockEndpoint's assertion is already latch-based internally.
## Test plan
- [ ] Verify the markdown renders correctly
- [ ] No code changes — documentation only
🤖 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]
