You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ba...@apache.org on 2015/02/21 18:55:10 UTC

svn commit: r1661417 - /continuum/trunk/continuum-test/src/main/java/org/apache/maven/continuum/AbstractContinuumTest.java

Author: batkinson
Date: Sat Feb 21 17:55:09 2015
New Revision: 1661417

URL: http://svn.apache.org/r1661417
Log:
[CONTINUUM-2736] Continuum's build intermittently fails due to failed test tear down

Modified:
    continuum/trunk/continuum-test/src/main/java/org/apache/maven/continuum/AbstractContinuumTest.java

Modified: continuum/trunk/continuum-test/src/main/java/org/apache/maven/continuum/AbstractContinuumTest.java
URL: http://svn.apache.org/viewvc/continuum/trunk/continuum-test/src/main/java/org/apache/maven/continuum/AbstractContinuumTest.java?rev=1661417&r1=1661416&r2=1661417&view=diff
==============================================================================
--- continuum/trunk/continuum-test/src/main/java/org/apache/maven/continuum/AbstractContinuumTest.java (original)
+++ continuum/trunk/continuum-test/src/main/java/org/apache/maven/continuum/AbstractContinuumTest.java Sat Feb 21 17:55:09 2015
@@ -103,8 +103,16 @@ public abstract class AbstractContinuumT
     protected void tearDown()
         throws Exception
     {
-        daoUtils.eraseDatabase();
-        super.tearDown();
+
+        // Failure to remove database should gracefully tear down *and* fail the test by throwing
+        try
+        {
+            daoUtils.eraseDatabase();
+        }
+        finally
+        {
+            super.tearDown();
+        }
     }
 
     protected void createDefaultProjectGroup()