FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.10] bpo-45624: make test_graphlib not depend on the iteration order of sets (GH-29233) by miss-islington · Pull Request #29293 · python/cpython · GitHub

Repository navigation

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

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

ambv merged 1 commit into
python:3.10from
miss-islington:backport-7401694-3.10

Conversation

miss-islington commented Oct 28, 2021 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

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>

Copy link
Copy Markdown
Contributor Author

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

ambv merged commit eccb753 into python:3.10 Oct 28, 2021
miss-islington deleted the backport-7401694-3.10 branch October 28, 2021 21:15
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL