fix(clippy): fix some formatting issues

Author: oleonardolima

examples/example_wallet_electrum/src/main.rs

@@ -94,7 +94,8 @@ fn main() -> Result<(), anyhow::Error> {
     let (_, internal_keymap) = Descriptor::parse_descriptor(&secp, INTERNAL_DESC)?;
     let key_map = external_keymap.into_iter().chain(internal_keymap).collect();
 
-    // It's using the signer implementation from `test_utils`, you should implement your own signing implementation for `KeyMap`.
+    // It's using the signer implementation from `test_utils`, you should implement your own signing
+    // implementation for `KeyMap`.
     let signer = bdk_wallet::test_utils::SignerWrapper::new(key_map);
     let _ = psbt.sign(&signer, &secp);
 

examples/example_wallet_esplora_async/src/main.rs

@@ -3,7 +3,10 @@ use std::{collections::BTreeSet, io::Write};
 use anyhow::Ok;
 use bdk_esplora::{esplora_client, EsploraAsyncExt};
 use bdk_wallet::{
-    bitcoin::{Amount, Network}, miniscript::Descriptor, rusqlite::Connection, KeychainKind, SignOptions, Wallet
+    bitcoin::{Amount, Network},
+    miniscript::Descriptor,
+    rusqlite::Connection,
+    KeychainKind, SignOptions, Wallet,
 };
 
 const SEND_AMOUNT: Amount = Amount::from_sat(5000);
@@ -85,7 +88,8 @@ async fn main() -> Result<(), anyhow::Error> {
     let (_, internal_keymap) = Descriptor::parse_descriptor(&secp, INTERNAL_DESC)?;
     let key_map = external_keymap.into_iter().chain(internal_keymap).collect();
 
-    // It's using the signer implementation from `test_utils`, you should implement your own signing implementation for `KeyMap`.
+    // It's using the signer implementation from `test_utils`, you should implement your own signing
+    // implementation for `KeyMap`.
     let signer = bdk_wallet::test_utils::SignerWrapper::new(key_map);
     let _ = psbt.sign(&signer, &secp);
 

examples/example_wallet_esplora_blocking/src/main.rs

@@ -2,7 +2,10 @@ use std::{collections::BTreeSet, io::Write};
 
 use bdk_esplora::{esplora_client, EsploraExt};
 use bdk_wallet::{
-    bitcoin::{Amount, Network}, file_store::Store, miniscript::Descriptor, KeychainKind, SignOptions, Wallet
+    bitcoin::{Amount, Network},
+    file_store::Store,
+    miniscript::Descriptor,
+    KeychainKind, SignOptions, Wallet,
 };
 
 const DB_MAGIC: &str = "bdk_wallet_esplora_example";
@@ -85,7 +88,8 @@ fn main() -> Result<(), anyhow::Error> {
     let (_, internal_keymap) = Descriptor::parse_descriptor(&secp, INTERNAL_DESC)?;
     let key_map = external_keymap.into_iter().chain(internal_keymap).collect();
 
-    // It's using the signer implementation from `test_utils`, you should implement your own signing implementation for `KeyMap`.
+    // It's using the signer implementation from `test_utils`, you should implement your own signing
+    // implementation for `KeyMap`.
     let signer = bdk_wallet::test_utils::SignerWrapper::new(key_map);
     let _ = psbt.sign(&signer, &secp);
 

wallet/tests/psbt.rs

@@ -39,7 +39,8 @@ fn test_psbt_malformed_psbt_input_segwit() {
     let _ = psbt.sign(&signer, &secp).unwrap();
 }
 
-// FIXME: (@leonardo) this expect an error from `finalize_psbt` method, should be fixed when removing the `SignerErrors`
+// FIXME: (@leonardo) this expect an error from `finalize_psbt` method, should be fixed when
+// removing the `SignerErrors`
 #[test]
 #[should_panic(expected = "InputIndexOutOfRange")]
 fn test_psbt_malformed_tx_input() {
@@ -61,9 +62,9 @@ fn test_psbt_malformed_tx_input() {
     let _ = wallet.finalize_psbt(&mut psbt, options).unwrap();
 }
 
-// FIXME: (@leonardo) this test needs a refactoring, it fails due to rust-bitcoin's `spend_utxo` method while signing,
-// it adds an input field from BIP-174 PSBT which is missing the `witness_utxo` field. If this input field is not added
-// the signing works successfully.
+// FIXME: (@leonardo) this test needs a refactoring, it fails due to rust-bitcoin's `spend_utxo`
+// method while signing, it adds an input field from BIP-174 PSBT which is missing the
+// `witness_utxo` field. If this input field is not added the signing works successfully.
 // see: https://github.com/rust-bitcoin/rust-bitcoin/blob/ef5e3256dfafd84d40cabb0c09dd3f49ea117c61/bitcoin/src/psbt/mod.rs#L621-L633
 #[test]
 #[ignore = "FIXME: it needs refactoring, in order to properly use a finalized input and not one missing `witness_utxo` field."]
@@ -181,15 +182,14 @@ fn test_psbt_fee_rate_with_missing_txout() {
 }
 
 #[test]
-// #[ignore = "FIXME: it needs refactoring, how should we handle the expected behavior of adding external signers, and usage of wrong internal key ?"]
+// #[ignore = "FIXME: it needs refactoring, how should we handle the expected behavior of adding
+// external signers, and usage of wrong internal key ?"]
 fn test_psbt_multiple_internalkey_signers() {
-    
     use bdk_wallet::KeychainKind;
     use bitcoin::key::TapTweak;
     use bitcoin::secp256k1::{schnorr, Keypair, Message, Secp256k1, XOnlyPublicKey};
     use bitcoin::sighash::{Prevouts, SighashCache, TapSighashType};
     use bitcoin::{PrivateKey, TxOut};
-    
 
     let secp = Secp256k1::new();
     let wif = "cNJmN3fH9DDbDt131fQNkVakkpzawJBSeybCUNmP1BovpmGQ45xG";
@@ -226,7 +226,9 @@ fn test_psbt_multiple_internalkey_signers() {
     let signer = get_wallet_signer(&desc, Some(change_desc));
     let _ = psbt.sign(&signer, &secp).unwrap();
 
-    let finalized = wallet.finalize_psbt(&mut psbt, SignOptions::default()).unwrap();
+    let finalized = wallet
+        .finalize_psbt(&mut psbt, SignOptions::default())
+        .unwrap();
     assert!(finalized);
 
     // To verify, we need the signature, message, and pubkey

wallet/tests/wallet.rs

@@ -3264,7 +3264,8 @@ fn test_signing_only_one_of_multiple_inputs() {
     )
 }
 
-// FIXME: (@leonardo) this test should probably be removed when `SignOptions` is fully removed. The `try_finalize` option is only used in `Wallet::sign` method.
+// FIXME: (@leonardo) this test should probably be removed when `SignOptions` is fully removed. The
+// `try_finalize` option is only used in `Wallet::sign` method.
 #[test]
 fn test_try_finalize_sign_option() {
     let secp = Secp256k1::new();
@@ -3300,7 +3301,8 @@ fn test_try_finalize_sign_option() {
     }
 }
 
-// FIXME: (@leonardo) this test should probably be removed when `SignOptions` is fully removed. The `try_finalize` option is only used in `Wallet::sign` method.
+// FIXME: (@leonardo) this test should probably be removed when `SignOptions` is fully removed. The
+// `try_finalize` option is only used in `Wallet::sign` method.
 #[test]
 fn test_taproot_try_finalize_sign_option() {
     let secp = Secp256k1::new();
@@ -4058,7 +4060,6 @@ fn test_taproot_script_spend_sign_include_some_leaves() {
 
 #[test]
 fn test_taproot_script_spend_sign_exclude_some_leaves() {
-
     use bitcoin::taproot::TapLeafHash;
 
     let secp = Secp256k1::new();