Skip to content

[3.8] bpo-38270: Fix indentation of test_hmac assertions (GH-17446) #17450

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 5, 2019

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Dec 3, 2019

Since c64a1a6 two assertions were indented and thus ignored when running test_hmac.

This PR fixes it. As the change is quite trivial I didn't add a NEWS entry.

https://bugs.python.org/issue38270
(cherry picked from commit 8943318)

Co-authored-by: stratakis cstratak@redhat.com

https://bugs.python.org/issue38270

Automerge-Triggered-By: @tiran

Since python@c64a1a6 two assertions were indented and thus ignored when running test_hmac.

This PR fixes it. As the change is quite trivial I didn't add a NEWS entry.

https://bugs.python.org/issue38270
(cherry picked from commit 8943318)

Co-authored-by: stratakis <cstratak@redhat.com>
@miss-islington
Copy link
Contributor Author

@stratakis: Status check is done, and it's a success ✅ .

2 similar comments
@miss-islington
Copy link
Contributor Author

@stratakis: Status check is done, and it's a success ✅ .

@miss-islington
Copy link
Contributor Author

@stratakis: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit cfdaf92 into python:3.8 Dec 5, 2019
@miss-islington miss-islington deleted the backport-8943318-3.8 branch December 5, 2019 16:51
stratakis pushed a commit to stratakis/cpython that referenced this pull request Dec 12, 2019
) (pythonGH-17450)

Since python@c64a1a6 two assertions were indented and thus ignored when running test_hmac.

This PR fixes it. As the change is quite trivial I didn't add a NEWS entry.


https://bugs.python.org/issue38270
(cherry picked from commit 8943318)


Co-authored-by: stratakis <cstratak@redhat.com>


https://bugs.python.org/issue38270



Automerge-Triggered-By: @tiran
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants