KEMBAR78
(chore) Refactor deprecated unittest aliases for Python 3.11 compatibility. by tirkarthi · Pull Request #5380 · tensorflow/tensorboard · GitHub
Skip to content

Conversation

@tirkarthi
Copy link
Contributor

@tirkarthi tirkarthi commented Oct 14, 2021

  • Motivation for features / changes

The deprecated aliases were removed in Python 3.11 in python/cpython#28268 . This PR uses Python 3 recommended aliases for Python 3.11 support

See also "Deprecated Aliases" here : https://docs.python.org/3/library/unittest.html#deprecated-aliases

  • Technical description of changes

Following replacements have been made.

assertEquals -> assertEqual
assertRaisesRegexp -> assertRaisesRegex
assertRegexpMatch -> assertRegex

@google-cla google-cla bot added the cla: yes label Oct 14, 2021
@bileschi bileschi self-requested a review October 14, 2021 15:48
Copy link
Collaborator

@bileschi bileschi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@bileschi
Copy link
Collaborator

Thanks for the PR! Also added a link to the docs table of deprecated aliases.

@bileschi
Copy link
Collaborator

Looks like this test failed in CI : //tensorboard/webapp:karma_test_chromium-local Can you try locally while I trigger another CI run? Thanks.

@bileschi
Copy link
Collaborator

Nevermind, looks like it passed this time. Possibly a flake. Thanks for the contribution!

@bileschi bileschi changed the title Refactor deprecated unittest aliases for Python 3.11 compatibility. (chore) Refactor deprecated unittest aliases for Python 3.11 compatibility. Oct 15, 2021
@bileschi bileschi merged commit a734e5a into tensorflow:master Oct 15, 2021
yatbear pushed a commit to yatbear/tensorboard that referenced this pull request Mar 27, 2023
dna2github pushed a commit to dna2fork/tensorboard that referenced this pull request May 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants