What the developer wrote
Authored by MarcoFalke
test: Remove unused AddressType
unused after commit c847dee1488a294c9a9632a00ba1134b21e41947
What changed, and why it matters
This commit simply removes an unused helper class from Bitcoin Core's test framework. It is a routine code cleanup with no effect on the actual Bitcoin network software or its security.
No action needed. This is a benign test-only cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit deletes the AddressType enum from test/functional/test_framework/address.py, noting it became unused after a prior commit. The change is purely in test infrastructure, removing dead code. No production logic, consensus code, networking, or cryptography is touched.
Changed components
test/functional/test_framework/address.pyInspect captured patch +0 / −8
diff --git a/test/functional/test_framework/address.py b/test/functional/test_framework/address.py
index 0f7de20b..f07019d2 100644
--- a/test/functional/test_framework/address.py
+++ b/test/functional/test_framework/address.py
@@ -8,7 +8,6 @@
- bech32 segwit v0 P2WPKH and P2WSH addresses.
- bech32m segwit v1 P2TR addresses."""
-import enum
import unittest
from .script import (
@@ -37,13 +36,6 @@ ADDRESS_BCRT1_UNSPENDABLE_DESCRIPTOR = 'addr(bcrt1qqqqqqqqqqqqqqqqqqqqqqqqqqqqqq
# Coins sent to this address can be spent with a witness stack of just OP_TRUE
ADDRESS_BCRT1_P2WSH_OP_TRUE = 'bcrt1qft5p2uhsdcdc3l2ua4ap5qqfg4pjaqlp250x7us7a8qqhrxrxfsqseac85'
-
-class AddressType(enum.Enum):
- bech32 = 'bech32'
- p2sh_segwit = 'p2sh-segwit'
- legacy = 'legacy' # P2PKH
-
-
b58chars = '123456789ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnopqrstuvwxyz'
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.