aldettinger commented on a change in pull request #876: Fixed kafka itests as
@Inject is not supported in native tests
URL: https://github.com/apache/camel-quarkus/pull/876#discussion_r391110003
##########
File path:
integration-tests/kafka/src/test/java/org/apache/camel/quarkus/component/kafka/it/CamelKafkaTestResource.java
##########
@@ -32,14 +31,6 @@
private static final String CONFLUENT_PLATFORM_VERSION = "5.3.1";
private KafkaContainer container;
- private CamelMain main;
-
- @Override
- public void inject(Object testInstance) {
- if (testInstance instanceof CamelKafkaTest) {
- this.main = ((CamelKafkaTest) testInstance).main;
Review comment:
There must be 2 subjects:
\@Inject not supported in native mode is documented
[there](https://github.com/quarkusio/quarkus/blob/master/docs/src/main/asciidoc/getting-started-testing.adoc#9-native-executable-testing),
so it could have behaved differently previously but it's no more the case.
`testInstance instanceof CamelKafkaTest == false` because of distinct class
loaders, I found no mention, so I would better open a question on quarkus side
so that we know what behavior we can rely on.
----------------------------------------------------------------
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:
us...@infra.apache.org
With regards,
Apache Git Services
