Merge bitcoin/bitcoin#34781: test: Remove fixed TODO in address_to_scriptpubkey

fa0587a306430b35d2852ee32f7eee2adb8c1e64 test: Remove fixed TODO in address_to_scriptpubkey (MarcoFalke)

Pull request description:

  After commit d178082996dc3000f42816f89afcf3fa4d31e159 added the bech32 format, the TODO about adding other formats can be removed.

ACKs for top commit:
  sedited:
    ACK fa0587a306430b35d2852ee32f7eee2adb8c1e64
  rkrux:
    crACK fa0587a306
  musaHaruna:
    ACK [fa0587a](fa0587a306)

Tree-SHA512: 49a8a514bed11e945bd8f9b13b84ae14b4dbc8a7ebb7224b1746776d9dbf68abc3b53d67f1b7fff83bc4360b15324fb96611550f4aca808b16beb03bcbfd0a55
This commit is contained in:
merge-script 2026-03-10 10:38:38 +00:00
commit 5f9068bdcd
No known key found for this signature in database
GPG Key ID: 2EEB9F5CC09526C1

View File

@ -193,9 +193,7 @@ def address_to_scriptpubkey(address):
return keyhash_to_p2pkh_script(payload)
elif version == 196: # testnet script hash
return scripthash_to_p2sh_script(payload)
# TODO: also support other address formats
else:
assert False
raise ValueError(f"Unsupported address type: {address}")
class TestFrameworkScript(unittest.TestCase):