svn commit: r1404474 - /camel/trunk/components/camel-quickfix/src/test/java/org/apache/camel/component/quickfixj/QuickfixjEngineTest.java

[bv...@apache.org]

Author: bvahdat
Date: Thu Nov  1 06:41:17 2012
New Revision: 1404474

URL: http://svn.apache.org/viewvc?rev=1404474&view=rev
Log:
CAMEL-5751: excluded the assert failing from time to time on JDK7.

Modified:
    camel/trunk/components/camel-quickfix/src/test/java/org/apache/camel/component/quickfixj/QuickfixjEngineTest.java

Modified: camel/trunk/components/camel-quickfix/src/test/java/org/apache/camel/component/quickfixj/QuickfixjEngineTest.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-quickfix/src/test/java/org/apache/camel/component/quickfixj/QuickfixjEngineTest.java?rev=1404474&r1=1404473&r2=1404474&view=diff
==============================================================================
--- camel/trunk/components/camel-quickfix/src/test/java/org/apache/camel/component/quickfixj/QuickfixjEngineTest.java (original)
+++ camel/trunk/components/camel-quickfix/src/test/java/org/apache/camel/component/quickfixj/QuickfixjEngineTest.java Thu Nov  1 06:41:17 2012
@@ -564,12 +564,14 @@ public class QuickfixjEngineTest extends
         assertThat(quickfixjEngine.getLogFactory(), instanceOf(ScreenLogFactory.class));
         assertThat(quickfixjEngine.getMessageFactory(), instanceOf(DefaultMessageFactory.class));
 
-        // TODO: just from time to time the following assert fails on JDK 7
-        if (!isJava17()) {
-            MBeanServer mbeanServer = ManagementFactory.getPlatformMBeanServer();
-            Set<ObjectName> names = mbeanServer.queryNames(new ObjectName("org.quickfixj:type=Connector,role=Acceptor,*"), null);
-            assertTrue("QFJ mbean should not have been registered", names.isEmpty());
+        if (isJava17()) {
+            // TODO: just from time to time the assert below fails on JDK 7
+            return;
         }
+
+        MBeanServer mbeanServer = ManagementFactory.getPlatformMBeanServer();
+        Set<ObjectName> names = mbeanServer.queryNames(new ObjectName("org.quickfixj:type=Connector,role=Acceptor,*"), null);
+        assertTrue("QFJ mbean should not have been registered", names.isEmpty());
     }
 
     private void writeSettings() throws IOException {