QPID-6390: [Java System Tests] Ensure that Broker's system properties are loaded before the test begins to use the Model to create test config

git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1660604 13f79535-47bb-0310-9956-ffa450edef68

2 files changed, 8 insertions, 5 deletions

diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/Broker.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/Broker.java
index 0d291fb6ea..6dd6c58853 100644
--- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/Broker.java
+++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/Broker.java
@@ -320,12 +320,12 @@ public class Broker implements BrokerShutdownProvider
         }
     }
 
-    private void populateSystemPropertiesFromDefaults(final String initialProperties) throws IOException
+    public static void populateSystemPropertiesFromDefaults(final String initialProperties) throws IOException
     {
         URL initialPropertiesLocation;
         if(initialProperties == null)
         {
-            initialPropertiesLocation = getClass().getClassLoader().getResource("system.properties");
+            initialPropertiesLocation = Broker.class.getClassLoader().getResource("system.properties");
         }
         else
         {
@@ -335,7 +335,11 @@ public class Broker implements BrokerShutdownProvider
         Properties props = new Properties(QpidProperties.asProperties());
         if(initialPropertiesLocation != null)
         {
-            props.load(initialPropertiesLocation.openStream());
+
+            try(InputStream inStream = initialPropertiesLocation.openStream())
+            {
+                props.load(inStream);
+            }
         }
 
         Set<String> propertyNames = new HashSet<>(props.stringPropertyNames());
@@ -344,7 +348,6 @@ public class Broker implements BrokerShutdownProvider
         {
             System.setProperty(propName, props.getProperty(propName));
         }
-
     }
 
 
diff --git a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/ConfiguredObjectFactoryImpl.java b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/ConfiguredObjectFactoryImpl.java
index 27d914c639..5026df0e19 100644
--- a/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/ConfiguredObjectFactoryImpl.java
+++ b/qpid/java/broker-core/src/main/java/org/apache/qpid/server/model/ConfiguredObjectFactoryImpl.java
@@ -156,7 +156,7 @@ public class ConfiguredObjectFactoryImpl implements ConfiguredObjectFactory
             factory = categoryFactories.get(_defaultTypes.get(category));
             if(factory == null)
             {
-                throw new NoFactoryForTypeException(category, _defaultTypes.get(category));
+                throw new NoFactoryForTypeException(category, type);
             }
         }
         return factory;