gnodet opened a new pull request, #2947: URL: https://github.com/apache/cxf/pull/2947
## Summary Addresses two separate test reliability issues: **Undertow (`UndertowHTTPServerEngineTest`)** - Added `@After` cleanup method to properly destroy server engines between tests - Without this, a test that creates an engine on a port but doesn't clean up causes subsequent tests to fail with port-bind errors **JMS (`AbstractJMSTester`)** - Fixed a TOCTOU race condition in `waitForReceiveInMessage()` and `waitForReceiveDestMessage()` - The null check on the `AtomicReference` was outside the `synchronized` block, so if a message arrived between checking `inMessage.get() == null` and calling `inMessage.wait()`, the `notifyAll()` was lost and the test waited the full 10-second timeout - Moved the null check inside the `synchronized` block to ensure the check-then-wait is atomic Note: This replaces #2946 which was too broad — per @reta's feedback, targeting fixes individually. ## Test plan - [x] `UndertowHTTPServerEngineTest` — 4 tests pass - [x] `JMSDestinationTest` — 16 tests pass (1 skipped) 🤖 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]
