Minor Clippy/Test fixes

Signed-off-by: lli <[email protected]>
hyperledger · Jul 25, 2024 · a65a52c · a65a52c
1 parent cae1399
commit a65a52c
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 6 deletions.
diff --git a/aries/agents/aath-backchannel/src/controllers/did_exchange.rs b/aries/agents/aath-backchannel/src/controllers/did_exchange.rs
@@ -202,7 +202,7 @@ impl HarnessAgent {
         let (thid, pthid, my_did, their_did) = self
             .aries_agent
             .did_exchange()
-            .handle_msg_request(request, &inviter_key, opt_invitation)
+            .handle_msg_request(request, &inviter_key.clone(), opt_invitation)
             .await?;
 
         if let Some(pthid) = pthid {

diff --git a/aries/agents/aath-backchannel/src/controllers/issuance.rs b/aries/agents/aath-backchannel/src/controllers/issuance.rs
@@ -95,6 +95,7 @@ async fn download_tails_file(
     let url = match tails_base_url.to_string().matches('/').count() {
         0 => format!("{}/{}", tails_base_url, rev_reg_id),
         1.. => tails_base_url.to_string(),
+        _ => todo!(),
     };
     let client = reqwest::Client::new();
     let tails_folder_path = std::env::current_dir()

diff --git a/aries/agents/aries-vcx-agent/src/handlers/did_exchange.rs b/aries/agents/aries-vcx-agent/src/handlers/did_exchange.rs
@@ -125,7 +125,7 @@ impl<T: BaseWallet> DidcommHandlerDidExchange<T> {
     pub async fn handle_msg_request(
         &self,
         request: AnyRequest,
-        request_did: &String,
+        request_did: &str,
         invitation: Option<OobInvitation>,
     ) -> AgentResult<(String, Option<String>, String, String)> {
         // todo: type the return type

diff --git a/aries/aries_vcx/src/protocols/did_exchange/mod.rs b/aries/aries_vcx/src/protocols/did_exchange/mod.rs
@@ -116,11 +116,11 @@ pub async fn resolve_enc_key_from_invitation(
 }
 
 pub async fn resolve_enc_key_from_did(
-    did: &String,
+    did: &str,
     resolver_registry: &Arc<ResolverRegistry>,
 ) -> Result<Key, AriesVcxError> {
     let output = resolver_registry
-        .resolve(&did.clone().try_into()?, &Default::default())
+        .resolve(&did.try_into()?, &Default::default())
         .await
         .map_err(|err| {
             AriesVcxError::from_msg(

diff --git a/aries/aries_vcx/tests/test_did_exchange.rs b/aries/aries_vcx/tests/test_did_exchange.rs
@@ -135,7 +135,7 @@ async fn did_exchange_test(
         &resolver_registry,
         request,
         &responders_peer_did,
-        Some(invitation_key.clone()),
+        invitation_key.clone(),
     )
     .await
     .unwrap();
@@ -337,7 +337,7 @@ async fn did_exchange_test_with_invalid_rotation_signature() -> Result<(), Box<d
         request,
         &responders_peer_did,
         // sign with NOT the invitation key
-        Some(incorrect_invitation_key),
+        incorrect_invitation_key,
     )
     .await?;