Skip to content

Commit

Permalink
KAFKA-18320; Ensure that assignors are at the right place (#18750)
Browse files Browse the repository at this point in the history
The full class name of the assignors if part of our public api. Hence, we should ensure that they are not changed by mistake. This patch adds a unit test verifying them.

Reviewers: Sean Quah <squah@confluent.io>, Jeff Kim <jeff.kim@confluent.io>
  • Loading branch information
dajac authored Jan 31, 2025
1 parent 0ff4daf commit d19b605
Showing 1 changed file with 13 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,19 @@ public class GroupCoordinatorConfigTest {
GroupCoordinatorConfig.CONSUMER_GROUP_CONFIG_DEF,
GroupCoordinatorConfig.SHARE_GROUP_CONFIG_DEF);

@Test
public void testConsumerGroupAssignorsDefault() {
// The full class name of the assignors is part of our public api. Hence,
// we should ensure that they are not changed by mistake.
assertEquals(
List.of(
"org.apache.kafka.coordinator.group.assignor.UniformAssignor",
"org.apache.kafka.coordinator.group.assignor.RangeAssignor"
),
GroupCoordinatorConfig.CONSUMER_GROUP_ASSIGNORS_DEFAULT
);
}

@Test
public void testConfigs() {
Map<String, Object> configs = new HashMap<>();
Expand Down

0 comments on commit d19b605

Please # to comment.