This is an automated email from the ASF dual-hosted git repository.

schofielaj pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 567a03dd149 HOTFIX: Parameter mismatch in test for 
BrokerMetadataPublisher (#19733)
567a03dd149 is described below

commit 567a03dd149c04e2f7d15b376ce9222a132c877f
Author: Andrew Schofield <[email protected]>
AuthorDate: Thu May 15 23:20:29 2025 +0100

    HOTFIX: Parameter mismatch in test for BrokerMetadataPublisher (#19733)
    
    Recent changes to BrokerMetadataPublisher seem to have caused a mismatch
    between the BrokerMetadataPublisher and its test.
    
    Reviewers: Apoorv Mittal <[email protected]>, Sushant Mahajan
     <[email protected]>
---
 .../unit/kafka/server/metadata/BrokerMetadataPublisherTest.scala      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/core/src/test/scala/unit/kafka/server/metadata/BrokerMetadataPublisherTest.scala
 
b/core/src/test/scala/unit/kafka/server/metadata/BrokerMetadataPublisherTest.scala
index 4ef0c0353c1..9a7c18ad079 100644
--- 
a/core/src/test/scala/unit/kafka/server/metadata/BrokerMetadataPublisherTest.scala
+++ 
b/core/src/test/scala/unit/kafka/server/metadata/BrokerMetadataPublisherTest.scala
@@ -202,6 +202,7 @@ class BrokerMetadataPublisherTest {
       groupCoordinator,
       mock(classOf[TransactionCoordinator]),
       mock(classOf[ShareCoordinator]),
+      mock(classOf[SharePartitionManager]),
       mock(classOf[DynamicConfigPublisher]),
       mock(classOf[DynamicClientQuotaPublisher]),
       mock(classOf[DynamicTopicClusterQuotaPublisher]),
@@ -209,8 +210,7 @@ class BrokerMetadataPublisherTest {
       mock(classOf[DelegationTokenPublisher]),
       mock(classOf[AclPublisher]),
       faultHandler,
-      faultHandler,
-      mock(classOf[SharePartitionManager])
+      faultHandler
     )
 
     val topicId = Uuid.randomUuid()

Reply via email to