Move GetTempPath() to testutil.

src/Makefile.gtest.include

@@ -9,6 +9,8 @@ noinst_PROGRAMS += zcash-gtest
 # test_checktransaction.cpp MUST be before
 # any test that calls SelectParams().
 zcash_gtest_SOURCES = \
+	test/testutil.cpp \
+	test/testutil.h \
 	gtest/main.cpp \
 	gtest/utils.cpp \
 	gtest/utils.h \

src/Makefile.test.include

@@ -102,6 +102,8 @@ BITCOIN_TESTS =\
   test/test_random.h \
   test/test_util.cpp \
   test/test_util.h \
+  test/testutil.cpp \
+  test/testutil.h \
   test/torcontrol_tests.cpp \
   test/transaction_tests.cpp \
   test/txvalidationcache_tests.cpp \

src/gtest/test_deprecation.cpp

@@ -9,6 +9,8 @@
 #include "util.h"
 #include "utilstrencodings.h"
 
+#include "test/testutil.h"
+
 #include <boost/filesystem/operations.hpp>
 #include <fstream>
 

src/test/alert_tests.cpp

@@ -17,11 +17,11 @@
 #include "rpc/server.h"
 #include "serialize.h"
 #include "streams.h"
-#include "util.h"
 #include "utilstrencodings.h"
 #include "utiltest.h"
 #include "warnings.h"
 
+#include "test/testutil.h"
 #include "test/test_bitcoin.h"
 
 #include <fstream>

src/test/test_bitcoin.cpp

@@ -25,6 +25,8 @@
 #include "rpc/server.h"
 #include "rpc/register.h"
 
+#include "test/testutil.h"
+
 #include <boost/filesystem.hpp>
 #include <boost/test/unit_test.hpp>
 #include <boost/thread.hpp>

src/test/testutil.cpp

@@ -1,3 +1,33 @@
 // Copyright (c) 2009-2016 The Bitcoin Core developers
 // Distributed under the MIT software license, see the accompanying
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
+
+#include "testutil.h"
+
+#ifdef WIN32
+#include <shlobj.h>
+#endif
+
+#include <boost/filesystem.hpp>
+
+boost::filesystem::path GetTempPath() {
+#if BOOST_FILESYSTEM_VERSION == 3
+    return boost::filesystem::temp_directory_path();
+#else
+    // TODO: remove when we don't support filesystem v2 anymore
+    boost::filesystem::path path;
+#ifdef WIN32
+    char pszPath[MAX_PATH] = "";
+
+    if (GetTempPathA(MAX_PATH, pszPath))
+        path = boost::filesystem::path(pszPath);
+#else
+    path = boost::filesystem::path("/tmp");
+#endif
+    if (path.empty() || !boost::filesystem::is_directory(path)) {
+        LogPrintf("GetTempPath(): failed to find temp path\n");
+        return boost::filesystem::path("");
+    }
+    return path;
+#endif
+}

src/test/testutil.h

@@ -8,4 +8,8 @@
 #ifndef BITCOIN_TEST_TESTUTIL_H
 #define BITCOIN_TEST_TESTUTIL_H
 
+#include <boost/filesystem/path.hpp>
+
+boost::filesystem::path GetTempPath();
+
 #endif // BITCOIN_TEST_TESTUTIL_H

src/util.cpp

@@ -605,28 +605,6 @@ boost::filesystem::path GetSpecialFolderPath(int nFolder, bool fCreate)
 }
 #endif
 
-boost::filesystem::path GetTempPath() {
-#if BOOST_FILESYSTEM_VERSION == 3
-    return boost::filesystem::temp_directory_path();
-#else
-    // TODO: remove when we don't support filesystem v2 anymore
-    boost::filesystem::path path;
-#ifdef WIN32
-    char pszPath[MAX_PATH] = "";
-
-    if (GetTempPathA(MAX_PATH, pszPath))
-        path = boost::filesystem::path(pszPath);
-#else
-    path = boost::filesystem::path("/tmp");
-#endif
-    if (path.empty() || !boost::filesystem::is_directory(path)) {
-        LogPrintf("GetTempPath(): failed to find temp path\n");
-        return boost::filesystem::path("");
-    }
-    return path;
-#endif
-}
-
 void runCommand(const std::string& strCommand)
 {
     int nErr = ::system(strCommand.c_str());

src/util.h

@@ -95,7 +95,6 @@ void ReadConfigFile(const std::string& confPath, std::map<std::string, std::stri
 #ifdef WIN32
 boost::filesystem::path GetSpecialFolderPath(int nFolder, bool fCreate = true);
 #endif
-boost::filesystem::path GetTempPath();
 void runCommand(const std::string& strCommand);
 const boost::filesystem::path GetExportDir();