We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
test_binascii
$ python -m pip install ruff ... $ ruff --select F811 Lib/test/test_binascii.py Lib/test/test_binascii.py:279:9: F811 Redefinition of unused `test_hex_roundtrip` from line 236 Found 1 error.
Source:
cpython/Lib/test/test_binascii.py
Lines 232 to 239 in f63d378
Lines 278 to 282 in f63d378
The first test_hex_roundtrip is doing a hexlify/unhexlify roundtrip and is correctly named.
test_hex_roundtrip
hexlify
unhexlify
The second test_hex_roundtrip is doing a b2a_uu/a2b_uu roundtrip, so can be renamed like test_b2a_roundtrip.
b2a_uu
a2b_uu
test_b2a_roundtrip
This was introduced after python/core-workflow#505 (comment).
The text was updated successfully, but these errors were encountered:
gh-109140: Rename duplicated tests in test_binascii (#109141)
aa51182
Successfully merging a pull request may close this issue.
Bug report
Source:
cpython/Lib/test/test_binascii.py
Lines 232 to 239 in f63d378
cpython/Lib/test/test_binascii.py
Lines 278 to 282 in f63d378
The first
test_hex_roundtrip
is doing ahexlify
/unhexlify
roundtrip and is correctly named.The second
test_hex_roundtrip
is doing ab2a_uu
/a2b_uu
roundtrip, so can be renamed liketest_b2a_roundtrip
.This was introduced after python/core-workflow#505 (comment).
Linked PRs
test_binascii
#109141The text was updated successfully, but these errors were encountered: