svn commit: r1069455 - /camel/trunk/components/camel-spring/src/test/java/org/apache/camel/spring/DefaultJMXAgentTest.java

[da...@apache.org]

Author: davsclaus
Date: Thu Feb 10 16:25:46 2011
New Revision: 1069455

URL: http://svn.apache.org/viewvc?rev=1069455&view=rev
Log:
CAMEL-3641: Camel is less aggresive registering services in JMX. Now it only does that when starting and during adding new routes. This avoid some situations where we would enlist too many mbeans and eat memory.

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

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=1069455&r1=1069454&r2=1069455&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 Thu Feb 10 16:25:46 2011
@@ -66,10 +66,9 @@ public class DefaultJMXAgentTest extends
         // start route should enlist the consumer to JMX
         context.startRoute("foo");
 
-        // TODO: Fix me
         int after = mbsc.queryNames(new ObjectName("org.apache.camel" + ":type=consumers,*"), null).size();
 
-        //assertTrue("Should have added consumer to JMX, before: " + before + ", after: " + after, after > before);
+        assertTrue("Should have added consumer to JMX, before: " + before + ", after: " + after, after > before);
     }
 
     @Override