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

   ## Summary
   
   _Claude Code on behalf of gnodet_
   
   Fix flaky `XsltFromFileExceptionTest` by reordering assertions to wait for 
the file consumer to complete before checking mock expectations.
   
   **Root cause**: The test called `assertMockEndpointsSatisfied()` before 
`oneExchangeDone.matchesWaitTime()`. On slow CI machines, the file consumer may 
not have picked up and processed the file yet when the mock assertion starts, 
causing intermittent timeout failures.
   
   **Fix**:
   - Wait for `oneExchangeDone` latch first (confirms file consumer processed 
the file)
   - Then assert mock expectations (which now pass immediately since the 
exchange already completed)
   - Assert the `oneExchangeDone` result with `assertTrue()` for better failure 
diagnostics
   
   **Develocity data**: 6 failures out of recent runs.
   
   ## Test plan
   
   - [x] `XsltFromFileExceptionTest` passes locally (2 tests, 0 failures)
   - [ ] CI build passes
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   
   Co-Authored-By: Claude Opus 4.6 <[email protected]>


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