-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
bpo-45859: fix test_field_descriptor in test_collections for pypy #29691
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.
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
bpo-45859: fix test_field_descriptor in test_collections for pypy #29691
Conversation
that test tries to pickle the descriptors of a namedtuple's fields, which is _collections._itemgetter on CPython. However, on PyPy that class doesn't exist. The code in collections deals fine with that fact, but the above-mentioned test does not make sense in that situation, since you can't pickle properties.
Perhaps just mark this test as CPython only. |
works for me too, done! |
Thanks @cfbolz for the PR, and @rhettinger for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10. |
…only (pythonGH-29691) (cherry picked from commit 4fad314) Co-authored-by: Carl Friedrich Bolz-Tereick <cfbolz@gmx.de>
GH-29708 is a backport of this pull request to the 3.10 branch. |
GH-29709 is a backport of this pull request to the 3.9 branch. |
Thanks @cfbolz for the PR, and @rhettinger for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8. |
Thanks @cfbolz for the PR, and @rhettinger for merging it 🌮🎉.. I'm working now to backport this PR to: 3.9. |
…only (pythonGH-29691) (cherry picked from commit 4fad314) Co-authored-by: Carl Friedrich Bolz-Tereick <cfbolz@gmx.de>
GH-29710 is a backport of this pull request to the 3.8 branch. |
|
|
that test tries to pickle the descriptors of a namedtuple's fields,
which is _collections._itemgetter on CPython. However, on PyPy that
class doesn't exist. The code in collections deals fine with that fact,
but the above-mentioned test does not make sense in that situation,
since you can't pickle properties.
https://bugs.python.org/issue45859