apupier commented on code in PR #20500:
URL: https://github.com/apache/camel/pull/20500#discussion_r2630211004


##########
components/camel-milo/src/test/java/org/apache/camel/component/milo/MonitorItemMultiConnectionsCertTest.java:
##########
@@ -39,6 +41,8 @@
 /**
  * Testing monitoring items over multiple connections
  */
+// Disabled due to Milo 1.0.5 API migration - certificate-based authentication 
needs further investigation
+@Disabled

Review Comment:
   the reason can be put in the annotation directly.
   Advantages:
   - allows to see it in reports when tests are played
   - less chance to have an outdated comment remaining when the disbaled 
annotation is removed
   
   ```
   @Disabled("due to Milo 1.0.5 API migration - certificate-based 
authentication needs further investigation")



-- 
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]

Reply via email to