vincenzopalazzo
/
rgb-cln
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

pytest: fix autoclean test flake 

```
        l3.rpc.setconfig('autoclean-cycle', 10)
    
        # First it expires.
>       wait_for(lambda: only_one(l3.rpc.listinvoices('inv1')['invoices'])['status'] == 'expired')

```

If we're slow enough, the invoice is cleaned before we see it expire!
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
This commit is contained in:
Rusty Russell 2023-08-21 10:17:20 +09:30
parent 763ec6ad43
commit dec8ad3b8d
1 changed files with 16 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
tests/test_plugin.py
View File

@ -3191,10 +3191,22 @@ def test_autoclean(node_factory):
l3.rpc.setconfig('autoclean-cycle', 10)
# First it expires.
wait_for(lambda: only_one(l3.rpc.listinvoices('inv1')['invoices'])['status'] == 'expired')
# Now will get autocleaned
wait_for(lambda: l3.rpc.listinvoices('inv1')['invoices'] == [])
# It will always go unpaid->expired->deleted, but we might miss it!
was_expired = False
while True:
# Is it deleted yet?
invs = l3.rpc.listinvoices('inv1')['invoices']
if invs == []:
break
if was_expired:
assert only_one(invs)['status'] == 'expired'
else:
if only_one(invs)['status'] == 'expired':
was_expired = True
else:
assert only_one(invs)['status'] == 'unpaid'
time.sleep(1)
assert l3.rpc.autoclean_status()['autoclean']['expiredinvoices']['cleaned'] == 1
# Keeps settings across restarts