test: Remove unused self.p2p_conn_index = 1
What the developer wrote
Authored by MarcoFalke
test: Remove unused self.p2p_conn_index = 1
Was never used
What changed, and why it matters
This commit removes a single unused variable from Bitcoin Core's test framework. It is a cleanup change with no effect on live Bitcoin node software or user funds, and no security relevance.
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 change deletes the line self.p2p_conn_index = 1 from TestNode.__init__ in test/functional/test_framework/test_node.py. The commit message states it was never used. The diff confirms only this assignment is removed, with no functional code changes to the test framework or production code.
Changed components
test/functional/test_framework/test_node.pyInspect captured patch +0 / −1
diff --git a/test/functional/test_framework/test_node.py b/test/functional/test_framework/test_node.py
index 4bcf01e3..f2032bc4 100755
--- a/test/functional/test_framework/test_node.py
+++ b/test/functional/test_framework/test_node.py
@@ -120,7 +120,6 @@ class TestNode():
"""
self.index = i
- self.p2p_conn_index = 1
self.datadir_path = datadir_path
self.bitcoinconf = self.datadir_path / "bitcoin.conf"
self.stdout_dir = self.datadir_path / "stdout"
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.