wallet: add "reserved_to_block" field to listfunds.

We already have this field in reserveinputs and unreserveinputs.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-Added: JSON-RPC: `listfunds` has a new `reserved_to_block` field.

doc/lightning-listfunds.7.md

@@ -34,6 +34,7 @@ Each entry in *outputs* will include:
 -   *redeemscript* (the redeemscript of the output, in hex, only if it's p2sh-wrapped)
 -   *status* (whether *unconfirmed*, *confirmed*, or *spent*)
 -   *reserved* (whether this is UTXO is currently reserved for an in-flight tx)
+-   *reserved_to_block* (when reservation expires, if *reserved* is true)
 
 Each entry in *channels* will include:
 -   *peer\_id* - the peer with which the channel is opened.

tests/test_misc.py

@@ -634,6 +634,7 @@ def dont_spend_outputs(n, txid):
     for out in l1.rpc.listfunds()['outputs']:
         if out['reserved']:
             inputs += [{'txid': out['txid'], 'vout': out['output']}]
+            assert out['reserved_to_block'] > bitcoind.rpc.getblockchaininfo()['blocks']
     l1.rpc.unreserveinputs(bitcoind.rpc.createpsbt(inputs, []))
 
     # Test withdrawal to self.

tests/test_wallet.py

@@ -441,27 +441,34 @@ def test_reserveinputs(node_factory, bitcoind, chainparams):
         l1.rpc.reserveinputs(psbt)
 
     assert all(o['reserved'] for o in l1.rpc.listfunds()['outputs'])
+    reserveheight = bitcoind.rpc.getblockchaininfo()['blocks'] + 72
+    assert all(o['reserved_to_block'] == reserveheight for o in l1.rpc.listfunds()['outputs'])
 
     # Unreserve as a batch.
     psbt = bitcoind.rpc.createpsbt([{'txid': out[0], 'vout': out[1]} for out in outputs], [])
     l1.rpc.unreserveinputs(psbt)
     assert not any(o['reserved'] for o in l1.rpc.listfunds()['outputs'])
+    assert not any('reserved_to_block' in o for o in l1.rpc.listfunds()['outputs'])
 
     # Reserve twice fails unless exclusive.
     l1.rpc.reserveinputs(psbt)
     with pytest.raises(RpcError, match=r"already reserved"):
         l1.rpc.reserveinputs(psbt)
     l1.rpc.reserveinputs(psbt, False)
+    assert all(o['reserved_to_block'] == reserveheight + 72 for o in l1.rpc.listfunds()['outputs'])
     l1.rpc.unreserveinputs(psbt)
     assert all(o['reserved'] for o in l1.rpc.listfunds()['outputs'])
+    assert all(o['reserved_to_block'] == reserveheight for o in l1.rpc.listfunds()['outputs'])
 
     # Stays reserved across restarts.
     l1.restart()
     assert all(o['reserved'] for o in l1.rpc.listfunds()['outputs'])
+    assert all(o['reserved_to_block'] == reserveheight for o in l1.rpc.listfunds()['outputs'])
 
     # Final unreserve works.
     l1.rpc.unreserveinputs(psbt)
     assert not any(o['reserved'] for o in l1.rpc.listfunds()['outputs'])
+    assert not any('reserved_to_block' in o for o in l1.rpc.listfunds()['outputs'])
 
 
 def test_fundpsbt(node_factory, bitcoind, chainparams):

wallet/walletrpc.c

@@ -247,6 +247,7 @@ static void json_add_utxo(struct json_stream *response,
 			  const struct utxo *utxo)
 {
 	const char *out;
+	bool reserved;
 
 	json_object_start(response, fieldname);
 	json_add_txid(response, "txid", &utxo->txid);
@@ -284,9 +285,12 @@ static void json_add_utxo(struct json_stream *response,
 	} else
 		json_add_string(response, "status", "unconfirmed");
 
-	json_add_bool(response, "reserved",
-		      utxo_is_reserved(utxo,
-				       get_block_height(wallet->ld->topology)));
+	reserved = utxo_is_reserved(utxo,
+				    get_block_height(wallet->ld->topology));
+	json_add_bool(response, "reserved", reserved);
+	if (reserved)
+		json_add_num(response, "reserved_to_block",
+			     utxo->reserved_til);
 	json_object_end(response);
 }