Polish stream Java test projects

Format, bump log dependencies, configure auto-format to get latest
dependencies.

deps/rabbitmq_stream/test/rabbit_stream_SUITE_data/pom.xml

@@ -26,14 +26,15 @@
     </developers>
 
     <properties>
-        <stream-client.version>[0.12.0-SNAPSHOT,)</stream-client.version>
+        <stream-client.version>[1.2.0-SNAPSHOT,)</stream-client.version>
         <junit.jupiter.version>5.13.3</junit.jupiter.version>
         <assertj.version>3.27.3</assertj.version>
-        <logback.version>1.2.13</logback.version>
+        <slf4j.version>2.0.17</slf4j.version>
+        <logback.version>1.5.18</logback.version>
         <maven.compiler.plugin.version>3.14.0</maven.compiler.plugin.version>
         <maven-surefire-plugin.version>3.5.3</maven-surefire-plugin.version>
         <spotless.version>2.44.5</spotless.version>
-        <google-java-format.version>1.17.0</google-java-format.version>
+        <google-java-format.version>1.27.0</google-java-format.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     </properties>
 
@@ -45,6 +46,12 @@
             <version>${stream-client.version}</version>
         </dependency>
 
+        <dependency>
+            <groupId>org.slf4j</groupId>
+            <artifactId>slf4j-api</artifactId>
+            <version>${slf4j.version}</version>
+        </dependency>
+
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter-engine</artifactId>
@@ -73,6 +80,14 @@
             <scope>test</scope>
         </dependency>
 
+        <!-- add explicitly to update automatically with dependabot -->
+        <dependency>
+            <groupId>com.google.googlejavaformat</groupId>
+            <artifactId>google-java-format</artifactId>
+            <version>${google-java-format.version}</version>
+            <scope>test</scope>
+        </dependency>
+
     </dependencies>
 
     <build>

deps/rabbitmq_stream/test/rabbit_stream_SUITE_data/src/test/java/com/rabbitmq/stream/ClusterSizeTest.java

@@ -11,7 +11,8 @@
 // The Original Code is RabbitMQ.
 //
 // The Initial Developer of the Original Code is Pivotal Software, Inc.
-// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
+// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom
+// Inc. and/or its subsidiaries. All rights reserved.
 //
 
 package com.rabbitmq.stream;

deps/rabbitmq_stream/test/rabbit_stream_SUITE_data/src/test/java/com/rabbitmq/stream/FailureTest.java

@@ -11,7 +11,8 @@
 // The Original Code is RabbitMQ.
 //
 // The Initial Developer of the Original Code is Pivotal Software, Inc.
-// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
+// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom
+// Inc. and/or its subsidiaries. All rights reserved.
 //
 
 package com.rabbitmq.stream;
@@ -33,7 +34,6 @@ import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.concurrent.atomic.AtomicInteger;
 import java.util.concurrent.atomic.AtomicLong;
 import java.util.concurrent.atomic.AtomicReference;
