Test _PyEval_AddPendingCall() instead of Py_AddPendingCall()

Apparently, Py_AddPendingCall() schedules for the main interpreter.

Author: ZeroIntensity

Lib/test/test_capi/test_misc.py

@@ -1645,7 +1645,7 @@ def subthread():
     @threading_helper.requires_working_threading()
     def test_pending_call_creates_thread(self):
         source = """
-        import _testcapi
+        import _testinternalcapi
         import threading
         import time
 
@@ -1662,7 +1662,7 @@ def callback():
 
         def create_pending_call():
             time.sleep(1)
-            _testcapi.simple_pending_call(callback)
+            _testinternalcapi.simple_pending_call(callback)
 
 
         threading.Thread(target=create_pending_call).start()

Modules/_testcapimodule.c

@@ -2572,16 +2572,6 @@ toggle_reftrace_printer(PyObject *ob, PyObject *arg)
     Py_RETURN_NONE;
 }
 
-static PyObject *
-simple_pending_call(PyObject *self, PyObject *callable)
-{
-    if (Py_AddPendingCall(_pending_callback, Py_NewRef(callable)) < 0) {
-        return NULL;
-    }
-
-    Py_RETURN_NONE;
-}
-
 static PyMethodDef TestMethods[] = {
     {"set_errno",               set_errno,                       METH_VARARGS},
     {"test_config",             test_config,                     METH_NOARGS},
@@ -2676,7 +2666,6 @@ static PyMethodDef TestMethods[] = {
     {"test_atexit", test_atexit, METH_NOARGS},
     {"code_offset_to_line", _PyCFunction_CAST(code_offset_to_line), METH_FASTCALL},
     {"toggle_reftrace_printer", toggle_reftrace_printer, METH_O},
-    {"simple_pending_call", simple_pending_call, METH_O},
     {NULL, NULL} /* sentinel */
 };
 

Modules/_testinternalcapi.c

@@ -2376,6 +2376,16 @@ emscripten_set_up_async_input_device(PyObject *self, PyObject *Py_UNUSED(ignored
 }
 #endif
 
+static PyObject *
+simple_pending_call(PyObject *self, PyObject *callable)
+{
+    if (_PyEval_AddPendingCall(_PyInterpreterState_GET(), _pending_callback, Py_NewRef(callable), 0) < 0) {
+        return NULL;
+    }
+
+    Py_RETURN_NONE;
+}
+
 static PyMethodDef module_functions[] = {
     {"get_configs", get_configs, METH_NOARGS},
     {"get_recursion_depth", get_recursion_depth, METH_NOARGS},
@@ -2481,6 +2491,7 @@ static PyMethodDef module_functions[] = {
 #ifdef __EMSCRIPTEN__
     {"emscripten_set_up_async_input_device", emscripten_set_up_async_input_device, METH_NOARGS},
 #endif
+    {"simple_pending_call", simple_pending_call, METH_O},
     {NULL, NULL} /* sentinel */
 };