Skip to content

KAFKA-20812 Move additional KRaftClusterTest tests to KafkaClusterTestKitTest - #23635

Open
dino2895 wants to merge 1 commit into
apache:trunkfrom
dino2895:KAFKA-20812-1
Open

dino2895 wants to merge 1 commit into
apache:trunkfrom
dino2895:KAFKA-20812-1

Conversation

@dino2895

@dino2895 dino2895 commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Follow up on #23589 by moving seven additional cluster setup, lifecycle,
and controller tests from KRaftClusterTest to
KafkaClusterTestKitTest.

Preserve existing assertions and parameterized coverage, move the
dedicated helpers, and add the server-common test-fixture dependency
required by the advertised listener tests.

Test:
All 26 tests in KafkaClusterTestKitTest passed.

./gradlew :test-common:test-common-runtime:test \
  --tests org.apache.kafka.common.test.KafkaClusterTestKitTest

Reviewers: Ken Huang s7133700@gmail.com

@github-actions github-actions Bot added triage PRs from the community core Kafka Broker build Gradle build or GitHub Actions labels Sep 30, 2026

@m1a2st m1a2st left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions github-actions Bot removed the triage PRs from the community label Oct 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Gradle build or GitHub Actions ci-approved core Kafka Broker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants