atiaomar1978-hub commented on PR #25228:
URL: https://github.com/apache/camel/pull/25228#issuecomment-5155636845

   _AI-generated comment on behalf of 
[atiaomar1978-hub](https://github.com/atiaomar1978-hub)_
   
   Pushed `a9f0680dae5` addressing @gnodet's follow-up suggestions from the 
re-review:
   
   1. **`duplicate()` coverage** — added 
`duplicateCopiesAllDeclaredInstanceFields()` reflection test to ensure every 
non-static instance field is copied (guards against copy staleness when new 
fields are added).
   2. **Null guard** — `withExecuteToolsConcurrently(Executor)` now uses 
`Objects.requireNonNull`; added 
`withExecuteToolsConcurrentlyRejectsNullExecutor` test.
   3. **Test class visibility** — `AgentConfigurationTest` is now 
package-private.
   
   **Tests (all green locally):**
   - `AgentConfigurationTest`
   - `LangChain4jAgentExecuteToolsConcurrentlyTest`


-- 
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