Fix walletcreatefundedpsbt macro

The client macro passed an `Amount` to the RPC which parses to the
wrong format resulting in 0 BTC for that address.

The `inputs` struct has private fields and no constructor.

Add a constructor for `WalletCreateFundedPsbtInput`.

Convert the `Amount` in the `outputs` argument to btc before calling
the RPC.

Author: jamillambert

client/src/client_sync/v17/mod.rs

@@ -241,6 +241,11 @@ pub struct WalletCreateFundedPsbtInput {
     vout: u32,
 }
 
+impl WalletCreateFundedPsbtInput {
+    /// Create a new walletcreatefundedpsbt input entry.
+    pub fn new(txid: Txid, vout: u32) -> Self { Self { txid, vout } }
+}
+
 /// Args for the `addnode` method.
 #[derive(Clone, Copy, Debug, Deserialize, Serialize)]
 #[serde(rename_all = "lowercase")]

client/src/client_sync/v17/wallet.rs

@@ -706,7 +706,16 @@ macro_rules! impl_client_v17__wallet_create_funded_psbt {
                 inputs: Vec<WalletCreateFundedPsbtInput>,
                 outputs: Vec<BTreeMap<Address, Amount>>,
             ) -> Result<WalletCreateFundedPsbt> {
-                self.call("walletcreatefundedpsbt", &[into_json(inputs)?, into_json(outputs)?])
+                // Convert outputs: Vec<BTreeMap<Address, Amount>> to Vec<BTreeMap<String, f64>>
+                let outputs_json: Vec<_> = outputs
+                    .into_iter()
+                    .map(|map| {
+                        map.into_iter()
+                            .map(|(addr, amt)| (addr.to_string(), amt.to_btc()))
+                            .collect::<BTreeMap<_, _>>()
+                    })
+                    .collect();
+                self.call("walletcreatefundedpsbt", &[into_json(inputs)?, into_json(outputs_json)?])
             }
         }
     };