Skip to content

[3.10] bpo-45624: make test_graphlib not depend on the iteration order of sets (GH-29233)#29293

Merged
ambv merged 1 commit intopython:3.10from
miss-islington:backport-7401694-3.10
Oct 28, 2021
Merged

[3.10] bpo-45624: make test_graphlib not depend on the iteration order of sets (GH-29233)#29293
ambv merged 1 commit intopython:3.10from
miss-islington:backport-7401694-3.10

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 cfbolz@gmx.de

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 <cfbolz@gmx.de>
@miss-islington
Copy link
Contributor Author

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

@ambv ambv merged commit eccb753 into python:3.10 Oct 28, 2021
@miss-islington miss-islington deleted the backport-7401694-3.10 branch October 28, 2021 21:15
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