test_renepay.py: remove data dump into /tmp
What the developer wrote
Authored by Matt Whitlock
test_renepay.py: remove data dump into /tmp
Changelog-None
What changed, and why it matters
This commit simply removes leftover debugging code from a test file that was writing temporary data dumps to /tmp. It is a cleanup change with no security relevance.
No action needed; this is a benign test cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch deletes three blocks in tests/test_renepay.py that opened files under /tmp and wrote JSON output from listchannels() and listpeerchannels() RPC calls. These were clearly developer debug dumps accidentally left in the test, not production code. No functional behavior of the test or the lightning node is changed.
Changed components
tests/test_renepay.pyInspect captured patch +0 / −8
diff --git a/tests/test_renepay.py b/tests/test_renepay.py
index b8b5d706..047d4a4d 100644
--- a/tests/test_renepay.py
+++ b/tests/test_renepay.py
@@ -379,19 +379,11 @@ def test_hardmpp(node_factory):
]
)
- with open("/tmp/l1-chans.txt", "w") as f:
- print(json.dumps(l1.rpc.listchannels()), file=f)
-
inv = l4.rpc.invoice("any", "any", "description")
l2.rpc.call("pay", {"bolt11": inv["bolt11"], "amount_msat": 2000000000})
l2.wait_for_htlcs()
assert l4.rpc.listinvoices()["invoices"][0]["amount_received_msat"] == 2000000000
- with open("/tmp/l2-peerchan.txt", "w") as f:
- print(json.dumps(l2.rpc.listpeerchannels()), file=f)
- with open("/tmp/l3-peerchan.txt", "w") as f:
- print(json.dumps(l3.rpc.listpeerchannels()), file=f)
-
inv2 = l6.rpc.invoice("1800000sat", "inv2", "description")
out = subprocess.check_output(
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.