lianetm commented on code in PR #17694:
URL: https://github.com/apache/kafka/pull/17694#discussion_r1831174733
##########
group-coordinator/src/test/java/org/apache/kafka/coordinator/group/GroupMetadataManagerTest.java:
##########
@@ -15086,6 +15087,63 @@ public void testShareGroupDescribeOnConsumerGroup() {
assertEquals(expected, actual);
}
+ @Test
+ public void testReplayConsumerGroupRegularExpression() {
+ GroupMetadataManagerTestContext context = new
GroupMetadataManagerTestContext.Builder()
+ .build();
+
+ ResolvedRegularExpression resolvedRegularExpression = new
ResolvedRegularExpression(
+ Set.of("abc", "abcd"),
+ 10L,
+ 12345L
+ );
+
+
context.replay(GroupCoordinatorRecordHelpers.newConsumerGroupRegularExpressionRecord(
+ "foo",
+ "abc*",
+ resolvedRegularExpression
+ ));
+
+ assertEquals(
+ Optional.of(resolvedRegularExpression),
+
context.groupMetadataManager.consumerGroup("foo").regularExpression("abc*")
+ );
+ }
+
+ @Test
+ public void testReplayConsumerGroupRegularExpressionTombstone() {
+ GroupMetadataManagerTestContext context = new
GroupMetadataManagerTestContext.Builder()
+ .build();
+
+ // The group may not exist at all. Replaying the ConsumerGroupMetadata
tombstone
Review Comment:
```suggestion
// The group may not exist at all. Replaying the
ConsumerGroupRegularExpression tombstone
```
--
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]
