[jboss-cvs] JBossAS SVN: r64198 - branches/Branch_4_4/testsuite/src/main/org/jboss/test/jbossmessaging/test.

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

Author: clebert.suconic at jboss.com
Date: 2007-07-23 11:43:16 -0400 (Mon, 23 Jul 2007)
New Revision: 64198

Modified:
   branches/Branch_4_4/testsuite/src/main/org/jboss/test/jbossmessaging/test/SecurityUnitTestCase.java
Log:
Disabling invalid test for JBossMessaging

Modified: branches/Branch_4_4/testsuite/src/main/org/jboss/test/jbossmessaging/test/SecurityUnitTestCase.java
===================================================================
--- branches/Branch_4_4/testsuite/src/main/org/jboss/test/jbossmessaging/test/SecurityUnitTestCase.java	2007-07-23 11:29:12 UTC (rev 64197)
+++ branches/Branch_4_4/testsuite/src/main/org/jboss/test/jbossmessaging/test/SecurityUnitTestCase.java	2007-07-23 15:43:16 UTC (rev 64198)
@@ -224,14 +224,10 @@
    /**
     An number of tests to verrify that clientID works as expected:
     
-    1. Nothing. getClientID should return a string starting withID
-    2. user/pwd with preconfigured clientID, should return preconf
-    3. setClientID, should return the set clientID
-    4. setClienID starting with ID, should trow invalid clientID
-    5. setClientID same as a preconf, should trow invalid clientID
-    6. setClientID after any method beeing invoked on con, throw invalid state
+    This tests a specific a specific behavior from JBossMQ.
+    This tests creates a TopicSession and expects the clientID to be set. There is no requirement for this on JMS Spec.
     */
-   public void testClientIDNormalTest() throws Exception
+   /*public void testClientIDNormalTest() throws Exception
    {
       TopicWorker pub1 = null;
       try
@@ -263,7 +259,7 @@
          {
          }
       }
-   }
+   } */
 
    public void testClientIDPreconfTest() throws Exception
    {
@@ -317,13 +313,6 @@
          Assert.assertEquals("Client did not get a valid clientID", "myId", pub1.connection.getClientID());
          pub1.close();
       }
-      catch (Throwable t)
-      {
-         if (t instanceof junit.framework.AssertionFailedError)
-            throw (junit.framework.AssertionFailedError) t;
-         log.error("Error in test: " + t, t);
-         throw new Exception(t.getMessage());
-      }
       finally
       {
          try
@@ -361,13 +350,6 @@
                e instanceof javax.jms.InvalidClientIDException);
          pub1.close();
       }
-      catch (Throwable t)
-      {
-         if (t instanceof junit.framework.AssertionFailedError)
-            throw (junit.framework.AssertionFailedError) t;
-         log.error("Error in test: " + t, t);
-         throw new Exception(t.getMessage());
-      }
       finally
       {
          try
@@ -406,13 +388,6 @@
                e instanceof javax.jms.IllegalStateException);
          pub1.close();
       }
-      catch (Throwable t)
-      {
-         if (t instanceof junit.framework.AssertionFailedError)
-            throw (junit.framework.AssertionFailedError) t;
-         log.error("Error in test: " + t, t);
-         throw new Exception(t.getMessage());
-      }
       finally
       {
          try