pytest: listpays: test grouping by calling xpay
What the developer wrote
Authored by Lagrang3
pytest: listpays: test grouping by calling xpay
Changelog-None
Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
What changed, and why it matters
This commit changes a single line in a test file, switching a test from using the older `pay` RPC command to the newer `xpay` RPC command. It is purely a test/quality change and does not modify any production code, fix a bug, or address a security issue.
No security action needed. This is a normal test maintenance commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In tests/test_pay.py, the test_sendpay_grouping functional test is updated to exercise the listpays grouping behavior through xpay instead of pay. The diff is one line: l1.rpc.pay(inv, amount_msat='10000msat') becomes l1.rpc.xpay(inv, amount_msat='10000msat'). No implementation code is touched.
Changed components
tests/test_pay.pyInspect captured patch +1 / −1
diff --git a/tests/test_pay.py b/tests/test_pay.py
index e0b68f09..0f357547 100644
--- a/tests/test_pay.py
+++ b/tests/test_pay.py
@@ -5307,7 +5307,7 @@ def test_sendpay_grouping(node_factory, bitcoind):
wait_for(lambda: only_one(l3.rpc.listpeers()['peers'])['connected'] is True)
scid = l3.rpc.listpeerchannels()['channels'][0]['short_channel_id']
wait_for(lambda: [c['active'] for c in l1.rpc.listchannels(scid)['channels']] == [True, True])
- l1.rpc.pay(inv, amount_msat='10000msat')
+ l1.rpc.xpay(inv, amount_msat='10000msat')
# And finally we should have all 3 attempts to pay the invoice
pays = l1.rpc.listpays()['pays']
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.