Clippy cleanup

Author: jclapis

Cargo.lock

@@ -1,7 +1,7 @@
 use std::{ops::Deref, str::FromStr};
 
 use alloy::{
-    primitives::{Address, B256, Signature, aliases::B32},
+    primitives::{Address, B256, Signature},
     signers::{SignerSync, local::PrivateKeySigner},
 };
 use eyre::ensure;

crates/common/src/signer/schemes/ecdsa.rs

@@ -709,7 +709,7 @@ mod test {
                             .join(consensus_signer.pubkey().to_string())
                             .join("TEST_MODULE")
                             .join("bls")
-                            .join(format!("{}.sig", proxy_signer.pubkey().to_string()))
+                            .join(format!("{}.sig", proxy_signer.pubkey()))
                     )
                     .unwrap()
                 )

crates/common/src/signer/store.rs

@@ -1,9 +1,6 @@
 use std::{collections::HashMap, io::Write, path::PathBuf};
 
-use alloy::{
-    hex,
-    primitives::{B256, aliases::B32},
-};
+use alloy::{hex, primitives::B256};
 use blsful::inner_types::{Field, G2Affine, G2Projective, Group, Scalar};
 use cb_common::{
     commit::request::{ConsensusProxyMap, ProxyDelegation, SignedProxyDelegation},

crates/signer/src/manager/dirk.rs

@@ -30,12 +30,11 @@ fn test_missing_registration_field(field_name: &str) -> String {
     let mut values: Value = serde_json::from_str(data).unwrap();
 
     // Remove specified field from the first validator's message
-    if let Value::Array(arr) = &mut values {
-        if let Some(first_validator) = arr.get_mut(0) {
-            if let Some(Value::Object(msg_obj)) = first_validator.get_mut("message") {
-                msg_obj.remove(field_name);
-            }
-        }
+    if let Value::Array(arr) = &mut values &&
+        let Some(first_validator) = arr.get_mut(0) &&
+        let Some(Value::Object(msg_obj)) = first_validator.get_mut("message")
+    {
+        msg_obj.remove(field_name);
     }
 
     // This should fail since the field is required