[jboss-cvs] JBossAS SVN: r107686 - branches/JBPAPP_5_1/testsuite/src/main/org/jboss/test/deadlock/test.

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

Author: bstansberry at jboss.com
Date: 2010-08-19 11:35:11 -0400 (Thu, 19 Aug 2010)
New Revision: 107686

Modified:
   branches/JBPAPP_5_1/testsuite/src/main/org/jboss/test/deadlock/test/BeanStressTestCase.java
Log:
[JBPAPP-4758] Work around by disabling useless test that is screwing up the server

Modified: branches/JBPAPP_5_1/testsuite/src/main/org/jboss/test/deadlock/test/BeanStressTestCase.java
===================================================================
--- branches/JBPAPP_5_1/testsuite/src/main/org/jboss/test/deadlock/test/BeanStressTestCase.java	2010-08-19 15:32:48 UTC (rev 107685)
+++ branches/JBPAPP_5_1/testsuite/src/main/org/jboss/test/deadlock/test/BeanStressTestCase.java	2010-08-19 15:35:11 UTC (rev 107686)
@@ -495,11 +495,15 @@
    }
    */
 
-   public void testCleanup() throws Exception
-   {
-      // Restart the db pool
-      super.restartDBPool();
-   }
+    // JBPAPP-4758 -- disabled as this test is screwing up the server, without actually
+    // failing. AFAICT the only point of the test is to see if there is an exception after
+    // restarting DefaultDS, so if it's not detecting a broken server and is leaving the
+    // server broken for other tests, it's useless. 
+//   public void testCleanup() throws Exception
+//   {
+//      // Restart the db pool
+//      super.restartDBPool();
+//   }
 
    public static Test suite() throws Exception
    {