Remove signer

xmtp · Apr 7, 2024 · a93bfe6 · a93bfe6
1 parent 4174ba6
commit a93bfe6
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 74 deletions.
diff --git a/xmtp_id/src/associations/builder.rs b/xmtp_id/src/associations/builder.rs
@@ -247,7 +247,7 @@ fn build_action(
                 .get(&SignatureField::InitialAddress)
                 .ok_or(SignatureRequestError::MissingSigner)?;
             let initial_address_signature = signatures
-                .get(&signer_identity)
+                .get(signer_identity)
                 .cloned()
                 .ok_or(SignatureRequestError::MissingSigner)?;
 
@@ -268,12 +268,12 @@ fn build_action(
                 .ok_or(SignatureRequestError::MissingSigner)?;
 
             let existing_member_signature = signatures
-                .get(&existing_member_signer_identity)
+                .get(existing_member_signer_identity)
                 .cloned()
                 .ok_or(SignatureRequestError::MissingSigner)?;
 
             let new_member_signature = signatures
-                .get(&new_member_signer_identity)
+                .get(new_member_signer_identity)
                 .cloned()
                 .ok_or(SignatureRequestError::MissingSigner)?;
 
@@ -289,7 +289,7 @@ fn build_action(
                 .get(&SignatureField::RecoveryAddress)
                 .ok_or(SignatureRequestError::MissingSigner)?;
             let recovery_address_signature = signatures
-                .get(&signer_identity)
+                .get(signer_identity)
                 .cloned()
                 .ok_or(SignatureRequestError::MissingSigner)?;
 
@@ -305,7 +305,7 @@ fn build_action(
                 .ok_or(SignatureRequestError::MissingSigner)?;
 
             let recovery_address_signature = signatures
-                .get(&signer_identity)
+                .get(signer_identity)
                 .cloned()
                 .ok_or(SignatureRequestError::MissingSigner)?;
 

diff --git a/xmtp_id/src/associations/mod.rs b/xmtp_id/src/associations/mod.rs
@@ -3,7 +3,6 @@ pub mod builder;
 mod hashes;
 mod member;
 mod signature;
-mod signer;
 mod state;
 #[cfg(test)]
 mod test_utils;

diff --git a/xmtp_id/src/associations/signer.rs b/xmtp_id/src/associations/signer.rs
diff --git a/xmtp_id/src/associations/test_utils.rs b/xmtp_id/src/associations/test_utils.rs
@@ -72,36 +72,3 @@ impl Signature for MockSignature {
         sig.as_bytes().to_vec()
     }
 }
-
-// #[derive(Clone)]
-// pub struct MockSigner {
-//     identity: MemberIdentifier,
-//     signature_kind: SignatureKind,
-// }
-
-// impl Signer for MockSigner {
-//     fn signature_kind(&self) -> SignatureKind {
-//         self.signature_kind.clone()
-//     }
-//     fn signer_identity(&self) -> MemberIdentifier {
-//         self.identity.clone()
-//     }
-
-//     fn sign(&self, text: &str) -> Result<Box<dyn Signature>, SignerError> {
-//         Ok(MockSignature::new_boxed(
-//             true,
-//             self.signer_identity(),
-//             self.signature_kind(),
-//             Some(text.to_string()),
-//         ))
-//     }
-// }
-
-// impl MockSigner {
-//     pub fn new_boxed(identity: MemberIdentifier, signature_kind: SignatureKind) -> Box<Self> {
-//         Box::new(Self {
-//             identity,
-//             signature_kind,
-//         })
-//     }
-// }