Skip to content

[3.9] bpo-41003: Fix test_copyreg when numpy is installed (GH-20935)#20945

Merged
vstinner merged 1 commit into
python:3.9from
vstinner:save_restore_warnings_filters39
Jun 17, 2020
Merged

[3.9] bpo-41003: Fix test_copyreg when numpy is installed (GH-20935)#20945
vstinner merged 1 commit into
python:3.9from
vstinner:save_restore_warnings_filters39

Conversation

@vstinner

@vstinner vstinner commented Jun 17, 2020

Copy link
Copy Markdown
Member

Fix test_copyreg when numpy is installed: test.pickletester now
saves/restores warnings.filters when importing numpy, to ignore
filters installed by numpy.

Add the save_restore_warnings_filters() function to the
test.support.warnings_helper module.

(cherry picked from commit 8362893)

https://bugs.python.org/issue41003

Fix test_copyreg when numpy is installed: test.pickletester now
saves/restores warnings.filters when importing numpy, to ignore
filters installed by numpy.

Add the save_restore_warnings_filters() function to the
test.support.warnings_helper module.

(cherry picked from commit 8362893)
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

@vstinner
vstinner deleted the save_restore_warnings_filters39 branch June 17, 2020 16:40
@miss-islington

Copy link
Copy Markdown
Contributor

Sorry, @vstinner, I could not cleanly backport this to 3.8 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker b39d41ba1b77f7bc51c4d6f6d0e336693192cb3a 3.8

@bedevere-bot

Copy link
Copy Markdown

GH-20946 is a backport of this pull request to the 3.8 branch.

vstinner added a commit that referenced this pull request Jun 17, 2020
) (GH-20946)

Fix test_copyreg when numpy is installed: test.pickletester now
saves/restores warnings.filters when importing numpy, to ignore
filters installed by numpy.

Add the save_restore_warnings_filters() function to the
test.support.warnings_helper module.

(cherry picked from commit 8362893)
(cherry picked from commit b39d41b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants