The Wayback Machine - https://web.archive.org/web/20220203064604/https://github.com/python/cpython/pull/28070
Skip to content
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

[3.10] bpo-43913: Fix bugs in cleaning up classes and modules in unittest. (GH-28006) #28070

Merged
merged 1 commit into from Aug 30, 2021

Conversation

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Aug 30, 2021

  • Functions registered with addModuleCleanup() were not called unless
    the user defines tearDownModule() in their test module.
  • Functions registered with addClassCleanup() were not called if
    tearDownClass is set to None.
  • Buffering in TestResult did not work with functions registered
    with addClassCleanup() and addModuleCleanup().
  • Errors in functions registered with addClassCleanup() and
    addModuleCleanup() were not handled correctly in buffered and
    debug modes.
  • Errors in setUpModule() and functions registered with
    addModuleCleanup() were reported in wrong order.
  • And several lesser bugs.
    (cherry picked from commit 08d9e59)

Co-authored-by: Serhiy Storchaka [email protected]

https://bugs.python.org/issue43913

…ythonGH-28006)

* Functions registered with addModuleCleanup() were not called unless
  the user defines tearDownModule() in their test module.
* Functions registered with addClassCleanup() were not called if
  tearDownClass is set to None.
* Buffering in TestResult did not work with functions registered
  with addClassCleanup() and addModuleCleanup().
* Errors in functions registered with addClassCleanup() and
  addModuleCleanup() were not handled correctly in buffered and
  debug modes.
* Errors in setUpModule() and functions registered with
  addModuleCleanup() were reported in wrong order.
* And several lesser bugs.
(cherry picked from commit 08d9e59)

Co-authored-by: Serhiy Storchaka <[email protected]>
@miss-islington
Copy link
Contributor Author

@miss-islington miss-islington commented Aug 30, 2021

@serhiy-storchaka and @ambv: Status check is done, and it's a success .

@miss-islington
Copy link
Contributor Author

@miss-islington miss-islington commented Aug 30, 2021

@serhiy-storchaka and @ambv: Status check is done, and it's a success .

@ambv ambv requested a review from pablogsal Aug 30, 2021
@miss-islington miss-islington merged commit d65fad0 into python:3.10 Aug 30, 2021
13 checks passed
@miss-islington miss-islington deleted the backport-08d9e59-3.10 branch Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants