You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2022/08/05 09:01:07 UTC

[GitHub] [camel] bvfalcon commented on a diff in pull request #8116: Unit-tests improvements

bvfalcon commented on code in PR #8116:
URL: https://github.com/apache/camel/pull/8116#discussion_r938606932


##########
core/camel-management/src/test/java/org/apache/camel/management/ManagedPooledExchangeTest.java:
##########
@@ -59,9 +60,7 @@ protected CamelContext createCamelContext() throws Exception {
     @Test
     public void testSameExchange() throws Exception {
         // JMX tests dont work well on AIX CI servers (hangs them)
-        if (isPlatform("aix")) {
-            return;
-        }
+        assumeFalse(isPlatform("aix"));

Review Comment:
   Excellent note! This annotation is much better, than assumption. I replace it there possible



-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org