aldettinger commented on a change in pull request #3124:
URL: https://github.com/apache/camel-quarkus/pull/3124#discussion_r714827343



##########
File path: 
integration-tests/kafka/src/test/java/org/apache/camel/quarkus/component/kafka/it/CamelKafkaTest.java
##########
@@ -86,4 +89,127 @@ void testQuarkusKafkaClientFactoryNotConfigured() {
                 .statusCode(200)
                 .body(is("true"));
     }
+
+    @Test

Review comment:
       Thinking aloud:
   We would have a kind of counter in the process() statement
   The process statement would do something like that setBody(counter)
   Send "content" to kafka, assert seda as seen counter = 1 and counter 2 (so 
delivered uncommitted and then delivered committed).
   That way, we wouldn't need the last startup ? Would it cover everything 
needed thought ?




-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

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


Reply via email to