[jbpm-commits] JBoss JBPM SVN: r2486 - jbpm3/trunk/modules/core/src/main/java/org/jbpm/db.

[do-not-reply at jboss.org]

Author: thomas.diesler at jboss.com
Date: 2008-10-06 10:06:05 -0400 (Mon, 06 Oct 2008)
New Revision: 2486

Modified:
   jbpm3/trunk/modules/core/src/main/java/org/jbpm/db/AbstractDbTestCase.java
Log:
System.exit(1) shows up as success in Hudson.
Fail the test instead and drop the schema.

Modified: jbpm3/trunk/modules/core/src/main/java/org/jbpm/db/AbstractDbTestCase.java
===================================================================
--- jbpm3/trunk/modules/core/src/main/java/org/jbpm/db/AbstractDbTestCase.java	2008-10-06 13:25:04 UTC (rev 2485)
+++ jbpm3/trunk/modules/core/src/main/java/org/jbpm/db/AbstractDbTestCase.java	2008-10-06 14:06:05 UTC (rev 2486)
@@ -71,8 +71,10 @@
     
     if (hasLeftOverRecords())
     {
-      // We exit because subsequent tests start in an undefined state
-      System.exit(1);
+      dropSchema();
+      // Failing here, will probably hide a potential previous failure.
+      // Is there a better way to fail without hiding a potential cause?
+      fail("Test failed to cleanup database records");
     }
     
     super.tearDown();