[Jboss-cvs] JBossAS SVN: r56194 - trunk/testsuite/src/main/org/jboss/test/security/test

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

Author: anil.saldhana at jboss.com
Date: 2006-08-23 15:12:08 -0400 (Wed, 23 Aug 2006)
New Revision: 56194

Modified:
   trunk/testsuite/src/main/org/jboss/test/security/test/DynamicLoginConfigServiceUnitTestCase.java
Log:
minor changes

Modified: trunk/testsuite/src/main/org/jboss/test/security/test/DynamicLoginConfigServiceUnitTestCase.java
===================================================================
--- trunk/testsuite/src/main/org/jboss/test/security/test/DynamicLoginConfigServiceUnitTestCase.java	2006-08-23 19:11:55 UTC (rev 56193)
+++ trunk/testsuite/src/main/org/jboss/test/security/test/DynamicLoginConfigServiceUnitTestCase.java	2006-08-23 19:12:08 UTC (rev 56194)
@@ -131,35 +131,28 @@
             sutil.stopAService(serviceName);
             assertTrue("Stopped?", sutil.isStopped(serviceName));
          } 
-      }
-      try
-      {
-         server.setAttribute(serviceName,
-               new Attribute("AuthConfig","login-config.xml"));
-      }
-      catch(Throwable t)
-      {
-         fail("Not able to set attribute:"+ t.getLocalizedMessage());
-      }
+      } 
         
       try
       { 
+         server.setAttribute(serviceName,
+               new Attribute("AuthConfig","login-config.xml"));
          sutil.startAService(serviceName);
          assertFalse("Should not Start?", sutil.isStarted(serviceName)); 
       }
       catch(Throwable t)
       {
-         log.debug("Service has rightly disagreed to start",t); 
+         fail("Failute to set an attribute and start a service:"+t.getLocalizedMessage()); 
       }
       finally
       {
          if(sutil.isStarted(serviceName)) 
          {
             sutil.stopAService(serviceName);
-            assertTrue("Stopped?", sutil.isStopped(serviceName));
-            sutil.destroyAService(serviceName);
-            assertTrue("Destroyed?", sutil.isDestroyed(serviceName));
+            assertTrue("Stopped?", sutil.isStopped(serviceName)); 
          }  
+         sutil.destroyAService(serviceName);
+         assertTrue("Destroyed?", sutil.isDestroyed(serviceName));
          if(server.isRegistered(serviceName))
             server.unregisterMBean(serviceName);
       }