Author: kwall
Date: Fri Jun 15 15:47:58 2012
New Revision: 1350665

URL: http://svn.apache.org/viewvc?rev=1350665&view=rev
Log:
QPID-4062 : [Java Tests] Java system tests sometimes fail due to JMX port 
already initialised (merged from trunk)

Added:
    
qpid/branches/java-config-and-management/qpid/java/common/src/test/java/org/apache/qpid/test/utils/PortHelper.java
      - copied unchanged from r1349531, 
qpid/trunk/qpid/java/common/src/test/java/org/apache/qpid/test/utils/PortHelper.java
Modified:
    qpid/branches/java-config-and-management/   (props changed)
    qpid/branches/java-config-and-management/qpid/   (props changed)
    qpid/branches/java-config-and-management/qpid/cpp/docs/api/   (props 
changed)
    
qpid/branches/java-config-and-management/qpid/cpp/examples/old_api/tradedemo/tradedemo_topic_publisher.vcproj
   (props changed)
    
qpid/branches/java-config-and-management/qpid/cpp/include/qmf/engine/Agent.h   
(props changed)
    
qpid/branches/java-config-and-management/qpid/cpp/include/qmf/engine/Console.h  
 (props changed)
    qpid/branches/java-config-and-management/qpid/cpp/src/   (props changed)
    qpid/branches/java-config-and-management/qpid/cpp/src/CMakeLists.txt   
(props changed)
    qpid/branches/java-config-and-management/qpid/cpp/src/qmf/engine/Agent.cpp  
 (props changed)
    qpid/branches/java-config-and-management/qpid/cpp/src/qpid/acl/   (props 
changed)
    qpid/branches/java-config-and-management/qpid/cpp/src/qpid/broker/   (props 
changed)
    
qpid/branches/java-config-and-management/qpid/cpp/src/qpid/cluster/UpdateClient.cpp
   (props changed)
    
qpid/branches/java-config-and-management/qpid/cpp/src/qpid/cluster/UpdateClient.h
   (props changed)
    
qpid/branches/java-config-and-management/qpid/cpp/src/qpid/management/ManagementAgent.cpp
   (props changed)
    
qpid/branches/java-config-and-management/qpid/cpp/src/qpid/management/ManagementAgent.h
   (props changed)
    qpid/branches/java-config-and-management/qpid/cpp/src/tests/   (props 
changed)
    
qpid/branches/java-config-and-management/qpid/cpp/src/tests/cluster_tests.py   
(props changed)
    qpid/branches/java-config-and-management/qpid/java/   (props changed)
    qpid/branches/java-config-and-management/qpid/java/amqp-1-0-client/   
(props changed)
    qpid/branches/java-config-and-management/qpid/java/amqp-1-0-client-jms/   
(props changed)
    qpid/branches/java-config-and-management/qpid/java/amqp-1-0-common/   
