gnodet commented on PR #2971: URL: https://github.com/apache/cxf/pull/2971#issuecomment-4141536154
Following up on @reta's [comment](https://github.com/apache/cxf/pull/2971#issuecomment-4135618558), I've created #2999 which improves the diagnostic output when these tests fail. The change replaces `await().until(() -> spans.size() == N)` with `await().untilAsserted(() -> assertThat("Unexpected span count. Spans: " + spanList, size, equalTo(N)))` across all tracing tests (Brave, OTel, OpenTracing; JAX-RS and JAX-WS). When CI next fails, the error message will show the actual span count and the full list of collected spans, which should help us determine whether the root cause is: - **Missing spans** (count < expected) — span never created/ended - **Extra spans** (count > expected) — cross-test contamination - **Timing** — spans eventually arrive but not within 5 seconds This should help narrow down the deeper issue beyond the `TestSpanHandler` thread-safety fix. _Claude Code on behalf of Guillaume Nodet_ -- 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]
