Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[3.10] gh-98458: unittest: bugfix for infinite loop while handling chained exceptions that contain cycles (GH-98459)#99996

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

miss-islington
Copy link
Contributor

@miss-islingtonmiss-islington commentedDec 4, 2022
edited by bedevere-bot
Loading

  • Bugfix addressing infinite loop while handling self-referencing chained exception in TestResult._clean_tracebacks()
  • Bugfix extended to properly handle exception cycles in _clean_tracebacks. The "seen" set follows the approach used in the TracebackException class (thank you@iritkatriel for pointing it out)
  • adds a test for a single chained exception that holds a self-loop in itscause andcontext attributes
    (cherry picked from commit72ec518)

Co-authored-by: AlexTate0xalextate@gmail.com

…ined exceptions that contain cycles (pythonGH-98459)* Bugfix addressing infinite loop while handling self-referencing chained exception in TestResult._clean_tracebacks()* Bugfix extended to properly handle exception cycles in _clean_tracebacks. The "seen" set follows the approach used in the TracebackException class (thank you@iritkatriel for pointing it out)* adds a test for a single chained exception that holds a self-loop in its __cause__ and __context__ attributes(cherry picked from commit72ec518)Co-authored-by: AlexTate <0xalextate@gmail.com>
@miss-islington
Copy link
ContributorAuthor

Sorry, I can't merge this PR. Reason:3 of 6 required status checks are expected..

@miss-islingtonmiss-islington merged commite699e5c intopython:3.10Dec 4, 2022
@miss-islingtonmiss-islington deleted the backport-72ec518-3.10 branchDecember 4, 2022 19:58
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@gpsheadgpsheadgpshead approved these changes

Assignees
No one assigned
Labels
type-bugAn unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@miss-islington@gpshead@bedevere-bot@AlexTate

[8]ページ先頭

©2009-2025 Movatter.jp