dajac commented on code in PR #15221:
URL: https://github.com/apache/kafka/pull/15221#discussion_r1466057301
##########
group-coordinator/src/test/java/org/apache/kafka/coordinator/group/OffsetMetadataManagerTest.java:
##########
@@ -501,13 +521,11 @@ public void testOffsetDeleteWith(
final
OffsetDeleteResponseData.OffsetDeleteResponsePartitionCollection
expectedResponsePartitionCollection =
new
OffsetDeleteResponseData.OffsetDeleteResponsePartitionCollection();
- if (hasOffset(groupId, topic, partition)) {
Review Comment:
Basically, all the tests using `testOffsetDeleteWith` (this method) verify
it. I only had to remove the condition here in order to always expect the
response.
--
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]
