xpay: fix reported value of amount_sent_msat

Changelog-None

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>

Author: Lagrang3

plugins/xpay/xpay.c

@@ -343,6 +343,13 @@ send_payment_req(struct command *aux_cmd,
 
 /* For self-pay, we don't have hops. */
 static struct amount_msat initial_sent(const struct attempt *attempt)
+{
+	if (tal_count(attempt->hops) == 0)
+		return attempt->delivers;
+	return attempt->hops[0].amount_out;
+}
+
+static struct amount_msat inject_amount(const struct attempt *attempt)
 {
 	if (tal_count(attempt->hops) == 0)
 		return attempt->delivers;
@@ -1143,7 +1150,7 @@ static struct command_result *do_inject(struct command *aux_cmd,
 	json_add_hex_talarr(req->js, "onion", onion);
 	json_add_sha256(req->js, "payment_hash", &attempt->payment->payment_hash);
 	/* If no route, its the same as delivery (self-pay) */
-	json_add_amount_msat(req->js, "amount_msat", initial_sent(attempt));
+	json_add_amount_msat(req->js, "amount_msat", inject_amount(attempt));
 	json_add_u32(req->js, "cltv_expiry", initial_cltv_delta(attempt) + effective_bheight);
 	json_add_u64(req->js, "partid", attempt->partid);
 	json_add_u64(req->js, "groupid", attempt->payment->group_id);

tests/test_xpay.py

@@ -1097,9 +1097,8 @@ def test_bolt12_fees(node_factory):
     assert ret["failed_parts"] == 0
     assert ret["successful_parts"] == 1
     assert ret["amount_msat"] == AMT_MSAT
-    assert ret["amount_sent_msat"] == AMT_MSAT + FEES_MSAT
+    assert ret["amount_sent_msat"] == AMT_MSAT
 
-    # we pay fees to ourselves
     htlcs = l1.rpc.listhtlcs()["htlcs"]
     assert len(htlcs) == 1
     assert htlcs[0]["payment_hash"] == b12_decode["invoice_payment_hash"]