Bad check for Mono return type in org.springframework.amqp.rabbit.listener.adapter.HandlerAdapter constructor

[andruskutt]

In class:
https://github.com/spring-projects/spring-amqp/blob/main/spring-rabbit/src/main/java/org/springframework/amqp/rabbit/listener/adapter/HandlerAdapter.java#L52

there seems to be check for Mono - MonoHandler.isMono(invokerHandlerMethod.getMethod().getReturnType()))
But MonoHandler.isMono contains "instanceof" check, not "isAssignableFrom", so this check will be always false, because check is done against Class<?>

[artembilan]

It looks like we need an overloaded:

	static boolean isMono(Class<?> result) {
		return Mono.class.isAssignableFrom(result);
	}

We still use an existing one in the AbstractAdaptableMessageListener.

I guess it works OK with the Mono processing, but that isAsyncReplies() has some effects on the acknowledges in some container implementations...

[garyrussell]

Right;

I added this to the test

		assertThat(TestUtils.getPropertyValue(registry.getListenerContainer("foo"), "asyncReplies", Boolean.class))
				.isTrue();
		assertThat(TestUtils.getPropertyValue(registry.getListenerContainer("bar"), "asyncReplies", Boolean.class))
				.isTrue();

And the second one fails.