Fixes the accounting of the number of signatures

Author: grarco

crates/sdk/src/signing.rs

@@ -348,6 +348,7 @@ where
         let mut used_pubkeys = HashSet::new();
 
         // First try to sign the raw header with the supplied signatures
+        // FIXME: remove this if, it's useless
         if !signing_tx_data.signatures.is_empty() {
             let signatures = signing_tx_data
                 .signatures
@@ -408,7 +409,11 @@ where
                     Some(FeeAuthorization::Signer {
                         pubkey: fee_payer,
                         ..
-                    }) if pubkey == fee_payer => {}
+                    }) if pubkey == fee_payer => {
+                        // We will sign the inner tx together with the wrapper,
+                        // we can anticipate the accounting of this signature
+                        used_pubkeys.insert(pubkey.clone());
+                    }
                     _ => {
                         if args.dry_run.is_some() {
                             mock_hw_sig(
@@ -486,15 +491,6 @@ where
                         )
                         .await?;
                     }
-                    // //FIXME: I need to add this to hte used pubkeys
-                    // //FIXME: actually what's the point of this?
-                    // if signing_wrapper_data
-                    //     .signing_data
-                    //     .public_keys
-                    //     .contains(pubkey)
-                    // {
-                    //     used_pubkeys.insert(pubkey.clone());
-                    // }
                 }
             }
         }