Fix ConditionalRejectingErrorHandler for IllegalStateException

Related to: #3396

Now that `DelegatingInvocableHandler` throws an `IllegalStateException` around `NoSuchMethodException`,
the logic in the `ConditionalRejectingErrorHandler` for `isFatal()` is not usable anymore.

* Change the `ConditionalRejectingErrorHandler.DefaultExceptionStrategy.isFatal()` logic
to rely on the `IllegalStateException` instead of `UndeclaredThrowableException`
to trigger expected by this error handler `AmqpRejectAndDontRequeueException` recovery

Author: artembilan

spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/ConditionalRejectingErrorHandler.java

@@ -202,7 +202,10 @@ public static class DefaultExceptionStrategy implements FatalExceptionStrategy {
 		@Override
 		public boolean isFatal(Throwable t) {
 			Throwable cause = t.getCause();
-			while ((cause instanceof MessagingException || cause instanceof  UndeclaredThrowableException)
+			while ((cause instanceof MessagingException
+					|| cause instanceof UndeclaredThrowableException
+					|| cause instanceof IllegalStateException)
+
 					&& !isCauseFatal(cause)) {
 
 				cause = cause.getCause();

spring-rabbit/src/test/java/org/springframework/amqp/rabbit/annotation/EnableRabbitIntegrationTests.java

@@ -141,6 +141,7 @@
 import org.springframework.transaction.annotation.EnableTransactionManagement;
 import org.springframework.transaction.annotation.Transactional;
 import org.springframework.util.ErrorHandler;
+import org.springframework.util.ReflectionUtils;
 import org.springframework.validation.Errors;
 import org.springframework.validation.Validator;
 import org.springframework.validation.annotation.Validated;
@@ -2072,6 +2073,14 @@ org.springframework.amqp.core.Queue brokerNamed() {
 			return QueueBuilder.nonDurable("").autoDelete().exclusive().build();
 		}
 
+		@Bean
+		public SimpleRabbitListenerContainerFactory noSuchMethodListenerContainerFactory() {
+			SimpleRabbitListenerContainerFactory factory = new SimpleRabbitListenerContainerFactory();
+			factory.setConnectionFactory(rabbitConnectionFactory());
+			factory.setErrorHandler(ReflectionUtils::rethrowRuntimeException);
+			return factory;
+		}
+
 		@Bean
 		NoSuchMethodService noSuchMethodService() {
 			return new NoSuchMethodService();
@@ -2509,7 +2518,8 @@ public String projection(Sample in) {
 		}
 	}
 
-	@RabbitListener(id = "no.such.method", queues = "no.such.method")
+	@RabbitListener(id = "no.such.method", queues = "no.such.method",
+			containerFactory = "noSuchMethodListenerContainerFactory")
 	static class NoSuchMethodService implements ApplicationListener<ListenerContainerConsumerFailedEvent> {
 
 		CountDownLatch eventLatch = new CountDownLatch(1);