JiriOndrusek commented on code in PR #3821:
URL: https://github.com/apache/camel-quarkus/pull/3821#discussion_r890113254
##########
integration-test-groups/foundation/bean/src/main/java/org/apache/camel/quarkus/component/bean/BeanResource.java:
##########
@@ -164,4 +169,18 @@ String awaitFirst(String key) {
return list.get(0);
}
+ @Path("/withDefaultBeanCount")
+ @GET
+ @Produces(MediaType.TEXT_PLAIN)
+ public int withDefaultBeanCount() {
+ return
camelContext.getRegistry().findByType(Producers.WithDefaultBeanInstance.class).size();
+ }
+
+ @Path("/withAlternativeBeanCount")
+ @GET
+ @Produces(MediaType.TEXT_PLAIN)
+ public int withAlternativeBeanCount() {
+ return
camelContext.getRegistry().findByType(Producers.WithAlternateBeanInstance.class).size();
+ }
Review Comment:
- I fixed tests to control also bean instances
- I added test to cover scenario with 2 beans (without default and
alternatives)
- I added test to cover scenario with more beans (from all tests covered by
this PR) - 1 of them is default and 1 of them is alternative - to cover
back-compatibility on ambiguous failure
--
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]
