Fix ClusterFailover.cpp and ForkedBroker.cpp to avoid using the default data directory.

Was causing tests to fail because of a pre-existing lock file in the default data dir.


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

1 files changed, 6 insertions, 0 deletions

diff --git a/cpp/src/tests/ForkedBroker.cpp b/cpp/src/tests/ForkedBroker.cpp
index 0aedb50ec1..9ff301e29d 100644
--- a/cpp/src/tests/ForkedBroker.cpp
+++ b/cpp/src/tests/ForkedBroker.cpp
@@ -42,6 +42,7 @@ namespace tests {
 
 ForkedBroker::ForkedBroker(const Args& constArgs) : running(false), exitStatus(0) {
     Args args(constArgs);
+    // Substitute the special value "TMP_DATA_DIR" with a temporary data dir.
     Args::iterator i = find(args.begin(), args.end(), string("TMP_DATA_DIR"));
     if (i != args.end()) {
         args.erase(i);
@@ -52,6 +53,11 @@ ForkedBroker::ForkedBroker(const Args& constArgs) : running(false), exitStatus(0
         args.push_back("--data-dir");
         args.push_back(dataDir);
     }
+    // Never use the default data directory, set --no-data-dir if no other data-dir arg.
+    Args::iterator j = find(args.begin(), args.end(), string("--data-dir"));    
+    Args::iterator k = find(args.begin(), args.end(), string("--no-data-dir"));
+    if (j == args.end() && k == args.end())
+        args.push_back("--no-data-dir");
     init(args);
 }