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

   ## Summary
   
   _Claude Code on behalf of gnodet_
   
   Fix flaky `AsyncWiretapTest` in both `camel-telemetry` and 
`camel-telemetry-dev` modules. Both tests had ~5-7% flaky rate in CI.
   
   **Root cause:** The tests send 10 messages through an async wiretap route 
with a 2-second delay. Traces are written asynchronously after exchange 
processing. The assertions on trace data need to wait for the async writing to 
complete.
   
   **Changes:**
   - **camel-telemetry AsyncWiretapTest**: Replace `Thread.sleep(10000)` with 
Awaitility polling until all 10 traces with 7 spans each arrive. Replace 
`mock.setAssertPeriod(5000)` + `mock.assertIsSatisfied(1000)` with 
`MockEndpoint.assertIsSatisfied(context, 30, TimeUnit.SECONDS)`.
   - **camel-telemetry-dev AsyncWiretapTest**: Add Awaitility polling for trace 
completion (previously had no wait at all). Same mock assertion fix.
   
   ## Test plan
   
   - [x] Both 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