Cleanup of test suite init.

This commit is contained in:
chriseth 2016-08-06 11:37:52 +02:00
parent 0d894a6832
commit 1bc0320811
4 changed files with 33 additions and 92 deletions

View File

@ -24,28 +24,22 @@
using namespace std; using namespace std;
using namespace dev::test; using namespace dev::test;
Options::Options(int argc, char** argv) Options const& Options::get()
{ {
tArgc = 0; static Options instance;
tArgv = new char*[argc];
for (auto i = 0; i < argc; i++)
{
string arg = argv[i];
if (arg == "--ipc" && i + 1 < argc)
{
ipcPath = argv[i + 1];
i++;
}
else
{
tArgv[i] = argv[i];
tArgc++;
}
}
}
Options const& Options::get(int argc, char** argv)
{
static Options instance(argc, argv);
return instance; return instance;
} }
Options::Options()
{
auto const& suite = boost::unit_test::framework::master_test_suite();
for (auto i = 0; i < suite.argc; i++)
if (string(suite.argv[i]) == "--ipc" && i + 1 < suite.argc)
{
ipcPath = suite.argv[i + 1];
i++;
}
if (ipcPath.empty())
if (auto path = getenv("ETH_TEST_IPC"))
ipcPath = path;
}

View File

@ -103,19 +103,15 @@ namespace test
while (0) while (0)
class Options struct Options: boost::noncopyable
{ {
public:
std::string ipcPath; std::string ipcPath;
int tArgc;
char **tArgv; Options const& get();
/// Get reference to options
/// The first time used, options are parsed with argc, argv
static Options const& get(int argc = 0, char** argv = 0);
private: private:
Options(int argc, char** argv = 0); Options();
Options(Options const&) = delete;
}; };
} }
} }

View File

@ -26,7 +26,6 @@
#pragma GCC diagnostic ignored "-Wunused-parameter" #pragma GCC diagnostic ignored "-Wunused-parameter"
#define BOOST_TEST_NO_MAIN
#if defined(_MSC_VER) #if defined(_MSC_VER)
#pragma warning(push) #pragma warning(push)
#pragma warning(disable:4535) // calling _set_se_translator requires /EHa #pragma warning(disable:4535) // calling _set_se_translator requires /EHa
@ -37,48 +36,3 @@
#endif #endif
#pragma GCC diagnostic pop #pragma GCC diagnostic pop
#include <stdlib.h>
#include <boost/version.hpp>
#include "TestHelper.h"
using namespace boost::unit_test;
std::vector<char*> parameters;
static std::ostringstream strCout;
std::streambuf* oldCoutStreamBuf;
std::streambuf* oldCerrStreamBuf;
//Custom Boost Initialization
test_suite* fake_init_func(int argc, char* argv[])
{
//Required for boost. -nowarning
(void)argc;
(void)argv;
return 0;
}
//Custom Boost Unit Test Main
int main(int argc, char* argv[])
{
//Initialize options before boost reads it
dev::test::Options const& opt = dev::test::Options::get(argc, argv);
return unit_test_main(fake_init_func, opt.tArgc, opt.tArgv);
}
/*
#else
#if defined(_MSC_VER)
#pragma warning(push)
#pragma warning(disable:4535) // calling _set_se_translator requires /EHa
#endif
#include <boost/test/included/unit_test.hpp>
#if defined(_MSC_VER)
#pragma warning(pop)
#endif
#pragma GCC diagnostic pop
#include <test/TestHelper.h>
using namespace boost::unit_test;
#endif*/

View File

@ -38,9 +38,6 @@ namespace // anonymous
string getIPCSocketPath() string getIPCSocketPath()
{ {
string ipcPath = dev::test::Options::get().ipcPath; string ipcPath = dev::test::Options::get().ipcPath;
if (ipcPath.empty())
if (auto path = getenv("ETH_TEST_IPC"))
ipcPath = path;
if (ipcPath.empty()) if (ipcPath.empty())
BOOST_FAIL("ERROR: ipcPath not set! (use --ipc <path> or the environment variable ETH_TEST_IPC)"); BOOST_FAIL("ERROR: ipcPath not set! (use --ipc <path> or the environment variable ETH_TEST_IPC)");