lianetm opened a new pull request, #16032: URL: https://github.com/apache/kafka/pull/16032
KafkaConsumerTest contained lots of unit tests that were not enabled for the new consumer, with TODOs for verifying if they cold be enabled. This PR enables all the unit tests that could be applied/pass for the new consumer (~16 unit tests enabled) and cleans up the TODOs. The tests that remain being applied to the legacy consumer only cannot run for the new consumer, due to: 1. bugs in the new consumer. Only 5 tests in this case, for those I left TODOs just to reference the Jira I filed to make sure the tests are enabled as soon as the bugs are fixed (only TODOs intentionally left) 2. the way the test is written or what it is testing is specific to the legacy consumer. -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
