ppalaga commented on PR #5117: URL: https://github.com/apache/camel-quarkus/pull/5117#issuecomment-1688210157
> I am still looking whether we could merge the ITs into fewer Maven modules to save some time on the CI. Now I see that `camel-k-loader` does not have any assertions for native mode (there is no `*IT` class). Also based on what the tests are doing, I'd vote for transforming it into a lightweight deployment test - a PR will follow. `camel-k-runtime-xml` and `camel-k-runtime-yaml` could perhaps be merged together. Although we normally strive to test extensions with [minimal set of ancillary dependencies](https://camel.apache.org/camel-quarkus/2.16.x/contributor-guide/extension-testing.html#_minimal_set_of_dependencies) (which would be a reason to keep the two test modules separate), but in this specific case, we can say that (1) XML and YAML DSLs are tested in isolation elsewhere and (2) the Camel K extension is rather unlikely to be "fixed" by any of the two. Let me try to merge them together. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
