Add RSocketServiceMethod support for suspending functions

See #34868

Signed-off-by: Dmitry Sulman <dmitry.sulman@gmail.com>

Author: dmitrysulman

spring-messaging/src/main/java/org/springframework/messaging/rsocket/service/RSocketServiceMethod.java

@@ -30,6 +30,7 @@
 import reactor.core.publisher.Mono;
 
 import org.springframework.core.DefaultParameterNameDiscoverer;
+import org.springframework.core.KotlinDetector;
 import org.springframework.core.MethodParameter;
 import org.springframework.core.ParameterizedTypeReference;
 import org.springframework.core.ReactiveAdapter;
@@ -82,6 +83,10 @@ private static MethodParameter[] initMethodParameters(Method method) {
 		if (count == 0) {
 			return new MethodParameter[0];
 		}
+		if (KotlinDetector.isSuspendingFunction(method)) {
+			count -= 1;
+		}
+
 		DefaultParameterNameDiscoverer nameDiscoverer = new DefaultParameterNameDiscoverer();
 		MethodParameter[] parameters = new MethodParameter[count];
 		for (int i = 0; i < count; i++) {
@@ -129,10 +134,15 @@ private static Function<RSocketRequestValues, Object> initResponseFunction(
 
 		MethodParameter returnParam = new MethodParameter(method, -1);
 		Class<?> returnType = returnParam.getParameterType();
+		boolean isSuspending = KotlinDetector.isSuspendingFunction(method);
+		if (isSuspending) {
+			returnType = Mono.class;
+		}
+
 		ReactiveAdapter reactiveAdapter = reactiveRegistry.getAdapter(returnType);
 
 		MethodParameter actualParam = (reactiveAdapter != null ? returnParam.nested() : returnParam.nestedIfOptional());
-		Class<?> actualType = actualParam.getNestedParameterType();
+		Class<?> actualType = isSuspending ? actualParam.getParameterType() : actualParam.getNestedParameterType();
 
 		Function<RSocketRequestValues, Publisher<?>> responseFunction;
 		if (ClassUtils.isVoidType(actualType) || (reactiveAdapter != null && reactiveAdapter.isNoValue())) {
@@ -147,7 +157,8 @@ else if (reactiveAdapter == null) {
 		}
 		else {
 			ParameterizedTypeReference<?> payloadType =
-					ParameterizedTypeReference.forType(actualParam.getNestedGenericParameterType());
+					ParameterizedTypeReference.forType(isSuspending ? actualParam.getGenericParameterType() :
+							actualParam.getNestedGenericParameterType());
 
 			responseFunction = values -> (
 					reactiveAdapter.isMultiValue() ?

spring-messaging/src/main/java/org/springframework/messaging/rsocket/service/RSocketServiceProxyFactory.java

@@ -31,6 +31,7 @@
 
 import org.springframework.aop.framework.ProxyFactory;
 import org.springframework.aop.framework.ReflectiveMethodInvocation;
+import org.springframework.core.KotlinDetector;
 import org.springframework.core.MethodIntrospector;
 import org.springframework.core.ReactiveAdapterRegistry;
 import org.springframework.core.annotation.AnnotatedElementUtils;
@@ -246,7 +247,9 @@ private ServiceMethodInterceptor(List<RSocketServiceMethod> methods) {
 			Method method = invocation.getMethod();
 			RSocketServiceMethod serviceMethod = this.serviceMethods.get(method);
 			if (serviceMethod != null) {
-				return serviceMethod.invoke(invocation.getArguments());
+				@Nullable Object[] arguments = KotlinDetector.isSuspendingFunction(method) ?
+						resolveCoroutinesArguments(invocation.getArguments()) : invocation.getArguments();
+				return serviceMethod.invoke(arguments);
 			}
 			if (method.isDefault()) {
 				if (invocation instanceof ReflectiveMethodInvocation reflectiveMethodInvocation) {
@@ -256,6 +259,12 @@ private ServiceMethodInterceptor(List<RSocketServiceMethod> methods) {
 			}
 			throw new IllegalStateException("Unexpected method invocation: " + method);
 		}
+
+		private static Object[] resolveCoroutinesArguments(@Nullable Object[] args) {
+			Object[] functionArgs = new Object[args.length - 1];
+			System.arraycopy(args, 0, functionArgs, 0, args.length - 1);
+			return functionArgs;
+		}
 	}
 
 }

spring-messaging/src/test/kotlin/org/springframework/messaging/rsocket/service/RSocketServiceMethodKotlinTests.kt

@@ -0,0 +1,83 @@
+/*
+ * Copyright 2002-present the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.springframework.messaging.rsocket.service
+
+import io.rsocket.util.DefaultPayload
+import kotlinx.coroutines.runBlocking
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.jupiter.api.BeforeEach
+import org.junit.jupiter.api.Test
+import org.springframework.messaging.rsocket.RSocketRequester
+import org.springframework.messaging.rsocket.RSocketStrategies
+import org.springframework.messaging.rsocket.TestRSocket
+import org.springframework.util.MimeTypeUtils.TEXT_PLAIN
+import reactor.core.publisher.Mono
+
+/**
+ * Kotlin tests for [RSocketServiceMethod].
+ *
+ * @author Dmitry Sulman
+ */
+class RSocketServiceMethodKotlinTests {
+
+	private lateinit var rsocket: TestRSocket
+
+	private lateinit var proxyFactory: RSocketServiceProxyFactory
+
+	@BeforeEach
+	fun setUp() {
+		rsocket = TestRSocket()
+		val requester = RSocketRequester.wrap(rsocket, TEXT_PLAIN, TEXT_PLAIN, RSocketStrategies.create())
+		proxyFactory = RSocketServiceProxyFactory.builder(requester).build()
+	}
+
+	@Test
+	fun fireAndForget(): Unit = runBlocking {
+		val service = proxyFactory.createClient(SuspendingFunctionsService::class.java)
+
+		val requestPayload = "request"
+		service.fireAndForget(requestPayload)
+
+		assertThat(rsocket.savedMethodName).isEqualTo("fireAndForget")
+		assertThat(rsocket.savedPayload?.metadataUtf8).isEqualTo("ff")
+		assertThat(rsocket.savedPayload?.dataUtf8).isEqualTo(requestPayload)
+	}
+
+	@Test
+	fun requestResponse(): Unit = runBlocking {
+		val service = proxyFactory.createClient(SuspendingFunctionsService::class.java)
+
+		val requestPayload = "request"
+		val responsePayload = "response"
+		rsocket.setPayloadMonoToReturn(Mono.just(DefaultPayload.create(responsePayload)))
+		val response = service.requestResponse(requestPayload)
+
+		assertThat(response).isEqualTo(responsePayload)
+		assertThat(rsocket.savedMethodName).isEqualTo("requestResponse")
+		assertThat(rsocket.savedPayload?.metadataUtf8).isEqualTo("rr")
+		assertThat(rsocket.savedPayload?.dataUtf8).isEqualTo(requestPayload)
+	}
+
+	private interface SuspendingFunctionsService {
+
+		@RSocketExchange("ff")
+		suspend fun fireAndForget(input: String)
+
+		@RSocketExchange("rr")
+		suspend fun requestResponse(input: String): String
+	}
+}