Skip to content

Commit 7f639df

Browse files
committed
[tests] Remove unused optional verify_checksum parameter
This optional parameter is never used, so remove it.
1 parent 011e784 commit 7f639df

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

test/functional/test_framework/address.py

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,10 @@ def byte_to_base58(b, version):
4545
return result
4646

4747

48-
def base58_to_byte(s, verify_checksum=True):
48+
def base58_to_byte(s):
49+
"""Converts a base58-encoded string to its data and version.
50+
51+
Throws if the base58 checksum is invalid."""
4952
if not s:
5053
return b''
5154
n = 0
@@ -65,8 +68,9 @@ def base58_to_byte(s, verify_checksum=True):
6568
else:
6669
break
6770
res = b'\x00' * pad + res
68-
if verify_checksum:
69-
assert_equal(hash256(res[:-4])[:4], res[-4:])
71+
72+
# Assert if the checksum is invalid
73+
assert_equal(hash256(res[:-4])[:4], res[-4:])
7074

7175
return res[1:-4], int(res[0])
7276

0 commit comments

Comments
 (0)