JiriOndrusek commented on PR #3847: URL: https://github.com/apache/camel-quarkus/pull/3847#issuecomment-1164349671
@aldettinger Thanks for your comment! > There is excludeTestSyntheticBeanDuplicities in InjectionPointProcesor. Is it useful to run it when not testing ? Would there be a way to do this step only in test mode ? if make sense I'll look into that in case it is possible to exclude duplications for the beans extendsing `CamelQuarkusTestSupport`, it would be great (or if the code could help me inn my code) > Is there a mix in the current test of the PR ? e.g. AzureStorageBlobProcessor is modified It shouldn't be. It is probably a result of the bad rebasing on my side. I will remove all unnecessary changes before making the PR final. > Naming is hard :) test-framework/junit5, if we implement pure annotation based tests in the future, it might also be called junit5 ? It could make sense to change the name of this module to `jvm-came-support` (or similar) to pinpoint tha fact, that it adds support of `CamelTestSupport` > The PR suggest that UseOverrideProperties is finally working ? Yes. > Would we gain value by adding a test like [RouteBuilderConfigureExceptionTest](https://github.com/apache/camel/blob/main/components/camel-test/camel-test-junit5/src/test/java/org/apache/camel/test/junit5/patterns/RouteBuilderConfigureExceptionTest.java) I'll look into that. -- 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]
