You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by pc...@apache.org on 2006/10/27 07:41:03 UTC

svn commit: r468283 - /incubator/openjpa/trunk/openjpa-kernel/src/test/java/org/apache/openjpa/util/TestWASTransformer.java

Author: pcl
Date: Thu Oct 26 22:41:03 2006
New Revision: 468283

URL: http://svn.apache.org/viewvc?view=rev&rev=468283
Log:
changed test case to include more debug info in failures. This test case does not pass on my machine.

Modified:
    incubator/openjpa/trunk/openjpa-kernel/src/test/java/org/apache/openjpa/util/TestWASTransformer.java

Modified: incubator/openjpa/trunk/openjpa-kernel/src/test/java/org/apache/openjpa/util/TestWASTransformer.java
URL: http://svn.apache.org/viewvc/incubator/openjpa/trunk/openjpa-kernel/src/test/java/org/apache/openjpa/util/TestWASTransformer.java?view=diff&rev=468283&r1=468282&r2=468283
==============================================================================
--- incubator/openjpa/trunk/openjpa-kernel/src/test/java/org/apache/openjpa/util/TestWASTransformer.java (original)
+++ incubator/openjpa/trunk/openjpa-kernel/src/test/java/org/apache/openjpa/util/TestWASTransformer.java Thu Oct 26 22:41:03 2006
@@ -37,18 +37,16 @@
      */
     public void testInterfaceAdded()throws ClassNotFoundException {
 
-        boolean caughtExpectedException = false;
-
-        Class syncClass = null;
+        String msg = null;
 
         try {
-             syncClass = Class.forName(WASTransformer._class);
+            Class.forName(WASTransformer._class);
+            fail("expected an exception to be thrown");
         } catch (NoClassDefFoundError e) {
-            if (e.getMessage().contains(WASTransformer._interface)) {
-                caughtExpectedException = true;
-            }
+            msg = e.getMessage();
         }
-        assertNull(syncClass);
-        assertTrue(caughtExpectedException);
+        assertTrue("message should have contained WASTransformer._interface, "
+            + " but was '" + msg + "'", 
+            msg.contains(WASTransformer._interface));
     }
 }