[hornetq-commits] JBoss hornetq SVN: r8085 - trunk/tests/src/org/hornetq/tests/integration/security.

[do-not-reply at jboss.org]

Author: timfox
Date: 2009-10-13 06:20:05 -0400 (Tue, 13 Oct 2009)
New Revision: 8085

Modified:
   trunk/tests/src/org/hornetq/tests/integration/security/SecurityTest.java
Log:
closed sessions on test

Modified: trunk/tests/src/org/hornetq/tests/integration/security/SecurityTest.java
===================================================================
--- trunk/tests/src/org/hornetq/tests/integration/security/SecurityTest.java	2009-10-13 10:13:26 UTC (rev 8084)
+++ trunk/tests/src/org/hornetq/tests/integration/security/SecurityTest.java	2009-10-13 10:20:05 UTC (rev 8085)
@@ -58,7 +58,7 @@
    private static final String addressA = "addressA";
 
    private static final String queueA = "queueA";
-
+   
    public void testCreateSessionWithNullUserPass() throws Exception
    {
       Configuration configuration = createDefaultConfig(false);
@@ -75,6 +75,8 @@
          try
          {
             ClientSession session = cf.createSession(false, true, true);
+            
+            session.close();
          }
          catch (HornetQException e)
          {
@@ -102,7 +104,7 @@
          try
          {
             ClientSession session = cf.createSession(false, true, true);
-            fail("should not throw exception");
+            fail("should throw exception");
          }
          catch (HornetQException e)
          {
@@ -164,6 +166,8 @@
          try
          {
             ClientSession session = cf.createSession("newuser", "apass", false, true, true, false, -1);
+            
+            session.close();
          }
          catch (HornetQException e)
          {
@@ -890,6 +894,8 @@
          session.close();
 
          senSession.close();
+         
+         sendingSession.close();
       }
       finally
       {
@@ -1041,6 +1047,8 @@
          try
          {
             ClientSession session = cf.createSession(false, true, true);
+            
+            session.close();
          }
          catch (HornetQException e)
          {
@@ -1217,6 +1225,16 @@
 
          //Step 18. Check permissions on news.us.usTopic for same: can't send but can receive
          checkUserReceiveNoSend(usQueueName, samConnection, adminSession);
+         
+         billConnection.close();
+
+         andrewConnection.close();
+
+         frankConnection.close();
+
+         samConnection.close();
+         
+         adminSession.close();
       }
       finally
       {