Skip to content

Commit 58d4e72

Browse files
committed
Work Around HOP/REST Incompatibility in Test
1 parent 1ebde8e commit 58d4e72

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

spring-rabbit/src/main/java/org/springframework/amqp/rabbit/config/StatelessRetryOperationsInterceptorFactoryBean.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ protected MethodInvocationRecoverer<?> createRecoverer() {
6767
return this::recover;
6868
}
6969

70+
@SuppressWarnings("unchecked")
7071
protected Object recover(Object[] args, Throwable cause) {
7172
MessageRecoverer messageRecoverer = getMessageRecoverer();
7273
Object arg = args[1];

spring-rabbit/src/test/java/org/springframework/amqp/rabbit/core/RabbitRestApiTests.java

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,10 @@
1717
package org.springframework.amqp.rabbit.core;
1818

1919
import static org.assertj.core.api.Assertions.assertThat;
20+
import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
2021
import static org.awaitility.Awaitility.await;
2122

23+
import java.io.IOException;
2224
import java.net.MalformedURLException;
2325
import java.net.URISyntaxException;
2426
import java.util.Collections;
@@ -29,6 +31,7 @@
2931
import org.junit.jupiter.api.AfterEach;
3032
import org.junit.jupiter.api.Test;
3133

34+
import org.springframework.amqp.AmqpException;
3235
import org.springframework.amqp.core.Binding;
3336
import org.springframework.amqp.core.BindingBuilder;
3437
import org.springframework.amqp.core.DirectExchange;
@@ -217,7 +220,12 @@ public void testDeleteExchange() {
217220
assertThat(exchangeToAssert.getName()).isEqualTo(testExchange.getName());
218221
assertThat(exchangeToAssert.getType()).isEqualTo(testExchange.getType());
219222
this.rabbitRestClient.deleteExchange("/", testExchange.getName());
220-
assertThat(this.rabbitRestClient.getExchange("/", exchangeName)).isNull();
223+
// 6.0.0 REST compatibility
224+
// assertThat(this.rabbitRestClient.getExchange("/", exchangeName)).isNull();
225+
RabbitTemplate template = new RabbitTemplate(this.connectionFactory);
226+
assertThatExceptionOfType(AmqpException.class)
227+
.isThrownBy(() -> template.execute(channel -> channel.exchangeDeclarePassive(exchangeName)))
228+
.withCauseExactlyInstanceOf(IOException.class);
221229
}
222230

223231
}

0 commit comments

Comments
 (0)