mmodzelewski opened a new pull request, #2959: URL: https://github.com/apache/iggy/pull/2959
Async integration tests duplicated CRUD coverage already exercised by the blocking test suite (which wraps the same async code path). This led to slow CI and made it unclear where to add new test cases. Removed AsyncPollMessageTest and UsersTcpClientTest entirely, slimmed AsyncClientIntegrationTest and AsyncConsumerGroupsTest to keep only async-specific scenarios (concurrency, chaining, error propagation). Moved missing coverage (polling strategies, content round-trip, user updates, password changes) into the blocking base tests where both TCP and HTTP transports benefit. Also hardened client builders: buildAndLogin() now closes the client on failure instead of leaking connections. TCP connection allows PING and GET_STATS without authentication, unblocking the BDD ping step. Added debug-level version logging on connect. -- 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]
