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

   ## Fix summary
   
   Addresses [CAMEL-24035](https://issues.apache.org/jira/browse/CAMEL-24035).
   
   **Context:** Core flaky-test fixes were already merged in 
[CAMEL-24037](https://issues.apache.org/jira/browse/CAMEL-24037) (#24635):
   - `TwoSchedulerTest` — timed mock assert + `assertEquals` for thread names
   - `DistributedTimeoutTest` — removed Awaitility-wrapped mock asserts
   - `ThrottlingExceptionRoutePolicyOpenViaConfigTest` — timed mock asserts
   
   This PR adds remaining polish on 2 files (14 lines changed):
   - `TIMEOUT_SECONDS` constant for timed `MockEndpoint.assertIsSatisfied` calls
   - Drop unnecessary `public` on test classes/methods
   - Keep bare `mock.assertIsSatisfied()` for count-0 expectations (timed 
variant adds no wait)
   - Keep 10s Awaitility for non-mock conditions (consumer suspend/resume, 
aggregation timeout callback)
   
   **Verified:**
   ```bash
   ./mvnw -pl core/camel-core -am test 
-Dtest=TwoSchedulerTest,DistributedTimeoutTest,ThrottlingExceptionRoutePolicyOpenViaConfigTest
   ```
   
   ---
   _AI-generated comment on behalf of atiaomar1978-hub_


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