diff options
author | Zachary Turner <zturner@google.com> | 2015-11-13 01:24:35 +0000 |
---|---|---|
committer | Zachary Turner <zturner@google.com> | 2015-11-13 01:24:35 +0000 |
commit | c946d46283b2578055c47de9b7ef7d182cd9a2ac (patch) | |
tree | 391bfae00a5feb40e4465f7d42e1868fa80b6639 /lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp | |
parent | b359b1072517cf2445f4bd6d2b7074b51740d2e4 (diff) | |
download | bcm5719-llvm-c946d46283b2578055c47de9b7ef7d182cd9a2ac.tar.gz bcm5719-llvm-c946d46283b2578055c47de9b7ef7d182cd9a2ac.zip |
gtest - Make a `PythonTestSuite` base class for setup / teardown.
This allows other potential unit test suites (of which one is
forthcoming in a subsequent patch) to re-use the same initialization
and teardown of the GIL.
llvm-svn: 252993
Diffstat (limited to 'lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp')
-rw-r--r-- | lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp | 22 |
1 files changed, 5 insertions, 17 deletions
diff --git a/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp b/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp index 86e6125c869..5c6925179c6 100644 --- a/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp +++ b/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp @@ -16,25 +16,17 @@ #include "Plugins/ScriptInterpreter/Python/PythonDataObjects.h" #include "Plugins/ScriptInterpreter/Python/ScriptInterpreterPython.h" +#include "PythonTestSuite.h" + using namespace lldb_private; -class PythonDataObjectsTest : public testing::Test +class PythonDataObjectsTest : public PythonTestSuite { public: void SetUp() override { - HostInfoBase::Initialize(); - // ScriptInterpreterPython::Initialize() depends on HostInfo being - // initializedso it can compute the python directory etc. - ScriptInterpreterPython::Initialize(); - - // Although we don't care about concurrency for the purposes of running - // this test suite, Python requires the GIL to be locked even for - // deallocating memory, which can happen when you call Py_DECREF or - // Py_INCREF. So acquire the GIL for the entire duration of this - // test suite. - m_gil_state = PyGILState_Ensure(); + PythonTestSuite::SetUp(); PythonString sys_module("sys"); m_sys_module.Reset(PyRefType::Owned, PyImport_Import(sys_module.get())); @@ -48,18 +40,14 @@ class PythonDataObjectsTest : public testing::Test m_sys_module.Reset(); m_main_module.Reset(); m_builtins_module.Reset(); - PyGILState_Release(m_gil_state); - ScriptInterpreterPython::Terminate(); + PythonTestSuite::TearDown(); } protected: PythonModule m_sys_module; PythonModule m_main_module; PythonModule m_builtins_module; - - private: - PyGILState_STATE m_gil_state; }; TEST_F(PythonDataObjectsTest, TestOwnedReferences) |