summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNikita Sobolev <mail@sobolevn.me>2022-10-16 04:18:59 (GMT)
committerGitHub <noreply@github.com>2022-10-16 04:18:59 (GMT)
commit7b48d02933639c91ebd957b2326d8c352d8eddec (patch)
tree596ec5fbaa7430ef99a30109b0f0d34aa908655f
parent660f10248ba321e7783c07f3801991275e2aee1e (diff)
downloadcpython-7b48d02933639c91ebd957b2326d8c352d8eddec.zip
cpython-7b48d02933639c91ebd957b2326d8c352d8eddec.tar.gz
cpython-7b48d02933639c91ebd957b2326d8c352d8eddec.tar.bz2
gh-94808: Cover `PyFunction_GetCode`, `PyFunction_GetGlobals`, `PyFunction_GetModule` (#98158)
-rw-r--r--Lib/test/test_capi.py35
-rw-r--r--Modules/_testcapimodule.c39
2 files changed, 74 insertions, 0 deletions
diff --git a/Lib/test/test_capi.py b/Lib/test/test_capi.py
index ae434c0..a2183cf 100644
--- a/Lib/test/test_capi.py
+++ b/Lib/test/test_capi.py
@@ -895,6 +895,41 @@ class CAPITest(unittest.TestCase):
self.assertEqual(_testcapi.eval_get_func_name(sum), "sum") # c function
self.assertEqual(_testcapi.eval_get_func_name(A), "type")
+ def test_function_get_code(self):
+ import types
+
+ def some():
+ pass
+
+ code = _testcapi.function_get_code(some)
+ self.assertIsInstance(code, types.CodeType)
+ self.assertEqual(code, some.__code__)
+
+ with self.assertRaises(SystemError):
+ _testcapi.function_get_code(None) # not a function
+
+ def test_function_get_globals(self):
+ def some():
+ pass
+
+ globals_ = _testcapi.function_get_globals(some)
+ self.assertIsInstance(globals_, dict)
+ self.assertEqual(globals_, some.__globals__)
+
+ with self.assertRaises(SystemError):
+ _testcapi.function_get_globals(None) # not a function
+
+ def test_function_get_module(self):
+ def some():
+ pass
+
+ module = _testcapi.function_get_module(some)
+ self.assertIsInstance(module, str)
+ self.assertEqual(module, some.__module__)
+
+ with self.assertRaises(SystemError):
+ _testcapi.function_get_module(None) # not a function
+
class TestPendingCalls(unittest.TestCase):
diff --git a/Modules/_testcapimodule.c b/Modules/_testcapimodule.c
index 76e619d..624e878 100644
--- a/Modules/_testcapimodule.c
+++ b/Modules/_testcapimodule.c
@@ -5658,6 +5658,42 @@ test_macros(PyObject *self, PyObject *Py_UNUSED(args))
Py_RETURN_NONE;
}
+static PyObject *
+function_get_code(PyObject *self, PyObject *func)
+{
+ PyObject *code = PyFunction_GetCode(func);
+ if (code != NULL) {
+ Py_INCREF(code);
+ return code;
+ } else {
+ return NULL;
+ }
+}
+
+static PyObject *
+function_get_globals(PyObject *self, PyObject *func)
+{
+ PyObject *globals = PyFunction_GetGlobals(func);
+ if (globals != NULL) {
+ Py_INCREF(globals);
+ return globals;
+ } else {
+ return NULL;
+ }
+}
+
+static PyObject *
+function_get_module(PyObject *self, PyObject *func)
+{
+ PyObject *module = PyFunction_GetModule(func);
+ if (module != NULL) {
+ Py_INCREF(module);
+ return module;
+ } else {
+ return NULL;
+ }
+}
+
static PyObject *test_buildvalue_issue38913(PyObject *, PyObject *);
static PyObject *getargs_s_hash_int(PyObject *, PyObject *, PyObject*);
@@ -5942,6 +5978,9 @@ static PyMethodDef TestMethods[] = {
{"watch_dict", watch_dict, METH_VARARGS, NULL},
{"unwatch_dict", unwatch_dict, METH_VARARGS, NULL},
{"get_dict_watcher_events", get_dict_watcher_events, METH_NOARGS, NULL},
+ {"function_get_code", function_get_code, METH_O, NULL},
+ {"function_get_globals", function_get_globals, METH_O, NULL},
+ {"function_get_module", function_get_module, METH_O, NULL},
{NULL, NULL} /* sentinel */
};