[jboss-cvs] JBossAS SVN: r61847 - in trunk/system-jmx/src/tests/org/jboss/test: system/controller/integration/test and 3 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Thu Mar 29 09:55:46 EDT 2007


Author: adrian at jboss.org
Date: 2007-03-29 09:55:46 -0400 (Thu, 29 Mar 2007)
New Revision: 61847

Modified:
   trunk/system-jmx/src/tests/org/jboss/test/classloading/UCLDomainClassLoaderTest.java
   trunk/system-jmx/src/tests/org/jboss/test/system/controller/integration/test/AbstractIntegrationTest.java
   trunk/system-jmx/src/tests/org/jboss/test/system/controller/integration/test/IntegrationTestDelegate.java
   trunk/system-jmx/src/tests/org/jboss/test/system/controller/legacy/OldServiceConfigurator.java
   trunk/system-jmx/src/tests/org/jboss/test/system/controller/support/ContainedMBeanService.java
   trunk/system-jmx/src/tests/org/jboss/test/system/controller/support/ContainedMBeanServiceMBean.java
   trunk/system-jmx/src/tests/org/jboss/test/system/metadata/test/AbstractMetaDataTest.java
Log:
Source code tidyup

Modified: trunk/system-jmx/src/tests/org/jboss/test/classloading/UCLDomainClassLoaderTest.java
===================================================================
--- trunk/system-jmx/src/tests/org/jboss/test/classloading/UCLDomainClassLoaderTest.java	2007-03-29 13:36:22 UTC (rev 61846)
+++ trunk/system-jmx/src/tests/org/jboss/test/classloading/UCLDomainClassLoaderTest.java	2007-03-29 13:55:46 UTC (rev 61847)
@@ -22,15 +22,14 @@
 package org.jboss.test.classloading;
 
 import java.net.URL;
-import java.net.URLClassLoader;
 import java.security.ProtectionDomain;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Enumeration;
 
-import org.jboss.test.AbstractSystemTest;
 import org.jboss.mx.loading.DomainClassLoaderUCLImpl;
 import org.jboss.mx.loading.UnifiedLoaderRepositoryDCL;
