From fdbf775763b0996c4f859eb415db767a2085c955 Mon Sep 17 00:00:00 2001 From: Allard Buijze Date: Tue, 13 Jan 2026 09:11:45 +0100 Subject: [PATCH 1/3] Updated dependency to TestContainer to 2.0 Configured maven build with Mockito agent to support JDK21+ where dynamic class definitions aren't allowed by default anymore. Removed some old references to org.junit.Assert, which is deprecated and no longer available after the upgrade. TestContainers update is (apparently) required for compatibility with new Docker APIs. --- pom.xml | 30 ++++++++++++++++--- .../admin/AdminChannelIntegrationTest.java | 4 +-- 2 files changed, 28 insertions(+), 6 deletions(-) diff --git a/pom.xml b/pom.xml index 7d1e7e30..fe59bedc 100644 --- a/pom.xml +++ b/pom.xml @@ -50,7 +50,7 @@ 4.33.1 2.0.16 - 2.25.2 + 2.25.3 1.3.2 @@ -60,7 +60,7 @@ 0.8.14 5.14.1 5.20.0 - 1.21.3 + 2.0.3 5.5.1 2.1.11 @@ -135,6 +135,21 @@ ${reactive.streams.version} test + + + + org.testng + testng + 7.11.0 + test + + + + org.apache.logging.log4j + log4j-core + ${log4j.version} + test + @@ -245,13 +260,13 @@ org.testcontainers - toxiproxy + testcontainers-toxiproxy ${testcontainers.version} test org.testcontainers - junit-jupiter + testcontainers-junit-jupiter ${testcontainers.version} test @@ -304,6 +319,12 @@ + + + + properties + + @@ -406,6 +427,7 @@ **/*Test_*.java **/*Tests_*.java + -javaagent:${org.mockito:mockito-core:jar} diff --git a/src/test/java/io/axoniq/axonserver/connector/admin/AdminChannelIntegrationTest.java b/src/test/java/io/axoniq/axonserver/connector/admin/AdminChannelIntegrationTest.java index af7e77a8..1c8dceb0 100644 --- a/src/test/java/io/axoniq/axonserver/connector/admin/AdminChannelIntegrationTest.java +++ b/src/test/java/io/axoniq/axonserver/connector/admin/AdminChannelIntegrationTest.java @@ -54,10 +54,10 @@ import static io.axoniq.axonserver.connector.testutils.AssertUtils.assertWithin; import static java.util.concurrent.TimeUnit.MILLISECONDS; import static java.util.concurrent.TimeUnit.SECONDS; -import static org.junit.Assert.assertFalse; -import static org.junit.Assert.fail; import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; import static org.junit.jupiter.api.Assertions.assertTrue; +import static org.junit.jupiter.api.Assertions.fail; import static org.mockito.Mockito.*; /** From a3502fa8043a4c2d21fb7fdc00c0186b714103fb Mon Sep 17 00:00:00 2001 From: Allard Buijze Date: Tue, 20 Jan 2026 08:50:39 -0500 Subject: [PATCH 2/3] Removed excessive logging from test case The DcbEndToEndTest printed all container logs to the test logging. While useful for debugging in certain cases, it is excessive as part of a regular build --- .../axoniq/axonserver/connector/event/dcb/DcbEndToEndTest.java | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/test/java/io/axoniq/axonserver/connector/event/dcb/DcbEndToEndTest.java b/src/test/java/io/axoniq/axonserver/connector/event/dcb/DcbEndToEndTest.java index 70be6730..d1b3381f 100644 --- a/src/test/java/io/axoniq/axonserver/connector/event/dcb/DcbEndToEndTest.java +++ b/src/test/java/io/axoniq/axonserver/connector/event/dcb/DcbEndToEndTest.java @@ -26,7 +26,6 @@ import org.junit.jupiter.api.*; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import org.testcontainers.containers.output.Slf4jLogConsumer; import reactor.core.publisher.Flux; import reactor.test.StepVerifier; @@ -92,8 +91,6 @@ void setUp() { } else { // Connect to Docker container builder.routingServers(axonServerAddress); - Slf4jLogConsumer logConsumer = new Slf4jLogConsumer(logger); - axonServerContainer.followOutput(logConsumer); } client = builder.build(); From 9bc29b3fd5abadc105ad251b29658703ecebbb73 Mon Sep 17 00:00:00 2001 From: Allard Buijze Date: Wed, 21 Jan 2026 15:51:16 -0600 Subject: [PATCH 3/3] Disabled the javadoc linter This seems to revert a change in default settings in a new javadoc plugin version. --- pom.xml | 1 + src/main/java/io/axoniq/axonserver/connector/Registration.java | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/pom.xml b/pom.xml index fe59bedc..3e328be4 100644 --- a/pom.xml +++ b/pom.xml @@ -444,6 +444,7 @@ UTF-8 + none apiNote diff --git a/src/main/java/io/axoniq/axonserver/connector/Registration.java b/src/main/java/io/axoniq/axonserver/connector/Registration.java index 6b7c045e..7b0afde2 100644 --- a/src/main/java/io/axoniq/axonserver/connector/Registration.java +++ b/src/main/java/io/axoniq/axonserver/connector/Registration.java @@ -58,7 +58,7 @@ default Registration awaitAck(long timeout, TimeUnit unit) throws TimeoutExcepti * Registers the given {@code runnable} to {@code this} {@link Registration} to be executed when the acknowledgement * of {@code this} {@link Registration} is received. Allows for the addition of further logic to {@code this * Registration}, like invoking {@link #awaitAck(long, TimeUnit)} for example. - *

+ *

* The given {@code runnable} is invoked, regardless of successful or failed acknowledgement. Use * {@link #onAck(Consumer)} to register an action that needs to distinguish between successful and failed * registration.