Add empty path test to rest-client-reactive

quarkusio · Jun 20, 2023 · d0f6637 · d0f6637
1 parent 73db38a
commit d0f6637
Show file tree

Hide file tree

Showing 4 changed files with 42 additions and 1 deletion.
diff --git a/...-reactive/deployment/src/test/java/io/quarkus/rest/client/reactive/ConfigurationTest.java b/...-reactive/deployment/src/test/java/io/quarkus/rest/client/reactive/ConfigurationTest.java
@@ -23,12 +23,16 @@ public class ConfigurationTest {
 
     @RegisterExtension
     static final QuarkusUnitTest TEST = new QuarkusUnitTest()
-            .withApplicationRoot(jar -> jar.addClasses(HelloClientWithBaseUri.class, EchoResource.class))
+            .withApplicationRoot(
+                    jar -> jar.addClasses(HelloClientWithBaseUri.class, EchoResource.class, EchoClientWithEmptyPath.class))
             .withConfigurationResource("configuration-test-application.properties");
 
     @RestClient
     HelloClientWithBaseUri client;
 
+    @RestClient
+    EchoClientWithEmptyPath echoClientWithEmptyPath;
+
     @Test
     void shouldHaveSingletonScope() {
         BeanManager beanManager = Arc.container().beanManager();
@@ -62,6 +66,11 @@ void checkClientSpecificConfigs() {
         verifyClientConfig(clientConfig, false);
     }
 
+    @Test
+    void emptyPathAnnotationShouldWork() {
+        assertThat(echoClientWithEmptyPath.echo("hello", "hello world")).isEqualTo("hello world");
+    }
+
     private void verifyClientConfig(RestClientConfig clientConfig, boolean checkExtraProperties) {
         assertThat(clientConfig.url).isPresent();
         assertThat(clientConfig.url.get()).endsWith("/hello");

diff --git a/...ive/deployment/src/test/java/io/quarkus/rest/client/reactive/EchoClientWithEmptyPath.java b/...ive/deployment/src/test/java/io/quarkus/rest/client/reactive/EchoClientWithEmptyPath.java
@@ -0,0 +1,24 @@
+package io.quarkus.rest.client.reactive;
+
+import jakarta.ws.rs.Consumes;
+import jakarta.ws.rs.GET;
+import jakarta.ws.rs.Path;
+import jakarta.ws.rs.PathParam;
+import jakarta.ws.rs.Produces;
+import jakarta.ws.rs.QueryParam;
+import jakarta.ws.rs.client.ClientRequestContext;
+import jakarta.ws.rs.client.ClientResponseContext;
+import jakarta.ws.rs.client.ClientResponseFilter;
+import jakarta.ws.rs.core.MediaType;
+import org.eclipse.microprofile.rest.client.inject.RegisterRestClient;
+
+@Path("")
+@RegisterRestClient
+public interface EchoClientWithEmptyPath {
+    @GET
+    @Produces(MediaType.TEXT_PLAIN)
+    @Consumes(MediaType.TEXT_PLAIN)
+    @Path("/{id}")
+    String echo(@PathParam("id") String id, @QueryParam("message") String message);
+
+}
diff --git a/.../deployment/src/test/java/io/quarkus/rest/client/reactive/configuration/EchoResource.java b/.../deployment/src/test/java/io/quarkus/rest/client/reactive/configuration/EchoResource.java
@@ -1,9 +1,11 @@
 package io.quarkus.rest.client.reactive.configuration;
 
 import jakarta.ws.rs.Consumes;
+import jakarta.ws.rs.GET;
 import jakarta.ws.rs.POST;
 import jakarta.ws.rs.Path;
 import jakarta.ws.rs.Produces;
+import jakarta.ws.rs.QueryParam;
 import jakarta.ws.rs.core.Context;
 import jakarta.ws.rs.core.HttpHeaders;
 import jakarta.ws.rs.core.MediaType;
@@ -21,4 +23,9 @@ public String echo(String name, @Context Request request, @Context HttpHeaders h
         return (message != null ? message : "hello") + (comma != null ? comma : "_") + " " + name
                 + (suffix != null ? suffix : "");
     }
+
+    @GET
+    public String echoQueryParam(@QueryParam("message") String message) {
+        return message;
+    }
 }
diff --git a/...t-client-reactive/deployment/src/test/resources/configuration-test-application.properties b/...t-client-reactive/deployment/src/test/resources/configuration-test-application.properties
@@ -18,6 +18,7 @@ quarkus.rest-client."io.quarkus.rest.client.reactive.HelloClientWithBaseUri".kee
 quarkus.rest-client."io.quarkus.rest.client.reactive.HelloClientWithBaseUri".max-redirects=5
 quarkus.rest-client."io.quarkus.rest.client.reactive.HelloClientWithBaseUri".headers.message=hi
 quarkus.rest-client."io.quarkus.rest.client.reactive.HelloClientWithBaseUri".headers.suffix=!
+quarkus.rest-client."io.quarkus.rest.client.reactive.EchoClientWithEmptyPath".url=http://localhost:${quarkus.http.test-port:8081}/
 
 # client identified by a configKey
 quarkus.rest-client.client-prefix.url=http://localhost:${quarkus.http.test-port:8081}/hello