summaryrefslogtreecommitdiffstats
path: root/lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp
diff options
context:
space:
mode:
authorZachary Turner <zturner@google.com>2015-11-13 01:24:35 +0000
committerZachary Turner <zturner@google.com>2015-11-13 01:24:35 +0000
commitc946d46283b2578055c47de9b7ef7d182cd9a2ac (patch)
tree391bfae00a5feb40e4465f7d42e1868fa80b6639 /lldb/unittests/ScriptInterpreter/Python/PythonDataObjectsTests.cpp
parentb359b1072517cf2445f4bd6d2b7074b51740d2e4 (diff)
downloadbcm5719-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.cpp22
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)
OpenPOWER on IntegriCloud