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

   [CAMEL-23431](https://issues.apache.org/jira/browse/CAMEL-23431)
   
   ## Summary
   
   Third PR for CAMEL-23431. Migrates all `camel-thrift` tests (10 test files) 
from `AvailablePortFinder` to port-0 binding, completely eliminating 
`AvailablePortFinder` from the thrift module.
   
   **Production code:**
   - Add `getLocalPort()` to `ThriftConsumer` to expose the OS-assigned port 
after server start
   
   **Producer tests (create own server):**
   - `ThriftProducerBaseTest`: Move server creation to `doPreSetup()`, bind 
`TNonblockingServerSocket` to port 0
   - `ThriftProducerSecurityTest`: Same pattern with `TSSLTransportFactory`
   - `ThriftProducerZlibCompressionTest`: Same pattern with `TServerSocket`
   - `ThriftThreadPoolServerTest`: Bind to port 0 in `@BeforeEach`
   
   **Consumer tests (Camel component creates server):**
   - All 5 consumer tests use port 0 in `from()` URI, then read back actual 
port from `ThriftConsumer.getLocalPort()` for client connections
   - `ThriftConsumerConcurrentTest`: 2 ports (sync + async) both use port 0
   
   ## Test plan
   - [x] `mvn test` in `camel-thrift` — 39 tests pass
   - [ ] CI green
   


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