davsclaus commented on code in PR #27577: URL: https://github.com/apache/camel/pull/27577#discussion_r4219519844
########## components/camel-ai/camel-wolf-defender/src/test/java/org/apache/camel/component/wolfdefender/WolfDefenderModelIT.java: ########## @@ -37,6 +38,8 @@ import static org.assertj.core.api.Assertions.within; /** Explicitly opt in with a local, pinned model; this test never downloads artifacts. */ +@EnabledIf(value = "org.apache.camel.component.wolfdefender.WolfDefenderInferenceTest#supportedRuntime", Review Comment: Optional, not blocking: this ties the IT to a unit-test class through a string reference, so renaming or removing `WolfDefenderInferenceTest` would only break when the test runs. A small shared helper (e.g. `WolfDefenderPlatform.supportedRuntime()`) used by both classes would be a bit more robust. Fine to leave as is. -- 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]
