fix(reverts): results for the future

Author: jarjk

src/otp/migrations/mod.rs

@@ -1,25 +1,27 @@
 use super::otp_element::OTPDatabase;
 struct Migration<'a> {
     to_version: u16, // Database version which we are migrating on
-    migration_function: &'a dyn Fn(&mut OTPDatabase), // Function to execute the migration
+    migration_function: &'a dyn Fn(&mut OTPDatabase) -> color_eyre::Result<()>, // Function to execute the migration
 }
 const MIGRATIONS_LIST: [Migration; 1] = [Migration {
     to_version: 2,
     migration_function: &migrate_to_2,
 }];
 
-fn migrate_to_2(database: &mut OTPDatabase) {
+fn migrate_to_2(database: &mut OTPDatabase) -> color_eyre::Result<()> {
     database.version = 2;
+    Ok(())
 }
 
-pub fn migrate(database: &mut OTPDatabase) {
+pub fn migrate(database: &mut OTPDatabase) -> color_eyre::Result<()> {
     let mut binding = MIGRATIONS_LIST;
     let migrations = binding.as_mut();
     migrations.sort_unstable_by(|c1, c2| c1.to_version.cmp(&c2.to_version));
     for i in migrations {
         if database.version < i.to_version {
             // Do the migration
-            (i.migration_function)(database);
+            (i.migration_function)(database)?;
         }
     }
+    Ok(())
 }

src/otp/otp_element.rs

@@ -59,7 +59,7 @@ impl OTPDatabase {
 
     pub fn save(&mut self, key: &Vec<u8>, salt: &[u8]) -> color_eyre::Result<()> {
         self.needs_modification = false;
-        migrate(self);
+        migrate(self)?;
         match self.overwrite_database_key(key, salt) {
             Ok(()) => Ok(()),
             Err(e) => Err(ErrReport::from(e)),