What the developer wrote
Authored by SomberNight
tests: rm some more dead code
What changed, and why it matters
This commit simply removes an unused helper function from a test file. It does not change any production code, user-facing behavior, or security-sensitive logic. There is no security issue here.
No action needed. This is a routine test-code cleanup commit with no security relevance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit deletes the keypair() helper function from tests/test_lnpeer.py. The function generated a random private key and returned a Keypair object, but it is no longer referenced anywhere in the test suite. The change is purely cleanup of dead test code.
Changed components
tests/test_lnpeer.pyInspect captured patch +0 / −8
diff --git a/tests/test_lnpeer.py b/tests/test_lnpeer.py
index 3192398..afb6f0a 100644
--- a/tests/test_lnpeer.py
+++ b/tests/test_lnpeer.py
@@ -58,14 +58,6 @@ from .test_bitcoin import needs_test_with_all_chacha20_implementations
from . import ElectrumTestCase, restore_wallet_from_text__for_unittest
-def keypair():
- priv = ECPrivkey.generate_random_key().get_secret_bytes()
- k1 = Keypair(
- pubkey=privkey_to_pubkey(priv),
- privkey=priv)
- return k1
-
-
class MockNetwork:
def __init__(self, *, config: SimpleConfig):
self.lnwatcher = None
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.