+import org.jboss.test.AbstractSystemTest;
 
 /**
  * Test of the legacy UCL class loader as a DomainClassLoader.

Modified: trunk/system-jmx/src/tests/org/jboss/test/system/controller/integration/test/AbstractIntegrationTest.java
===================================================================
--- trunk/system-jmx/src/tests/org/jboss/test/system/controller/integration/test/AbstractIntegrationTest.java	2007-03-29 13:36:22 UTC (rev 61846)
+++ trunk/system-jmx/src/tests/org/jboss/test/system/controller/integration/test/AbstractIntegrationTest.java	2007-03-29 13:55:46 UTC (rev 61847)
@@ -109,7 +109,7 @@
     *
     * @param url the deployment url
     * @return the deployment
-    * @throws Throwable for any error  
+    * @throws Exception for any error  
     */
    protected KernelDeployment deployMC(URL url) throws Exception
    {
@@ -121,7 +121,7 @@
     *
     * @param resource the deployment resource
     * @return the deployment
-    * @throws Throwable for any error  
+    * @throws Exception for any error  
     */
    protected KernelDeployment deployMC(String resource) throws Exception
    {

Modified: trunk/system-jmx/src/tests/org/jboss/test/system/controller/integration/test/IntegrationTestDelegate.java
===================================================================
--- trunk/system-jmx/src/tests/org/jboss/test/system/controller/integration/test/IntegrationTestDelegate.java	2007-03-29 13:36:22 UTC (rev 61846)
+++ trunk/system-jmx/src/tests/org/jboss/test/system/controller/integration/test/IntegrationTestDelegate.java	2007-03-29 13:55:46 UTC (rev 61847)
@@ -140,6 +140,7 @@
     * Get the kernel bootstrap
     * 
     * @return the bootstrap
+    * @throws Exception for any error
     */
    protected AbstractBootstrap getBootstrap() throws Exception
    {
@@ -179,6 +180,8 @@
 
    /**
     * Validate
+    * 
+    * @throws Exception for any error
     */
    protected void validateMC() throws Exception
    {
@@ -209,7 +212,7 @@
     *
     * @param url the deployment url
     * @return the deployment
-    * @throws Throwable for any error  
+    * @throws Exception for any error  
     */
    protected KernelDeployment deployMC(URL url) throws Exception
    {

Modified: trunk/system-jmx/src/tests/org/jboss/test/system/controller/legacy/OldServiceConfigurator.java
===================================================================
--- trunk/system-jmx/src/tests/org/jboss/test/system/controller/legacy/OldServiceConfigurator.java	2007-03-29 13:36:22 UTC (rev 61846)
+++ trunk/system-jmx/src/tests/org/jboss/test/system/controller/legacy/OldServiceConfigurator.java	2007-03-29 13:55:46 UTC (rev 61847)
@@ -254,7 +254,11 @@
     * The last two can include nested mbean configurations or ObjectNames.
     * SIDE-EFFECT: adds all mbeans this one depends on to the ServiceContext
     * structures.
+    * 
+    * @param objectName the object name
+    * @param loaderName the classloader name
     * @param mbeanElement an <code>Element</code> value
+    * @param mbeans the current list of mbeans
     * @throws Exception if an error occurs
     */
    protected void configure(ObjectName objectName, ObjectName loaderName,

Modified: trunk/system-jmx/src/tests/org/jboss/test/system/controller/support/ContainedMBeanService.java
===================================================================
--- trunk/system-jmx/src/tests/org/jboss/test/system/controller/support/ContainedMBeanService.java	2007-03-29 13:36:22 UTC (rev 61846)
+++ trunk/system-jmx/src/tests/org/jboss/test/system/controller/support/ContainedMBeanService.java	2007-03-29 13:55:46 UTC (rev 61847)
@@ -56,6 +56,7 @@
     * Create and register the contained SimpleMBean
     */
    @Override
+   @SuppressWarnings("unchecked")
    protected void createService() throws Exception
    {
       simpleMBean = new Simple();

Modified: trunk/system-jmx/src/tests/org/jboss/test/system/controller/support/ContainedMBeanServiceMBean.java
===================================================================
--- trunk/system-jmx/src/tests/org/jboss/test/system/controller/support/ContainedMBeanServiceMBean.java	2007-03-29 13:36:22 UTC (rev 61846)
+++ trunk/system-jmx/src/tests/org/jboss/test/system/controller/support/ContainedMBeanServiceMBean.java	2007-03-29 13:55:46 UTC (rev 61847)
@@ -36,8 +36,17 @@
 {
    ObjectName OBJECT_NAME = ObjectNameFactory.create("jboss.test:type=test");
 
-   /** Pass through AString for the container SimpleMBean */
+   /** 
+    * Pass through AString for the container SimpleMBean
+    * 
+    * @return a string
+    */
    String getAString();
-   /** Pass through AString for the container SimpleMBean */
+
+   /** 
+    * Pass through AString for the container SimpleMBean
+    * 
+    * @param string the string
+    */
    void setAString(String string);
 }

Modified: trunk/system-jmx/src/tests/org/jboss/test/system/metadata/test/AbstractMetaDataTest.java
===================================================================
--- trunk/system-jmx/src/tests/org/jboss/test/system/metadata/test/AbstractMetaDataTest.java	2007-03-29 13:36:22 UTC (rev 61846)
+++ trunk/system-jmx/src/tests/org/jboss/test/system/metadata/test/AbstractMetaDataTest.java	2007-03-29 13:55:46 UTC (rev 61847)
@@ -162,6 +162,7 @@
     * 
     * @param name the name
     * @return the url of the xml
+    * @throws Exception for any error
     */
    protected URL findXML(String name) throws Exception
    {




More information about the jboss-cvs-commits mailing list