aldettinger commented on pull request #2614:
URL: https://github.com/apache/camel-quarkus/pull/2614#issuecomment-843317905


   Nice addition, the test structure is good. In terms of coverage, there is a 
single point I'm not sure about. The 
[tokenize-language](https://camel.apache.org/components/latest/languages/tokenize-language.html)
 page references an `xml` option which is false by default. Is it still in use 
? Is it worth testing that ? I would be thinking about XML dependencies that 
might be needed for instance.


-- 
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:
[email protected]


Reply via email to