[KAFKA-16737] Clean up KafkaConsumerTest TODOs enabling tests for new consumer

• Type: Task
• Status: Resolved
• Priority: Blocker
• Resolution: Duplicate
• Affects Version/s: 3.7.0
• Component/s: clients, consumer
• Labels: kip-848-client-support
• Assignee: Lianet Magrans
• Reporter: Lianet Magrans
• Created: 13/May/24 13:33
• Updated: 17/Sep/24 16:52
• Resolved: 10/Jul/24 15:37

Description

KafkaConsumerTest.java contains lots of TODOs (50+) related to tests that are only enabled for the CLASSIC protocol and should be reviewed and enabled for the new CONSUMER group protocol when applicable. Some tests also have TODOs to enable them for the new consumer when certain features/bugs are addressed.

The new protocol and consumer implementation have evolved a lot since those TODOs where added, so we should review them all, enable tests for the new protocol when applicable and removing the TODOs from the code. Note that there is another AsyncKafkaConsumerTest.java, testing logic specific to the internals of the new consumer, but still many tests in the KafkaConsumerTest apply to both the new and legacy consumer, and we should enable them for both.

Issue Links

• relates to: KAFKA-16957 Enable KafkaConsumerTest#configurableObjectsShouldSeeGeneratedClientId to work with CLASSIC and CONSUMER
• relates to: KAFKA-17110 Enable valid test case in KafkaConsumerTest for AsyncKafkaConsumer
• links to: GitHub Pull Request #16032