[hornetq-commits] JBoss hornetq SVN: r9678 - in branches/Branch_2_1: tests/src/org/hornetq/tests/integration/jms/client and 1 other directory.

[do-not-reply at jboss.org]

Author: clebert.suconic at jboss.com
Date: 2010-09-13 17:30:19 -0400 (Mon, 13 Sep 2010)
New Revision: 9678

Added:
   branches/Branch_2_1/tests/src/org/hornetq/tests/integration/jms/client/ConnectionTest.java
Modified:
   branches/Branch_2_1/src/main/org/hornetq/jms/client/HornetQConnection.java
Log:
HORNETQ-510 - Allowing getClientID calls without changing the new flag

Modified: branches/Branch_2_1/src/main/org/hornetq/jms/client/HornetQConnection.java
===================================================================
--- branches/Branch_2_1/src/main/org/hornetq/jms/client/HornetQConnection.java	2010-09-13 18:08:17 UTC (rev 9677)
+++ branches/Branch_2_1/src/main/org/hornetq/jms/client/HornetQConnection.java	2010-09-13 21:30:19 UTC (rev 9678)
@@ -163,8 +163,6 @@
    {
       checkClosed();
 
-      justCreated = false;
-
       return clientID;
    }
 

Added: branches/Branch_2_1/tests/src/org/hornetq/tests/integration/jms/client/ConnectionTest.java
===================================================================
--- branches/Branch_2_1/tests/src/org/hornetq/tests/integration/jms/client/ConnectionTest.java	                        (rev 0)
+++ branches/Branch_2_1/tests/src/org/hornetq/tests/integration/jms/client/ConnectionTest.java	2010-09-13 21:30:19 UTC (rev 9678)
@@ -0,0 +1,59 @@
+/*
+ * Copyright 2010 Red Hat, Inc.
+ * Red Hat licenses this file to you under the Apache License, version
+ * 2.0 (the "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+ * implied.  See the License for the specific language governing
+ * permissions and limitations under the License.
+ */
+
+package org.hornetq.tests.integration.jms.client;
+
+import javax.jms.Connection;
+
+import org.hornetq.tests.util.JMSTestBase;
+
+/**
+ * A ConnectionTest
+ *
+ * @author <a href="mailto:clebert.suconic at jboss.org">Clebert Suconic</a>
+ *
+ *
+ */
+public class ConnectionTest extends JMSTestBase
+{
+
+   // Constants -----------------------------------------------------
+
+   // Attributes ----------------------------------------------------
+
+   // Static --------------------------------------------------------
+
+   // Constructors --------------------------------------------------
+
+   // Public --------------------------------------------------------
+   
+
+   public void testGetSetConnectionFactory() throws Exception
+   {
+      Connection conn = cf.createConnection();
+      
+      conn.getClientID();
+      
+      conn.setClientID("somethingElse");
+   }
+
+
+   // Package protected ---------------------------------------------
+
+   // Protected -----------------------------------------------------
+
+   // Private -------------------------------------------------------
+
+   // Inner classes -------------------------------------------------
+
+}