You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by da...@apache.org on 2010/07/11 10:05:48 UTC

svn commit: r963009 - in /camel/trunk: camel-core/src/test/java/org/apache/camel/management/JmxInstrumentationUsingDefaultsTest.java components/camel-spring/src/test/java/org/apache/camel/spring/DefaultJMXAgentTest.java

Author: davsclaus
Date: Sun Jul 11 08:05:47 2010
New Revision: 963009

URL: http://svn.apache.org/viewvc?rev=963009&view=rev
Log:
Fixed test due fork once.

Modified:
    camel/trunk/camel-core/src/test/java/org/apache/camel/management/JmxInstrumentationUsingDefaultsTest.java
    camel/trunk/components/camel-spring/src/test/java/org/apache/camel/spring/DefaultJMXAgentTest.java

Modified: camel/trunk/camel-core/src/test/java/org/apache/camel/management/JmxInstrumentationUsingDefaultsTest.java
URL: http://svn.apache.org/viewvc/camel/trunk/camel-core/src/test/java/org/apache/camel/management/JmxInstrumentationUsingDefaultsTest.java?rev=963009&r1=963008&r2=963009&view=diff
==============================================================================
--- camel/trunk/camel-core/src/test/java/org/apache/camel/management/JmxInstrumentationUsingDefaultsTest.java (original)
+++ camel/trunk/camel-core/src/test/java/org/apache/camel/management/JmxInstrumentationUsingDefaultsTest.java Sun Jul 11 08:05:47 2010
@@ -96,7 +96,7 @@ public class JmxInstrumentationUsingDefa
         assertEquals("Found mbeans: " + s, 1, s.size());
 
         Iterator<ObjectName> iter = s.iterator();
-        ObjectName pcob = (ObjectName)iter.next();
+        ObjectName pcob = iter.next();
 
         Long valueofNumExchanges = (Long)beanServer.getAttribute(pcob, "ExchangesTotal");
         assertNotNull("Expected attribute found. MBean registered under a "
@@ -190,7 +190,7 @@ public class JmxInstrumentationUsingDefa
     }
 
     protected void releaseMBeanServers() {
-        for (MBeanServer server : (List<MBeanServer>)MBeanServerFactory.findMBeanServer(null)) {
+        for (MBeanServer server : MBeanServerFactory.findMBeanServer(null)) {
             MBeanServerFactory.releaseMBeanServer(server);
         }
     }

Modified: camel/trunk/components/camel-spring/src/test/java/org/apache/camel/spring/DefaultJMXAgentTest.java
URL: http://svn.apache.org/viewvc/camel/trunk/components/camel-spring/src/test/java/org/apache/camel/spring/DefaultJMXAgentTest.java?rev=963009&r1=963008&r2=963009&view=diff
==============================================================================
--- camel/trunk/components/camel-spring/src/test/java/org/apache/camel/spring/DefaultJMXAgentTest.java (original)
+++ camel/trunk/components/camel-spring/src/test/java/org/apache/camel/spring/DefaultJMXAgentTest.java Sun Jul 11 08:05:47 2010
@@ -54,8 +54,7 @@ public class DefaultJMXAgentTest extends
 
     @SuppressWarnings("unchecked")
     protected void releaseMBeanServers() {
-        List<MBeanServer> servers =
-            (List<MBeanServer>)MBeanServerFactory.findMBeanServer(null);
+        List<MBeanServer> servers = MBeanServerFactory.findMBeanServer(null);
 
         for (MBeanServer server : servers) {
             MBeanServerFactory.releaseMBeanServer(server);
@@ -63,8 +62,11 @@ public class DefaultJMXAgentTest extends
     }
 
     public void testQueryMbeans() throws Exception {
-        assertEquals(1, mbsc.queryNames(new ObjectName("org.apache.camel" + ":type=routes,*"), null).size());
-        assertEquals(1, mbsc.queryNames(new ObjectName("org.apache.camel" + ":type=processors,*"), null).size());
+        int routes = mbsc.queryNames(new ObjectName("org.apache.camel" + ":type=routes,*"), null).size();
+        int processors = mbsc.queryNames(new ObjectName("org.apache.camel" + ":type=processors,*"), null).size();
+
+        assertTrue("Should contain routes", routes > 0);
+        assertTrue("Should contain processors", processors > 0);
     }
 
     @Override