homepage

PyPy includes modified Python 2.7 tests for testing compatibility. Some of changes skips tests for features that are not implemented in PyPy or are CPython specific, some reflects differences between error types or messages in CPython and PyPy, some are needed because PyPy doesn't use reference counting, some are workarounds for known PyPy or CPython bugs. The purpose of this issue is to port harmless non PyPy-specific changes. I'll split full patch on several parts for easier review.
The first patch in a series just adds gc.collect() or test.test_support.gc_collect() calls to force calling destructors or freeing memory.

Left some comments. There are a few test cases that I suspect are relying on side effects of garbage collection to perform some other test, and it would be better to rework the test. In the other cases, I agree that actual garbage collection behaviour is being tested, and it makes sense to call gc_collect() or similar.

Thank you for your comments Martin. I need a time to think over them and provide alternative solutions that less depend on garbage collecting. May be Maciej can answer questions about causes of some PyPy changes.

Just for reference here is a full difference between 2.7 tests in PyPy and CPython. It makes tests fail in CPython and skips some tests or makes them too lenient.

Hi
I can answer precise questions, which tests are you asking about?
Note that if the point is to unify the test suite, would be cool to make changes to both pypy and cpython and not just change cpython one.

Maciej: I put a few questions and comments in the code review. See the <https://bugs.python.org/review/25130/#ps15551> (also linked next to Serhiy’s first patch).

Hi
Looking through your comments, yes, maybe those tests or those things require fixing. We at pypy don't have enough will to fight python-dev most of the time, so the usual approach is to do "minimal hack that works" without trying to dwelve into why this or that works that way. Obviously feel free to fix the underlaying issue, we'll be happy to commit it to pypy

With 2.7 EOL, is this still relevant?

Unfortunately Martin's comments are not available now, after shutting down the Rietveld service.
Here is a new PR which contains extraction of PyPy changes related to calling gc.collect() for 3.8. I removed changes which look specific for PyPy and left changes which look common for every implementation without reference counting.

New changeset eb263f9a356f5c5f21b8d5ce20bac92f31c40cad by Carl Friedrich Bolz-Tereick in branch 'main':
bpo-25130: Make unit-test about restricting the maximum number of nested blocks cpython-only (GH-28002)
https://github.com/python/cpython/commit/eb263f9a356f5c5f21b8d5ce20bac92f31c40cad

New changeset 779b9aef4fb2750f57ce07f0efbc8ce8406e43ab by Miss Islington (bot) in branch '3.10':
bpo-25130: Make unit-test about restricting the maximum number of nested blocks cpython-only (GH-28002) (GH-28016)
https://github.com/python/cpython/commit/779b9aef4fb2750f57ce07f0efbc8ce8406e43ab

New changeset 1046cd06b0e2f20b3be93de83d49b684956af98d by Serhiy Storchaka in branch '3.9':
[3.9] bpo-25130: Make unit-test about restricting the maximum number of nested blocks cpython-only (GH-28002) (GH-28017)
https://github.com/python/cpython/commit/1046cd06b0e2f20b3be93de83d49b684956af98d

New changeset 07d3d54f4e84b1259b800884b202701f69e408d8 by Serhiy Storchaka in branch 'main':
bpo-25130: Make SQLite tests more compatible with PyPy (GH-28021)
https://github.com/python/cpython/commit/07d3d54f4e84b1259b800884b202701f69e408d8

New changeset a49398b643b3a2e6f9cc39729c528eb1e4385b1a by Miss Islington (bot) in branch '3.10':
bpo-25130: Make SQLite tests more compatible with PyPy (GH-28021)
https://github.com/python/cpython/commit/a49398b643b3a2e6f9cc39729c528eb1e4385b1a

New changeset dab74d68e3f8391cb4b404b397ab80e2e0188d1f by Serhiy Storchaka in branch '3.9':
[3.9] bpo-25130: Make SQLite tests more compatible with PyPy (GH-28021) (GH-28023)
https://github.com/python/cpython/commit/dab74d68e3f8391cb4b404b397ab80e2e0188d1f

New changeset 2a8127cafe1d196f858a3ecabf5f1df3eebf9a12 by Serhiy Storchaka in branch 'main':
bpo-25130: Add calls of gc.collect() in tests to support PyPy (GH-28005)
https://github.com/python/cpython/commit/2a8127cafe1d196f858a3ecabf5f1df3eebf9a12

New changeset 330aabbbbee39cbc556be01534b1dbf65d20f04c by Serhiy Storchaka in branch '3.9':
[3.9] bpo-25130: Add calls of gc.collect() in tests to support PyPy (GH-28005). (GH-28028)
https://github.com/python/cpython/commit/330aabbbbee39cbc556be01534b1dbf65d20f04c

New changeset 462c1f0403324efc27c11435da12b8d16f5387de by Serhiy Storchaka in branch '3.10':
[3.10] bpo-25130: Add calls of gc.collect() in tests to support PyPy (GH-28005) (GH-28027)
https://github.com/python/cpython/commit/462c1f0403324efc27c11435da12b8d16f5387de