(props changed)
    qpid/branches/java-config-and-management/qpid/java/broker/   (props changed)
    qpid/branches/java-config-and-management/qpid/java/broker/bin/   (props 
changed)
    
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/management/
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageMetaData_1_0.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/ProtocolEngine_1_0_0.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/ProtocolEngine_1_0_0_SASL.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/v1_0/
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/queue/
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/common/src/test/java/org/apache/qpid/test/utils/QpidTestCase.java
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ConfigurationManagement.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/LoggingManagement.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedBroker.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedConnection.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedExchange.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedQueue.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/UserManagement.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanAttribute.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanConstructor.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanDescription.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanOperation.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanOperationParameter.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/ConnectionTypeTabControl.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/ExchangeTypeTabControl.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/MBeanTypeTabControl.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/QueueTypeTabControl.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/resources/macosx/Contents/MacOS/qpidmc
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/server/SupportedProtocolVersionsTest.java
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/InternalBrokerHolder.java
    
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/QpidBrokerTestCase.java
   (contents, props changed)
    
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/SpawnedBrokerHolder.java
    qpid/branches/java-config-and-management/qpid/java/test-profiles/   (props 
changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/CPPExcludes   
(props changed)
    qpid/branches/java-config-and-management/qpid/java/test-profiles/Excludes   
(props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/JavaBDBExcludes
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/JavaExcludes   
(props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/JavaPre010Excludes
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/JavaTransientExcludes
   (props changed)
    qpid/branches/java-config-and-management/qpid/java/test-profiles/XAExcludes 
  (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.async.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.cluster.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.noprefetch.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.ssl.excludes
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.ssl.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-bdb-spawn.0-9-1.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-bdb.0-9-1.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-dby-spawn.0-9-1.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-dby.0-9-1.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-mms-spawn.0-10.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-mms-spawn.0-9-1.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-mms.0-9-1.testprofile
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/log4j-test.xml 
  (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/test-provider.properties
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/test_resources/
   (props changed)
    
qpid/branches/java-config-and-management/qpid/java/test-profiles/testprofile.defaults
   (props changed)
    qpid/branches/java-config-and-management/qpid/packaging/windows/   (props 
changed)
    qpid/branches/java-config-and-management/qpid/python/   (props changed)
    qpid/branches/java-config-and-management/qpid/python/examples/api/spout   
(props changed)
    qpid/branches/java-config-and-management/qpid/python/qpid/concurrency.py   
(props changed)
    
qpid/branches/java-config-and-management/qpid/tests/src/py/qpid_tests/broker_0_9/queue.py
   (props changed)

Propchange: qpid/branches/java-config-and-management/
------------------------------------------------------------------------------
  Merged /qpid/trunk:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/cpp/docs/api/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/docs/api:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/examples/old_api/tradedemo/tradedemo_topic_publisher.vcproj
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/cpp/examples/old_api/tradedemo/tradedemo_topic_publisher.vcproj:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/include/qmf/engine/Agent.h
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/include/qmf/engine/Agent.h:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/include/qmf/engine/Console.h
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/include/qmf/engine/Console.h:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/cpp/src/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/cpp/src/CMakeLists.txt
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src/CMakeLists.txt:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/src/qmf/engine/Agent.cpp
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src/qmf/engine/Agent.cpp:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/cpp/src/qpid/acl/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src/qpid/acl:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/cpp/src/qpid/broker/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src/qpid/broker:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/src/qpid/cluster/UpdateClient.cpp
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src/qpid/cluster/UpdateClient.cpp:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/src/qpid/cluster/UpdateClient.h
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src/qpid/cluster/UpdateClient.h:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/src/qpid/management/ManagementAgent.cpp
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/src/qpid/management/ManagementAgent.h
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/cpp/src/qpid/management/ManagementAgent.h:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/cpp/src/tests/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src/tests:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/cpp/src/tests/cluster_tests.py
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/cpp/src/tests/cluster_tests.py:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/java/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/java/amqp-1-0-client/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/amqp-1-0-client:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/amqp-1-0-client-jms/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/amqp-1-0-client-jms:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/java/amqp-1-0-common/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/amqp-1-0-common:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/java/broker/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/broker:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/java/broker/bin/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/broker/bin:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/management/
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/management:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageMetaData_1_0.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/message/MessageMetaData_1_0.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/ProtocolEngine_1_0_0.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/ProtocolEngine_1_0_0.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/ProtocolEngine_1_0_0_SASL.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/ProtocolEngine_1_0_0_SASL.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/v1_0/
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/protocol/v1_0:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/queue/
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/queue:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost/
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/broker/src/main/java/org/apache/qpid/server/virtualhost:r1349443,1349531

Modified: 
qpid/branches/java-config-and-management/qpid/java/common/src/test/java/org/apache/qpid/test/utils/QpidTestCase.java
URL: 
http://svn.apache.org/viewvc/qpid/branches/java-config-and-management/qpid/java/common/src/test/java/org/apache/qpid/test/utils/QpidTestCase.java?rev=1350665&r1=1350664&r2=1350665&view=diff
==============================================================================
--- 
qpid/branches/java-config-and-management/qpid/java/common/src/test/java/org/apache/qpid/test/utils/QpidTestCase.java
 (original)
+++ 
qpid/branches/java-config-and-management/qpid/java/common/src/test/java/org/apache/qpid/test/utils/QpidTestCase.java
 Fri Jun 15 15:47:58 2012
@@ -29,8 +29,6 @@ import java.io.BufferedReader;
 import java.io.File;
 import java.io.FileReader;
 import java.io.IOException;
-import java.net.DatagramSocket;
-import java.net.ServerSocket;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
@@ -166,9 +164,10 @@ public class QpidTestCase extends TestCa
             throw new IllegalArgumentException("Invalid start port: " + 
fromPort);
         }
 
+        PortHelper portHelper = new PortHelper();
         for (int i = fromPort; i <= MAX_PORT_NUMBER; i++)
         {
-            if (available(i)) {
+            if (portHelper.isPortAvailable(i)) {
                 return i;
             }
         }
@@ -176,54 +175,6 @@ public class QpidTestCase extends TestCa
         throw new NoSuchElementException("Could not find an available port 
above " + fromPort);
     }
 
-    /**
-     * Checks to see if a specific port is available.
-     *
-     * @param port the port to check for availability
-     */
-    private boolean available(int port)
-    {
-        if ((port < MIN_PORT_NUMBER) || (port > MAX_PORT_NUMBER))
-        {
-            throw new IllegalArgumentException("Invalid start port: " + port);
-        }
-
-        ServerSocket ss = null;
-        DatagramSocket ds = null;
-        try
-        {
-            ss = new ServerSocket(port);
-            ss.setReuseAddress(true);
-            ds = new DatagramSocket(port);
-            ds.setReuseAddress(true);
-            return true;
-        }
-        catch (IOException e)
-        {
-        }
-        finally
-        {
-            if (ds != null)
-            {
-                ds.close();
-            }
-
-            if (ss != null)
-            {
-                try
-                {
-                    ss.close();
-                }
-                catch (IOException e)
-                {
-                    /* should not be thrown */
-                }
-            }
-        }
-
-        return false;
-    }
-
     public int findFreePort()
     {
         return getNextAvailable(10000);

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ConfigurationManagement.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ConfigurationManagement.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/LoggingManagement.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/LoggingManagement.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedBroker.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedBroker.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedConnection.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedConnection.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedExchange.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedExchange.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedQueue.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/ManagedQueue.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/UserManagement.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/UserManagement.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanAttribute.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanAttribute.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanConstructor.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanConstructor.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanDescription.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanDescription.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanOperation.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanOperation.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanOperationParameter.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/common/src/main/java/org/apache/qpid/management/common/mbeans/annotations/MBeanOperationParameter.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/management/eclipse-plugin/src:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/ConnectionTypeTabControl.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/ConnectionTypeTabControl.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/ExchangeTypeTabControl.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/ExchangeTypeTabControl.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/MBeanTypeTabControl.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/MBeanTypeTabControl.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/QueueTypeTabControl.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/eclipse-plugin/src/main/java/org/apache/qpid/management/ui/views/type/QueueTypeTabControl.java:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/management/eclipse-plugin/src/main/resources/macosx/Contents/MacOS/qpidmc
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/management/eclipse-plugin/src/main/resources/macosx/Contents/MacOS/qpidmc:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/server/SupportedProtocolVersionsTest.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/server/SupportedProtocolVersionsTest.java:r1349443,1349531

Modified: 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/InternalBrokerHolder.java
URL: 
http://svn.apache.org/viewvc/qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/InternalBrokerHolder.java?rev=1350665&r1=1350664&r2=1350665&view=diff
==============================================================================
--- 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/InternalBrokerHolder.java
 (original)
+++ 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/InternalBrokerHolder.java
 Fri Jun 15 15:47:58 2012
@@ -20,6 +20,8 @@
  */
 package org.apache.qpid.test.utils;
 
+import java.util.Set;
+
 import org.apache.log4j.Logger;
 
 import org.apache.qpid.server.Broker;
@@ -31,7 +33,9 @@ public class InternalBrokerHolder implem
     private final Broker _broker;
     private final String _workingDirectory;
 
-    public InternalBrokerHolder(final Broker broker, String workingDirectory)
+    private Set<Integer> _portsUsedByBroker;
+
+    public InternalBrokerHolder(final Broker broker, String workingDirectory, 
Set<Integer> portsUsedByBroker)
     {
         if(broker == null)
         {
@@ -40,6 +44,7 @@ public class InternalBrokerHolder implem
 
         _broker = broker;
         _workingDirectory = workingDirectory;
+        _portsUsedByBroker = portsUsedByBroker;
     }
 
     @Override
@@ -53,7 +58,9 @@ public class InternalBrokerHolder implem
         LOGGER.info("Shutting down Broker instance");
 
         _broker.shutdown();
-        
+
+        waitUntilPortsAreFree();
+
         LOGGER.info("Broker instance shutdown");
     }
 
@@ -62,7 +69,12 @@ public class InternalBrokerHolder implem
     {
         // Can't kill a internal broker as we would also kill ourselves as we 
share the same JVM.
         shutdown();
-    }
 
+        waitUntilPortsAreFree();
+    }
 
+    private void waitUntilPortsAreFree()
+    {
+        new PortHelper().waitUntilPortsAreFree(_portsUsedByBroker);
+    }
 }

Modified: 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/QpidBrokerTestCase.java
URL: 
http://svn.apache.org/viewvc/qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/QpidBrokerTestCase.java?rev=1350665&r1=1350664&r2=1350665&view=diff
==============================================================================
--- 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/QpidBrokerTestCase.java
 (original)
+++ 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/QpidBrokerTestCase.java
 Fri Jun 15 15:47:58 2012
@@ -30,6 +30,7 @@ import java.util.List;
 import java.util.Map;
 import java.util.Set;
 import java.util.concurrent.TimeUnit;
+
 import javax.jms.BytesMessage;
 import javax.jms.Connection;
 import javax.jms.Destination;
@@ -46,6 +47,7 @@ import javax.jms.TextMessage;
 import javax.jms.Topic;
 import javax.naming.InitialContext;
 import javax.naming.NamingException;
+
 import org.apache.commons.configuration.ConfigurationException;
 import org.apache.commons.configuration.XMLConfiguration;
 import org.apache.commons.lang.StringUtils;
@@ -167,13 +169,13 @@ public class QpidBrokerTestCase extends 
     protected List<Connection> _connections = new ArrayList<Connection>();
     public static final String QUEUE = "queue";
     public static final String TOPIC = "topic";
-    
+
     /** Map to hold test defined environment properties */
     private Map<String, String> _env;
 
     /** Ensure our messages have some sort of size */
     protected static final int DEFAULT_MESSAGE_SIZE = 1024;
-    
+
     /** Size to create our message*/
     private int _messageSize = DEFAULT_MESSAGE_SIZE;
     /** Type of message*/
@@ -312,6 +314,24 @@ public class QpidBrokerTestCase extends 
     }
 
     /**
+     * The returned set of port numbers is only a guess because it assumes no 
ports have been overridden
+     * using system properties.
+     */
+    protected Set<Integer> guessAllPortsUsedByBroker(int mainPort)
+    {
+        Set<Integer> ports = new HashSet<Integer>();
+        int managementPort = getManagementPort(mainPort);
+        int connectorServerPort = managementPort + 
ServerConfiguration.JMXPORT_CONNECTORSERVER_OFFSET;
+
+        ports.add(mainPort);
+        ports.add(managementPort);
+        ports.add(connectorServerPort);
+        ports.add(DEFAULT_SSL_PORT);
+
+        return ports;
+    }
+
+    /**
      * Get the Port that is use by the current broker
      *
      * @return the current port
@@ -371,6 +391,8 @@ public class QpidBrokerTestCase extends 
             throw new IllegalStateException("There is already an existing 
broker running on port " + port);
         }
 
+        Set<Integer> portsUsedByBroker = guessAllPortsUsedByBroker(port);
+
         if (_brokerType.equals(BrokerType.INTERNAL) && 
!existingInternalBroker())
         {
             
setConfigurationProperty(ServerConfiguration.MGMT_CUSTOM_REGISTRY_SOCKET, 
String.valueOf(false));
@@ -397,7 +419,7 @@ public class QpidBrokerTestCase extends 
             _logger.info("starting internal broker (same JVM)");
             broker.startup(options);
 
-            _brokers.put(port, new InternalBrokerHolder(broker, 
System.getProperty("QPID_WORK")));
+            _brokers.put(port, new InternalBrokerHolder(broker, 
System.getProperty("QPID_WORK"), portsUsedByBroker));
         }
         else if (!_brokerType.equals(BrokerType.EXTERNAL))
         {
@@ -504,14 +526,14 @@ public class QpidBrokerTestCase extends 
                 // this is expect if the broker started successfully
             }
 
-            _brokers.put(port, new SpawnedBrokerHolder(process, qpidWork));
+            _brokers.put(port, new SpawnedBrokerHolder(process, qpidWork, 
portsUsedByBroker));
         }
     }
 
     private void addExcludedPorts(int port, int sslPort, BrokerOptions options)
     {
         final String protocolExcludesList = getProtocolExcludesList(port, 
sslPort);
-        
+
         if (protocolExcludesList.equals(""))
         {
             return;
@@ -1030,7 +1052,7 @@ public class QpidBrokerTestCase extends 
     {
         return (AMQConnectionFactory) getInitialContext().lookup(factoryName);
     }
-    
+
     public Connection getConnection() throws JMSException, NamingException
     {
         return getConnection(GUEST_USERNAME, GUEST_PASSWORD);
@@ -1324,14 +1346,14 @@ public class QpidBrokerTestCase extends 
 
     /**
      * Reloads the broker security configuration using the ApplicationRegistry 
(InVM brokers) or the
-     * ConfigurationManagementMBean via the JMX interface (Standalone brokers, 
management must be 
+     * ConfigurationManagementMBean via the JMX interface (Standalone brokers, 
management must be
      * enabled before calling the method).
      */
     public void reloadBrokerSecurityConfig() throws Exception
     {
         JMXTestUtils jmxu = new JMXTestUtils(this);
         jmxu.open();
-        
+
         try
         {
             ConfigurationManagement configMBean = 
jmxu.getConfigurationManagement();
@@ -1341,7 +1363,7 @@ public class QpidBrokerTestCase extends 
         {
             jmxu.close();
         }
-        
+
         LogMonitor _monitor = new LogMonitor(_outputFile);
         assertTrue("The expected server security configuration reload did not 
occur",
                 
_monitor.waitForMessage(ServerConfiguration.SECURITY_CONFIG_RELOADED, 
LOGMONITOR_TIMEOUT));

Propchange: 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/QpidBrokerTestCase.java
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/QpidBrokerTestCase.java:r1349443,1349531

Modified: 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/SpawnedBrokerHolder.java
URL: 
http://svn.apache.org/viewvc/qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/SpawnedBrokerHolder.java?rev=1350665&r1=1350664&r2=1350665&view=diff
==============================================================================
--- 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/SpawnedBrokerHolder.java
 (original)
+++ 
qpid/branches/java-config-and-management/qpid/java/systests/src/main/java/org/apache/qpid/test/utils/SpawnedBrokerHolder.java
 Fri Jun 15 15:47:58 2012
@@ -21,6 +21,7 @@
 package org.apache.qpid.test.utils;
 
 import java.io.IOException;
+import java.util.Set;
 
 import org.apache.log4j.Logger;
 
@@ -32,8 +33,9 @@ public class SpawnedBrokerHolder impleme
     private final Process _process;
     private final Integer _pid;
     private final String _workingDirectory;
+    private Set<Integer> _portsUsedByBroker;
 
-    public SpawnedBrokerHolder(final Process process, final String 
workingDirectory)
+    public SpawnedBrokerHolder(final Process process, final String 
workingDirectory, Set<Integer> portsUsedByBroker)
     {
         if(process == null)
         {
@@ -43,6 +45,7 @@ public class SpawnedBrokerHolder impleme
         _process = process;
         _pid = retrieveUnixPidIfPossible();
         _workingDirectory = workingDirectory;
+        _portsUsedByBroker = portsUsedByBroker;
     }
 
     @Override
@@ -57,6 +60,8 @@ public class SpawnedBrokerHolder impleme
         _process.destroy();
 
         reapChildProcess();
+
+        waitUntilPortsAreFree();
     }
 
     @Override
@@ -74,6 +79,8 @@ public class SpawnedBrokerHolder impleme
         }
 
         reapChildProcess();
+
+        waitUntilPortsAreFree();
     }
 
     private void sendSigkillForImmediateShutdown(Integer pid)
@@ -146,4 +153,9 @@ public class SpawnedBrokerHolder impleme
         }
     }
 
+    private void waitUntilPortsAreFree()
+    {
+        new PortHelper().waitUntilPortsAreFree(_portsUsedByBroker);
+    }
+
 }

