ppalaga commented on a change in pull request #2581:
URL: https://github.com/apache/camel-quarkus/pull/2581#discussion_r630993858



##########
File path: 
integration-tests/bean/src/main/java/org/apache/camel/quarkus/component/bean/BeanResource.java
##########
@@ -142,4 +142,27 @@ public String parameterBindingAnnotations(String 
statement, @PathParam("greeting
                 String.class);
     }
 
+    @Path("/produceInterface")
+    @POST
+    @Consumes(MediaType.TEXT_PLAIN)
+    @Produces(MediaType.TEXT_PLAIN)
+    public String produceInterface(String payload) {
+        produceInterface.sayHello(payload);

Review comment:
       Hm... you made me thinking. The purpose of the test is to make sure that 
the message goes via the Camel route starting with `direct:produceInterface`. 
Would there be any certainty if we just returned `sayHello(payload)`? I guess 
yes, but to be really sure, we'd have to add some transformation of the message 
to the route so that we see that it really passed it.

##########
File path: 
integration-tests/bean/src/main/java/org/apache/camel/quarkus/component/bean/BeanResource.java
##########
@@ -142,4 +142,27 @@ public String parameterBindingAnnotations(String 
statement, @PathParam("greeting
                 String.class);
     }
 
+    @Path("/produceInterface")
+    @POST
+    @Consumes(MediaType.TEXT_PLAIN)
+    @Produces(MediaType.TEXT_PLAIN)
+    public String produceInterface(String payload) {
+        produceInterface.sayHello(payload);

Review comment:
       I can try it in the next iteration, when implementing `@Consume`




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to