Skip to content

[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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 4, 2022

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Dec 4, 2022

  • 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 commit 72ec518)

Co-authored-by: AlexTate 0xalextate@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 commit 72ec518)

Co-authored-by: AlexTate <0xalextate@gmail.com>
@miss-islington
Copy link
Contributor Author

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

@miss-islington miss-islington merged commit e699e5c into python:3.10 Dec 4, 2022
@miss-islington miss-islington deleted the backport-72ec518-3.10 branch December 4, 2022 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy