From d3cf0366a239fd7ba58247ccf069aa04613e2d8a Mon Sep 17 00:00:00 2001 From: Martin Ritchie Date: Thu, 6 Aug 2009 09:44:20 +0000 Subject: QPID-2002: Removed the setting of QPID_WORK as this appears to upset Derby, local testing was running out of file descriptors. Also removed lookup of environment-path from the configuration file as not all the profiles have the environment-path set, check is purely to ensure there is a value there. git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk/qpid@801577 13f79535-47bb-0310-9956-ffa450edef68 --- .../apache/qpid/server/logging/DerbyMessageStoreLoggingTest.java | 8 +++----- .../apache/qpid/server/logging/MemoryMessageStoreLoggingTest.java | 8 -------- 2 files changed, 3 insertions(+), 13 deletions(-) (limited to 'java') diff --git a/java/systests/src/main/java/org/apache/qpid/server/logging/DerbyMessageStoreLoggingTest.java b/java/systests/src/main/java/org/apache/qpid/server/logging/DerbyMessageStoreLoggingTest.java index 0d365467d2..5ac50c9ad2 100644 --- a/java/systests/src/main/java/org/apache/qpid/server/logging/DerbyMessageStoreLoggingTest.java +++ b/java/systests/src/main/java/org/apache/qpid/server/logging/DerbyMessageStoreLoggingTest.java @@ -71,7 +71,7 @@ public class DerbyMessageStoreLoggingTest extends MemoryMessageStoreLoggingTest * This value will be logged on startup after the MessageStore has been * created. * Input: - * Default configuration + * Default configuration * Output: * * MST-1002 : Store location : @@ -115,14 +115,12 @@ public class DerbyMessageStoreLoggingTest extends MemoryMessageStoreLoggingTest // the index that the virtualhost is within the configuration. // we can retrive that from the vhosts list previously extracted. String fullStoreName = configuration.getString("virtualhosts.virtualhost(" + vhosts.indexOf(vhostName) + ")." + vhostName + ".store.class"); - String storePath = configuration.getString("virtualhosts.virtualhost(" + vhosts.indexOf(vhostName) + ")." + vhostName + ".store.environment-path"); // Get the Simple class name from the expected class name of o.a.q.s.s.MMS String storeName = fullStoreName.substring(fullStoreName.lastIndexOf(".") + 1); - assertTrue("MST-1002 does not contain requried store path'" - + storePath + "' found:" + getMessageString(result), - getMessageString(result).endsWith(storePath)); + assertTrue("MST-1002 does not contain a store path" + getMessageString(result), + getMessageString(result).length() > 0); assertEquals("The store name does not match expected value", storeName, AbstractTestLogSubject.getSlice("ms", fromSubject(result))); diff --git a/java/systests/src/main/java/org/apache/qpid/server/logging/MemoryMessageStoreLoggingTest.java b/java/systests/src/main/java/org/apache/qpid/server/logging/MemoryMessageStoreLoggingTest.java index 280fc7d83b..a1cbeca6de 100644 --- a/java/systests/src/main/java/org/apache/qpid/server/logging/MemoryMessageStoreLoggingTest.java +++ b/java/systests/src/main/java/org/apache/qpid/server/logging/MemoryMessageStoreLoggingTest.java @@ -25,7 +25,6 @@ import org.apache.qpid.server.configuration.ServerConfiguration; import org.apache.qpid.server.logging.subjects.AbstractTestLogSubject; import org.apache.qpid.util.LogMonitor; -import java.io.File; import java.util.List; /** @@ -50,13 +49,6 @@ public class MemoryMessageStoreLoggingTest extends AbstractTestLogging public void setUp() throws Exception { - // set QPID_WORK to be [QPID_WORK|io.tmpdir]/ - setSystemProperty("QPID_WORK", - System.getProperty("QPID_WORK", - System.getProperty("java.io.tmpdir")) - + File.separator + getName()); - - //We explicitly do not call super.setUp as starting up the broker is //part of the test case. // So we have to make the new Log Monitor here -- cgit v1.2.1