rpamu-jdev opened a new pull request, #23999:
URL: https://github.com/apache/camel/pull/23999

   # Description
   
   Replaces test-synchronization `Thread.sleep` calls in `camel-cxf` tests with 
[Awaitility](https://github.com/awaitility/awaitility), per the project 
guideline in AGENTS.md.
   
   - `CxfOneWayRouteTest`: polling `while`-loop → 
`await().untilTrue(AtomicBoolean)`
   - `CxfPayloadProviderRouterTest`: `Thread.sleep(3000)` → 
`await().untilAsserted(...)` on `implementor.getInvocationCount()`
   - `CxfMessageHeadersRelayTest`: removed two stray unused 
`Thread.sleep(5000)` calls
   - `camel-cxf-spring-soap/pom.xml`: added `org.awaitility:awaitility` test 
dependency
   
   Remaining `Thread.sleep` calls (in `GreeterImplWithSleep`, 
`CxfConsumerContinuationTimeoutTest`, the `*ClientDisconnected` tests, 
`MyProcessor`) are intentionally left — they simulate slow business logic the 
tests exercise, not test synchronization.
   
   Verified locally:
   `mvn test -pl components/camel-cxf/camel-cxf-spring-soap 
-Dtest="CxfOneWayRouteTest,CxfPayloadProviderRouterTest,CxfMessageHeadersRelayTest"`
 → 26 passed, 0 failed.
   
   JIRA: [CAMEL-19517](https://issues.apache.org/jira/browse/CAMEL-19517)
   
   # Target
   
   - [x] I checked that the commit is targeting the correct branch (Camel 4 
uses the `main` branch)
   
   # Tracking
   
   - [x] If this is a large change, bug fix, or code improvement, I checked 
there is a [JIRA issue](https://issues.apache.org/jira/browse/CAMEL-19517) 
filed for the change.


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