[jboss-cvs] JBossAS SVN: r60355 - in branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader: test and 1 other directory.

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

Author: scott.stark at jboss.org
Date: 2007-02-06 18:33:05 -0500 (Tue, 06 Feb 2007)
New Revision: 60355

Modified:
   branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/circularity/test/CircularLoadTests.java
   branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/circularity/test/RecursiveCCETests.java
   branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/test/CircularityUnitTestCase.java
Log:
JBAS-4066, Update logging and javadoc

Modified: branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/circularity/test/CircularLoadTests.java
===================================================================
--- branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/circularity/test/CircularLoadTests.java	2007-02-06 23:30:34 UTC (rev 60354)
+++ branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/circularity/test/CircularLoadTests.java	2007-02-06 23:33:05 UTC (rev 60355)
@@ -311,7 +311,8 @@
       t0.join(5000);
       if( t0.loadedClass != null || t0.loadError == null )
       {
-         log.error("T0 failed as no class should have been found");
+         log.error("T0 failed as no class should have been found, "
+               +t0.loadedClass+", "+t0.loadError);
          throw new IllegalStateException("T0 failed as no class should have been found");
       }
       log.debug("Load of Derivied failed as expected", t0.loadError);

Modified: branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/circularity/test/RecursiveCCETests.java
===================================================================
--- branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/circularity/test/RecursiveCCETests.java	2007-02-06 23:30:34 UTC (rev 60354)
+++ branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/circularity/test/RecursiveCCETests.java	2007-02-06 23:33:05 UTC (rev 60355)
@@ -28,7 +28,6 @@
 import org.jboss.mx.loading.UnifiedClassLoader;
 import org.jboss.mx.loading.UnifiedClassLoader3;
 import org.jboss.mx.loading.UnifiedLoaderRepository3;
-import org.jboss.mx.loading.ClassLoadingTask;
 import org.jboss.mx.loading.LoadMgr3;
 import EDU.oswego.cs.dl.util.concurrent.Semaphore;
 

Modified: branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/test/CircularityUnitTestCase.java
===================================================================
--- branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/test/CircularityUnitTestCase.java	2007-02-06 23:30:34 UTC (rev 60354)
+++ branches/Branch_4_2/testsuite/src/main/org/jboss/test/classloader/test/CircularityUnitTestCase.java	2007-02-06 23:33:05 UTC (rev 60355)
@@ -82,7 +82,8 @@
    }
 
    /**
-    * Setup the test suite.
+    * Setup the test suite by deploying the sar with the
+    * {@link org.jboss.test.classloader.circularity.Starter} mbean
     */
    public static Test suite() throws Exception
    {