[jboss-cvs] JBossAS SVN: r91335 - trunk/testsuite/src/main/org/jboss/test/cluster/testutil.

[jboss-cvs-commits at lists.jboss.org]

Author: bstansberry at jboss.com
Date: 2009-07-16 01:24:32 -0400 (Thu, 16 Jul 2009)
New Revision: 91335

Modified:
   trunk/testsuite/src/main/org/jboss/test/cluster/testutil/DelegatingClusteredTestCase.java
Log:
Don't let failure of one delegate prevent tear down of another

Modified: trunk/testsuite/src/main/org/jboss/test/cluster/testutil/DelegatingClusteredTestCase.java
===================================================================
--- trunk/testsuite/src/main/org/jboss/test/cluster/testutil/DelegatingClusteredTestCase.java	2009-07-16 05:11:36 UTC (rev 91334)
+++ trunk/testsuite/src/main/org/jboss/test/cluster/testutil/DelegatingClusteredTestCase.java	2009-07-16 05:24:32 UTC (rev 91335)
@@ -10,6 +10,7 @@
 import junit.framework.Test;
 import junit.framework.TestSuite;
 
+import org.jboss.logging.Logger;
 import org.jboss.test.AbstractTestSetup;
 import org.jboss.test.JBossClusteredTestCase;
 import org.jboss.test.JBossTestCase;
@@ -63,6 +64,7 @@
    
    public static class DelegatingTestSetup extends JBossTestClusteredSetup
    {
+      private static final Logger log = Logger.getLogger(DelegatingTestSetup.class);
       private final List<TestSetupDelegate> setupDelegates;
       private String jarNames = null;
       
@@ -111,7 +113,14 @@
                   for (ListIterator<TestSetupDelegate> it = setupDelegates.listIterator(); it.hasPrevious(); )
                   {
                      TestSetupDelegate setupDelegate = it.previous();
-                     setupDelegate.tearDown();
+                     try
+                     {
+                        setupDelegate.tearDown();
+                     }
+                     catch (Exception e)
+                     {
+                       log.error("Caught exception tearing down " + setupDelegate, e);
+                     }
                   }
                }
             }