Fix edit_as_admin tests

Author: nikwithak

crates/bitwarden-vault/src/cipher/cipher_client/create.rs

@@ -342,15 +342,13 @@ impl CiphersClient {
 #[cfg(test)]
 mod tests {
     use bitwarden_api_api::{apis::ApiClient, models::CipherResponseModel};
-    use bitwarden_core::Client;
     use bitwarden_core::{
-        ClientSettings, DeviceType, UserId,
+        Client, ClientSettings, DeviceType, UserId,
         key_management::crypto::{
             InitOrgCryptoRequest, InitUserCryptoMethod, InitUserCryptoRequest,
         },
     };
-    use bitwarden_crypto::SymmetricCryptoKey;
-    use bitwarden_crypto::{EncString, Kdf};
+    use bitwarden_crypto::{EncString, Kdf, SymmetricCryptoKey};
     use bitwarden_test::{MemoryRepository, start_api_mock};
     use chrono::Utc;
     use wiremock::{

crates/bitwarden-vault/src/cipher/cipher_client/delete.rs

@@ -41,7 +41,8 @@ impl CiphersClient {
         Ok(())
     }
 
-    /// Deletes all [Cipher] objects with a matching [CipherId] from the server, using the admin endpoint.
+    /// Deletes all [Cipher] objects with a matching [CipherId] from the server, using the admin
+    /// endpoint.
     pub async fn delete_many_as_admin(
         &self,
         cipher_ids: Vec<CipherId>,
@@ -97,7 +98,8 @@ impl CiphersClient {
         Ok(())
     }
 
-    /// Soft-deletes the [Cipher] with the matching [CipherId] from the server, using the admin endpoint.
+    /// Soft-deletes the [Cipher] with the matching [CipherId] from the server, using the admin
+    /// endpoint.
     pub async fn soft_delete_as_admin(&self, cipher_id: CipherId) -> Result<(), DeleteCipherError> {
         let configs = self.get_api_configurations().await;
         let api = configs.api_client.ciphers_api();
@@ -124,7 +126,8 @@ impl CiphersClient {
         Ok(())
     }
 
-    /// Soft-deletes all [Cipher] objects for the given [CipherId]s from the server, using the admin endpoint.
+    /// Soft-deletes all [Cipher] objects for the given [CipherId]s from the server, using the admin
+    /// endpoint.
     pub async fn soft_delete_many_as_admin(
         &self,
         cipher_ids: Vec<CipherId>,
@@ -151,7 +154,6 @@ mod tests {
     };
     use bitwarden_crypto::{EncString, Kdf};
     use bitwarden_test::{MemoryRepository, start_api_mock};
-
     use chrono::Utc;
     use wiremock::{
         Mock, MockServer, ResponseTemplate,

crates/bitwarden-vault/src/cipher/cipher_client/edit.rs

@@ -758,16 +758,25 @@ mod tests {
         ])
         .await;
         let client = create_client_with_wiremock(&mock_server).await;
+        let repository = client.get_repository().unwrap();
 
         let cipher_view = generate_test_cipher();
-        let request = cipher_view.try_into().unwrap();
+        repository
+            .set(
+                TEST_CIPHER_ID.to_string(),
+                client.encrypt(cipher_view.clone()).unwrap().cipher,
+            )
+            .await
+            .unwrap();
 
+        let request = cipher_view.try_into().unwrap();
+        let start_time = Utc::now();
         let result = client.edit_as_admin(request).await.unwrap();
 
-        let repository = client.get_repository().unwrap();
         let cipher = repository.get(TEST_CIPHER_ID.to_string()).await.unwrap();
         // Should not update local repository for admin endpoints.
-        assert!(matches!(cipher, None));
+        assert!(result.revision_date > start_time);
+        assert!(cipher.unwrap().revision_date < start_time);
 
         assert_eq!(result.id, TEST_CIPHER_ID.parse().ok());
         assert_eq!(result.name, "Test Login");

crates/bitwarden-vault/src/cipher/cipher_client/restore.rs

@@ -124,7 +124,6 @@ mod tests {
     };
     use bitwarden_crypto::{EncString, Kdf};
     use bitwarden_test::{MemoryRepository, start_api_mock};
-
     use chrono::Utc;
     use wiremock::{
         Mock, MockServer, ResponseTemplate,