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

rcordier pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/james-project.git


The following commit(s) were added to refs/heads/master by this push:
     new ba2f5ddc6f JAMES-3747 Fixup RabbitMQEventBusTest
ba2f5ddc6f is described below

commit ba2f5ddc6f66eedc572478287482d388c6cb4da4
Author: Tung Tran <vtt...@linagora.com>
AuthorDate: Thu Jul 25 18:56:28 2024 +0700

    JAMES-3747 Fixup RabbitMQEventBusTest
    
    - `eventBus.register(listener, KEY_1)` is a publisher, we need to subscribe 
it
    - resolve deprecated method
---
 .../apache/james/events/RabbitMQEventBusTest.java   | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git 
a/event-bus/distributed/src/test/java/org/apache/james/events/RabbitMQEventBusTest.java
 
b/event-bus/distributed/src/test/java/org/apache/james/events/RabbitMQEventBusTest.java
index d0a093036b..d2593e9602 100644
--- 
a/event-bus/distributed/src/test/java/org/apache/james/events/RabbitMQEventBusTest.java
+++ 
b/event-bus/distributed/src/test/java/org/apache/james/events/RabbitMQEventBusTest.java
@@ -452,7 +452,7 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
             class DispatchingWhenNetWorkIssue {
 
                 @RegisterExtension
-                RabbitMQExtension rabbitMQNetWorkIssueExtension = 
RabbitMQExtension.defaultRabbitMQ()
+                static RabbitMQExtension rabbitMQNetWorkIssueExtension = 
RabbitMQExtension.defaultRabbitMQ()
                     .restartPolicy(DockerRestartPolicy.PER_TEST)
                     .isolationPolicy(RabbitMQExtension.IsolationPolicy.WEAK);
 
@@ -472,7 +472,7 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
                     rabbitMQNetWorkIssueExtension.getRabbitMQ().pause();
 
                     assertThatThrownBy(() -> 
rabbitMQEventBusWithNetWorkIssue.dispatch(EVENT, NO_KEYS).block())
-                        .getCause()
+                        .cause()
                         .isInstanceOf(NoSuchElementException.class)
                         .hasMessageContaining("Timeout waiting for idle 
object");
 
@@ -488,7 +488,8 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
                 eventBus.start();
                 assertThat(rabbitManagementAPI.listExchanges())
                     .filteredOn(exchange -> 
exchange.getName().equals(TEST_NAMING_STRATEGY.exchange()))
-                    .hasOnlyOneElementSatisfying(exchange -> {
+                    .singleElement()
+                    .satisfies(exchange -> {
                         assertThat(exchange.isDurable()).isTrue();
                         
assertThat(exchange.getType()).isEqualTo(DIRECT_EXCHANGE);
                     });
@@ -595,7 +596,7 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
                 rabbitMQExtension.getRabbitMQ().pause();
 
                 assertThatThrownBy(() -> eventBus.dispatch(EVENT, 
NO_KEYS).block())
-                        .getCause()
+                        .cause()
                         .isInstanceOf(NoSuchElementException.class)
                         .hasMessageContaining("Timeout waiting for idle 
object");
 
@@ -633,7 +634,7 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
                 rabbitMQExtension.getRabbitMQ().pause();
 
                 assertThatThrownBy(() -> eventBus.dispatch(EVENT, 
NO_KEYS).block())
-                        .getCause()
+                        .cause()
                         .isInstanceOf(NoSuchElementException.class)
                         .hasMessageContaining("Timeout waiting for idle 
object");
 
@@ -652,7 +653,7 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
                 rabbitMQExtension.getRabbitMQ().pause();
 
                 assertThatThrownBy(() -> eventBus.dispatch(EVENT, 
NO_KEYS).block())
-                        .getCause()
+                        .cause()
                         .isInstanceOf(NoSuchElementException.class)
                         .hasMessageContaining("Timeout waiting for idle 
object");
 
@@ -703,7 +704,7 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
                 EventListenerCountingSuccessfulExecution listener = new 
EventListenerCountingSuccessfulExecution();
                 eventBus.register(listener, GROUP_A);
 
-                try (Closeable closeable = ConcurrentTestRunner.builder()
+                try (Closeable ignored = ConcurrentTestRunner.builder()
                     .operation((threadNumber, step) -> 
eventBus.dispatch(EVENT, KEY_1).block())
                     .threadCount(THREAD_COUNT)
                     .operationCount(OPERATION_COUNT)
@@ -800,7 +801,7 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
                 eventBus.register(listener, GROUP_A);
                 eventBus2.register(listener, GROUP_A);
 
-                try (Closeable closeable = ConcurrentTestRunner.builder()
+                try (Closeable ignored = ConcurrentTestRunner.builder()
                     .operation((threadNumber, step) -> 
eventBus.dispatch(EVENT, KEY_1).block())
                     .threadCount(THREAD_COUNT)
                     .operationCount(OPERATION_COUNT)
@@ -857,8 +858,8 @@ class RabbitMQEventBusTest implements 
GroupContract.SingleEventBusGroupContract,
         void eventBusPubSubWithDistinctNamingStrategiesShouldBeIsolated() 
throws Exception {
             EventCollector listener = new EventCollector();
             EventCollector otherListener = new EventCollector();
-            eventBus.register(listener, KEY_1);
-            otherEventBus.register(otherListener, KEY_1);
+            Mono.from(eventBus.register(listener, KEY_1)).block();
+            Mono.from(otherEventBus.register(otherListener, KEY_1)).block();
 
             eventBus.dispatch(EVENT, ImmutableSet.of(KEY_1)).block();
 


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org
For additional commands, e-mail: notifications-h...@james.apache.org

Reply via email to