Skip to content

gh-84436 Add integration C API tests for immortal objects #103962

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
corona10 merged 14 commits into from
May 2, 2023
Merged

gh-84436 Add integration C API tests for immortal objects #103962

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
644e883
gh-84436 Add intergration C API tests for immortal objects
corona10 Apr 28, 2023
2eea452
Update PCbuild/_testcapi.vcxproj
corona10 Apr 28, 2023
1c70211
Windows format
corona10 Apr 28, 2023
6a83bae
Add immortal.c
corona10 Apr 28, 2023
94ca054
Add test_immortal.py
corona10 Apr 28, 2023
73daa12
fix format
corona10 Apr 28, 2023
e5b8e45
make patchcheck
corona10 Apr 28, 2023
2f4ee5f
fix format
corona10 Apr 28, 2023
55a394b
address code review
corona10 Apr 28, 2023
4fb6719
Adjust small int range
corona10 Apr 28, 2023
e737489
Address code review
corona10 Apr 28, 2023
b03def1
nit
corona10 Apr 28, 2023
e0295b5
Address code review
corona10 May 2, 2023
5538ed4
Refactor test codes and fix test_immortal.py
corona10 May 2, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add immortal.c
  • Loading branch information
@corona10
corona10 committed May 2, 2023
commit 6a83baeccc5a53b2711a71d2b83959fa4cdea2de
105 changes: 105 additions & 0 deletions Modules/_testcapi/immortal.c
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,105 @@
#include "parts.h"

static PyObject*
Comment thread
corona10 marked this conversation as resolved.
Outdated
test_immortal_bool(PyObject *self, PyObject *Py_UNUSED(ignored))
{
PyObject* objects[] = {Py_True, Py_False};
Py_ssize_t n = sizeof(objects) / sizeof(objects[0]);
Comment thread
corona10 marked this conversation as resolved.
Outdated
for(Py_ssize_t i = 0; i < n; i++) {
Comment thread
corona10 marked this conversation as resolved.
Outdated
PyObject* obj = objects[i];
if (!_Py_IsImmortal(obj)) {
PyErr_Format(PyExc_RuntimeError, "%R should be the immportal object.", obj);
return NULL;
}
Py_ssize_t old_count = Py_REFCNT(obj);
for (int i = 0; i < 10000; i++) {
Comment thread
corona10 marked this conversation as resolved.
Outdated
Py_DECREF(obj);
}
Py_ssize_t current_count = Py_REFCNT(obj);
if (old_count != current_count) {
PyErr_SetString(PyExc_RuntimeError, "Reference count should not be changed.");
return NULL;
}
}
Py_RETURN_NONE;
}

static PyObject *
test_immortal_none(PyObject *self, PyObject *Py_UNUSED(ignored))
{
PyObject* none = Py_None;
Comment thread
corona10 marked this conversation as resolved.
Outdated
if (!_Py_IsImmortal(none)) {
PyErr_Format(PyExc_RuntimeError, "%R should be the immportal object.", none);
return NULL;
}
Py_ssize_t old_count = Py_REFCNT(none);
for (int i = 0; i < 10000; i++) {
Py_DECREF(none);
}
Py_ssize_t current_count = Py_REFCNT(none);
if (old_count != current_count) {
PyErr_SetString(PyExc_RuntimeError, "Reference count should not be changed.");
return NULL;
}
Py_RETURN_NONE;
}

static PyObject *
test_immortal_small_ints(PyObject *self, PyObject *Py_UNUSED(ignored))
{
for(int i = -5; i < 255; i++) {
PyObject *small_int = PyLong_FromLong(i);
if (!_Py_IsImmortal(small_int)) {
PyErr_Format(PyExc_RuntimeError, "Small int(%d) object should be the immportal object.", i);
return NULL;
}
Py_ssize_t old_count = Py_REFCNT(small_int);
for (int i = 0; i < 10000; i++) {
Comment thread
corona10 marked this conversation as resolved.
Outdated
Py_DECREF(small_int);
}
Py_ssize_t current_count = Py_REFCNT(small_int);
if (old_count != current_count) {
PyErr_Format(PyExc_RuntimeError, "Reference count of %d should not be changed.", i);
return NULL;
}
}
Py_RETURN_NONE;
}

static PyObject *
test_immortal_ellipsis(PyObject *self, PyObject *Py_UNUSED(ignored))
{
PyObject* ellipsis = Py_Ellipsis;
Comment thread
corona10 marked this conversation as resolved.
Outdated
if (!_Py_IsImmortal(ellipsis)) {
PyErr_SetString(PyExc_RuntimeError, "Ellipsis object should be the immportal object.");
return NULL;
}
Py_ssize_t old_count = Py_REFCNT(ellipsis);
for (int i = 0; i < 10000; i++) {
Py_DECREF(ellipsis);
}
Py_ssize_t current_count = Py_REFCNT(ellipsis);
if (old_count != current_count) {
PyErr_SetString(PyExc_RuntimeError, "Reference count should not be changed.");
return NULL;
}
Py_RETURN_NONE;
}


static PyMethodDef test_methods[] = {
{"test_immortal_bool", test_immortal_bool, METH_NOARGS},
{"test_immortal_none", test_immortal_none, METH_NOARGS},
{"test_immortal_small_ints", test_immortal_small_ints, METH_NOARGS},
{"test_immortal_ellipsis", test_immortal_ellipsis, METH_NOARGS},
{NULL},
};

int
_PyTestCapi_Init_Immortal(PyObject *mod)
{
if (PyModule_AddFunctions(mod, test_methods) < 0) {
return -1;
}
return 0;
}