Skip to content

Conversation

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Oct 28, 2021

the current test depended on integer sets being iterated on in a certain
fixed order. That order is different on PyPy (insertion based) and could
change in CPython in the future in theory. Make the test robust against
a different iteration order by sorting.
(cherry picked from commit 7401694)

Co-authored-by: Carl Friedrich Bolz-Tereick [email protected]

https://bugs.python.org/issue45624

…ts (pythonGH-29233)

the current test depended on integer sets being iterated on in a certain
fixed order. That order is different on PyPy (insertion based) and could
change in CPython in the future in theory. Make the test robust against
a different iteration order by sorting.
(cherry picked from commit 7401694)

Co-authored-by: Carl Friedrich Bolz-Tereick <[email protected]>
@bedevere-bot bedevere-bot added the tests Tests in the Lib/test dir label Oct 28, 2021
@miss-islington
Copy link
Contributor Author

@cfbolz and @ambv: Status check is done, and it's a failure ❌ .

@miss-islington
Copy link
Contributor Author

@cfbolz and @ambv: Status check is done, and it's a success ✅ .

@ambv ambv merged commit 67a1abb into python:3.9 Oct 28, 2021
@miss-islington miss-islington deleted the backport-7401694-3.9 branch October 28, 2021 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants