refactor: [torrust#1195] extract and move method

Author: josecelano

src/core/authentication/key/repository/in_memory.rs

@@ -27,4 +27,15 @@ impl InMemoryKeyRepository {
         let mut keys = self.keys.write().await;
         keys.clear();
     }
+
+    /// It resets the authentication keys with a new list of keys.
+    pub async fn reset_with(&self, peer_keys: Vec<PeerKey>) {
+        let mut keys_lock = self.keys.write().await;
+
+        keys_lock.clear();
+
+        for key in peer_keys {
+            keys_lock.insert(key.key.clone(), key.clone());
+        }
+    }
 }

src/core/authentication/mod.rs

@@ -239,22 +239,18 @@ impl Facade {
         self.in_memory_key_repository.remove(key).await;
     }
 
-    /// The `Tracker` stores the authentication keys in memory and in the database.
-    /// In case you need to restart the `Tracker` you can load the keys from the database
-    /// into memory with this function. Keys are automatically stored in the database when they
-    /// are generated.
+    /// The `Tracker` stores the authentication keys in memory and in the 
+    /// database. In case you need to restart the `Tracker` you can load the 
+    /// keys from the database into memory with this function. Keys are 
+    /// automatically stored in the database when they are generated.
     ///
     /// # Errors
     ///
     /// Will return a `database::Error` if unable to `load_keys` from the database.
     pub async fn load_keys_from_database(&self) -> Result<(), databases::error::Error> {
         let keys_from_database = self.db_key_repository.load_keys()?;
 
-        self.in_memory_key_repository.clear().await;
-
-        for key in keys_from_database {
-            self.in_memory_key_repository.insert(&key).await;
-        }
+        self.in_memory_key_repository.reset_with(keys_from_database).await;
 
         Ok(())
     }