Skip to content

Commit bf0e6ba

Browse files
authored
KAFKA-19384 The passing of BrokerRegistrationRequestTest is a false positive (#20338)
Fixes a false positive in `BrokerRegistrationRequestTest` caused by `isMigratingZkBroker`, and migrates the test from Scala to Java. Reviewers: Chia-Ping Tsai <[email protected]>
1 parent 990cb5c commit bf0e6ba

File tree

3 files changed

+249
-173
lines changed

3 files changed

+249
-173
lines changed

checkstyle/suppressions.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,7 @@
4949
<suppress checks="MethodLength|JavaNCSS|NPath" files="DescribeTopicPartitionsRequestHandlerTest.java"/>
5050
<suppress checks="CyclomaticComplexity" files="ListConsumerGroupTest.java"/>
5151
<suppress checks="ClassFanOutComplexity|CyclomaticComplexity|MethodLength|ParameterNumber|JavaNCSS|ImportControl" files="RequestConvertToJson.java"/>
52+
<suppress checks="ImportControl" files="BrokerRegistrationRequestTest.java"/>
5253

5354
<!-- Clients -->
5455
<suppress id="dontUseSystemExit"

core/src/test/scala/unit/kafka/server/BrokerRegistrationRequestTest.scala

Lines changed: 0 additions & 173 deletions
This file was deleted.

0 commit comments

Comments
 (0)