-
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -220,16 +220,23 @@ public class FailureTest {
             executorService.submit(
                 () -> {
                   connected.set(false);
-
+                  AtomicReference<Client> locator = new AtomicReference<>();
-                  try { Thread.sleep(2000); } catch (Exception e) {}
-                  Client locator =
-                      cf.get(new Client.ClientParameters().port(streamPortNode2()));
-                  // wait until there's a new leader
                   try {
                     waitAtMost(
                         Duration.ofSeconds(5),
                         () -> {
-                          Client.StreamMetadata m = locator.metadata(stream).get(stream);
+                          try {
+                            locator.set(
+                                cf.get(new Client.ClientParameters().port(streamPortNode2())));
+                            return true;
+                          } catch (Exception e) {
+                            return false;
+                          }
+                        });
+                    waitAtMost(
+                        Duration.ofSeconds(5),
+                        () -> {
+                          Client.StreamMetadata m = locator.get().metadata(stream).get(stream);
                           return m.getLeader() != null
                               && m.getLeader().getPort() != streamPortNode1();
                         });
@@ -238,7 +245,8 @@ public class FailureTest {
                     return;
                   }
 
-                  int newLeaderPort = locator.metadata(stream).get(stream).getLeader().getPort();
+                  int newLeaderPort =
+                      locator.get().metadata(stream).get(stream).getLeader().getPort();
                   Client newPublisher =
                       cf.get(
                           new Client.ClientParameters()
@@ -468,14 +476,23 @@ public class FailureTest {
             // avoid long-running task in the IO thread
             executorService.submit(
                 () -> {
-                  try { Thread.sleep(2000); } catch (Exception e) {}
+                  AtomicInteger newReplicaPort = new AtomicInteger(-1);
-                  Client.StreamMetadata m = metadataClient.metadata(stream).get(stream);
+                  waitAtMost(
-                  int newReplicaPort = m.getReplicas().get(0).getPort();
+                      Duration.ofSeconds(5),
+                      () -> {
+                        try {
+                          Client.StreamMetadata m = metadataClient.metadata(stream).get(stream);
+                          newReplicaPort.set(m.getReplicas().get(0).getPort());
+                          return true;
+                        } catch (Exception e) {
+                          return false;
+                        }
+                      });
 
                   Client newConsumer =
                       cf.get(
                           new Client.ClientParameters()
-                              .port(newReplicaPort)
+                              .port(newReplicaPort.get())
                               .shutdownListener(shutdownListenerReference.get())
                               .chunkListener(credit())
                               .messageListener(messageListener));
@@ -588,7 +605,8 @@ public class FailureTest {
   }
 
   @Test
-  void shouldReceiveMetadataUpdateWhenReplicaIsKilledWithPublisherAndConsumerOnSameConnection() throws Exception {
+  void shouldReceiveMetadataUpdateWhenReplicaIsKilledWithPublisherAndConsumerOnSameConnection()
+      throws Exception {
     Client metadataClient = cf.get(new Client.ClientParameters().port(streamPortNode1()));
     Map<String, Client.StreamMetadata> metadata = metadataClient.metadata(stream);
     Client.StreamMetadata streamMetadata = metadata.get(stream);
@@ -602,8 +620,7 @@ public class FailureTest {
     assertThat(streamMetadata.getLeader().getPort()).isEqualTo(streamPortNode1());
     Client.Broker broker =
         streamMetadata.getReplicas().stream()
-            .filter(
+            .filter(r -> r.getPort() == streamPortNode1() || r.getPort() == streamPortNode2())
-                r -> r.getPort() == streamPortNode1() || r.getPort() == streamPortNode2())
             .findFirst()
             .get();
 
@@ -612,8 +629,7 @@ public class FailureTest {
         cf.get(
             new ClientParameters()
                 .port(broker.getPort())
-                .metadataListener(
+                .metadataListener((stream, code) -> metadataNotifications.incrementAndGet()));
-                    (stream, code) -> metadataNotifications.incrementAndGet()));
     client.declarePublisher((byte) 42, null, stream);
     client.subscribe((byte) 66, stream, OffsetSpecification.first(), 1);
 

deps/rabbitmq_stream/test/rabbit_stream_SUITE_data/src/test/java/com/rabbitmq/stream/Host.java

@@ -11,7 +11,8 @@
 // The Original Code is RabbitMQ.
 //
 // The Initial Developer of the Original Code is Pivotal Software, Inc.
-// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
+// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom
+// Inc. and/or its subsidiaries. All rights reserved.
 //
 
 package com.rabbitmq.stream;
@@ -101,7 +102,8 @@ public class Host {
     return System.getProperty("node2.name", "rabbit-2@" + hostname());
   }
 
-  public static Process killStreamLocalMemberProcess(String stream, String nodename) throws IOException {
+  public static Process killStreamLocalMemberProcess(String stream, String nodename)
+      throws IOException {
     return rabbitmqctl(
         "eval 'case rabbit_stream_manager:lookup_local_member(<<\"/\">>, <<\""
             + stream

deps/rabbitmq_stream/test/rabbit_stream_SUITE_data/src/test/java/com/rabbitmq/stream/LeaderLocatorTest.java

@@ -11,7 +11,8 @@
 // The Original Code is RabbitMQ.
 //
 // The Initial Developer of the Original Code is Pivotal Software, Inc.
-// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
+// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom
+// Inc. and/or its subsidiaries. All rights reserved.
 //
 
 package com.rabbitmq.stream;
@@ -27,8 +28,8 @@ import com.rabbitmq.stream.impl.Client.Broker;
 import com.rabbitmq.stream.impl.Client.ClientParameters;
 import com.rabbitmq.stream.impl.Client.Response;
 import com.rabbitmq.stream.impl.Client.StreamMetadata;
-import java.util.Collections;
 import java.time.Duration;
+import java.util.Collections;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Set;
@@ -58,9 +59,8 @@ public class LeaderLocatorTest {
   void clientLocalLocatorShouldMakeLeaderOnConnectedNode() {
     int[] ports = new int[] {TestUtils.streamPortNode1(), TestUtils.streamPortNode2()};
     for (int port : ports) {
-      Client client = cf.get(new Client.ClientParameters()
+      Client client =
-              .port(port)
+          cf.get(new Client.ClientParameters().port(port).rpcTimeout(Duration.ofSeconds(30)));
-              .rpcTimeout(Duration.ofSeconds(30)));
       String s = UUID.randomUUID().toString();
       try {
         Response response =

deps/rabbitmq_stream/test/rabbit_stream_SUITE_data/src/test/java/com/rabbitmq/stream/StreamTest.java

@@ -11,7 +11,8 @@
 // The Original Code is RabbitMQ.
 //
 // The Initial Developer of the Original Code is Pivotal Software, Inc.
-// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
+// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom
+// Inc. and/or its subsidiaries. All rights reserved.
 //
 
 package com.rabbitmq.stream;

deps/rabbitmq_stream/test/rabbit_stream_SUITE_data/src/test/java/com/rabbitmq/stream/TestUtils.java

@@ -11,7 +11,8 @@
 // The Original Code is RabbitMQ.
 //
 // The Initial Developer of the Original Code is Pivotal Software, Inc.
-// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
+// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom
+// Inc. and/or its subsidiaries. All rights reserved.
 //
 
 package com.rabbitmq.stream;
@@ -52,7 +53,7 @@ public class TestUtils {
     waitAtMost(Duration.ofSeconds(10), condition);
   }
 
-  static void waitAtMost(Duration duration, BooleanSupplier condition) throws InterruptedException {
+  static void waitAtMost(Duration duration, BooleanSupplier condition) {
     if (condition.getAsBoolean()) {
       return;
     }
@@ -60,7 +61,12 @@ public class TestUtils {
     int waitedTime = 0;
     long timeoutInMs = duration.toMillis();
     while (waitedTime <= timeoutInMs) {
-      Thread.sleep(waitTime);
+      try {
+        Thread.sleep(waitTime);
+      } catch (InterruptedException e) {
+        Thread.currentThread().interrupt();
+        throw new RuntimeException(e);
+      }
       if (condition.getAsBoolean()) {
         return;
       }

deps/rabbitmq_stream_management/test/http_SUITE_data/pom.xml

@@ -26,14 +26,15 @@
     </developers>
 
     <properties>
-        <stream-client.version>[0.12.0-SNAPSHOT,)</stream-client.version>
+        <stream-client.version>[1.2.0-SNAPSHOT,)</stream-client.version>
         <junit.jupiter.version>5.13.3</junit.jupiter.version>
         <assertj.version>3.27.3</assertj.version>
-        <logback.version>1.2.13</logback.version>
+        <slf4j.version>2.0.17</slf4j.version>
+        <logback.version>1.5.18</logback.version>
         <maven.compiler.plugin.version>3.14.0</maven.compiler.plugin.version>
         <maven-surefire-plugin.version>3.5.3</maven-surefire-plugin.version>
         <spotless.version>2.44.5</spotless.version>
-        <google-java-format.version>1.18.1</google-java-format.version>
+        <google-java-format.version>1.27.0</google-java-format.version>
         <okhttp.version>5.0.0</okhttp.version>
         <gson.version>2.13.1</gson.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
@@ -47,6 +48,12 @@
             <version>${stream-client.version}</version>
         </dependency>
 
+        <dependency>
+            <groupId>org.slf4j</groupId>
+            <artifactId>slf4j-api</artifactId>
+            <version>${slf4j.version}</version>
+        </dependency>
+
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter-engine</artifactId>
@@ -89,6 +96,14 @@
             <scope>test</scope>
         </dependency>
 
+        <!-- add explicitly to update automatically with dependabot -->
+        <dependency>
+            <groupId>com.google.googlejavaformat</groupId>
+            <artifactId>google-java-format</artifactId>
+            <version>${google-java-format.version}</version>
+            <scope>test</scope>
+        </dependency>
+
     </dependencies>
 
     <build>

deps/rabbitmq_stream_management/test/http_SUITE_data/src/test/java/com/rabbitmq/stream/HttpTest.java

@@ -11,7 +11,8 @@
 // The Original Code is RabbitMQ.
 //
 // The Initial Developer of the Original Code is Pivotal Software, Inc.
-// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
+// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom
+// Inc. and/or its subsidiaries. All rights reserved.
 //
 
 package com.rabbitmq.stream;

deps/rabbitmq_stream_management/test/http_SUITE_data/src/test/java/com/rabbitmq/stream/TestUtils.java

@@ -11,7 +11,8 @@
 // The Original Code is RabbitMQ.
 //
 // The Initial Developer of the Original Code is Pivotal Software, Inc.
-// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom Inc. and/or its subsidiaries. All rights reserved.
+// Copyright (c) 2007-2025 Broadcom. All Rights Reserved. The term “Broadcom” refers to Broadcom
+// Inc. and/or its subsidiaries. All rights reserved.
 //
 
 package com.rabbitmq.stream;