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));
     }
 }


Reply via email to