You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by cu...@apache.org on 2010/10/21 19:58:47 UTC

svn commit: r1026086 - /openjpa/trunk/openjpa-integration/jmx/src/test/java/org/apache/openjpa/integration/jmx/TestJMXPlatformMBeans.java

Author: curtisr7
Date: Thu Oct 21 17:58:46 2010
New Revision: 1026086

URL: http://svn.apache.org/viewvc?rev=1026086&view=rev
Log:
OPENJPA-1844: Fix TestJMXPlatformMBeans.

Modified:
    openjpa/trunk/openjpa-integration/jmx/src/test/java/org/apache/openjpa/integration/jmx/TestJMXPlatformMBeans.java

Modified: openjpa/trunk/openjpa-integration/jmx/src/test/java/org/apache/openjpa/integration/jmx/TestJMXPlatformMBeans.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-integration/jmx/src/test/java/org/apache/openjpa/integration/jmx/TestJMXPlatformMBeans.java?rev=1026086&r1=1026085&r2=1026086&view=diff
==============================================================================
--- openjpa/trunk/openjpa-integration/jmx/src/test/java/org/apache/openjpa/integration/jmx/TestJMXPlatformMBeans.java (original)
+++ openjpa/trunk/openjpa-integration/jmx/src/test/java/org/apache/openjpa/integration/jmx/TestJMXPlatformMBeans.java Thu Oct 21 17:58:46 2010
@@ -83,13 +83,9 @@ public class TestJMXPlatformMBeans exten
         ce = oem.find(CachedEntity.class, id);
         
         assertTrue(dci.getHitCount() > 0);
-        try {
-            assertTrue(dci.getHitCount(CachedEntity.class.getName()) > 0);
-        } catch (ClassNotFoundException e) {
-            fail("CachedEntity class not found");
-        }
+        assertTrue(dci.getHitCount(CachedEntity.class.getName()) > 0);
         assertTrue(dci.getWriteCount() > 0);
-        assertTrue(dci.classNames().contains(CachedEntity.class.getName()));
+        assertTrue(dci.getCacheStatistics().classNames().contains(CachedEntity.class.getName()));
         // Thread out to do out-of-band MBean-based validation.  This could
         // have been done on the same thread, but threading out makes for a
         // more realistic test.