Propchange: qpid/branches/java-config-and-management/qpid/java/test-profiles/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/CPPExcludes
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/CPPExcludes:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/Excludes
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/Excludes:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/JavaBDBExcludes
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/JavaBDBExcludes:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/JavaExcludes
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/JavaExcludes:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/JavaPre010Excludes
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/JavaPre010Excludes:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/JavaTransientExcludes
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/JavaTransientExcludes:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/XAExcludes
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/XAExcludes:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.async.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/cpp.async.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.cluster.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/cpp.cluster.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.noprefetch.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/cpp.noprefetch.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.ssl.excludes
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/cpp.ssl.excludes:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.ssl.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/cpp.ssl.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/cpp.testprofile
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/cpp.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-bdb-spawn.0-9-1.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/java-bdb-spawn.0-9-1.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-bdb.0-9-1.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/java-bdb.0-9-1.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-dby-spawn.0-9-1.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/java-dby-spawn.0-9-1.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-dby.0-9-1.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/java-dby.0-9-1.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-mms-spawn.0-10.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/java-mms-spawn.0-10.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-mms-spawn.0-9-1.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/java-mms-spawn.0-9-1.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/java-mms.0-9-1.testprofile
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/java-mms.0-9-1.testprofile:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/log4j-test.xml
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/log4j-test.xml:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/test-provider.properties
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/test-provider.properties:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/test_resources/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/java/test-profiles/test_resources:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/java/test-profiles/testprofile.defaults
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/java/test-profiles/testprofile.defaults:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/packaging/windows/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/packaging/windows:r1349443,1349531

Propchange: qpid/branches/java-config-and-management/qpid/python/
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/python:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/python/examples/api/spout
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/python/examples/api/spout:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/python/qpid/concurrency.py
------------------------------------------------------------------------------
  Merged /qpid/trunk/qpid/python/qpid/concurrency.py:r1349443,1349531

Propchange: 
qpid/branches/java-config-and-management/qpid/tests/src/py/qpid_tests/broker_0_9/queue.py
------------------------------------------------------------------------------
  Merged 
/qpid/trunk/qpid/tests/src/py/qpid_tests/broker_0_9/queue.py:r1349443,1349531



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to