tests, objects: improve coverage (99%) and fix found issues

Signed-off-by: Eloïse Brocas <ebrocas@quarkslab.com>

Author: ebrocas

src/pyrrha_mapper/common/objects.py

@@ -333,7 +333,9 @@ def replace_function(self, new_func: Symbol, old_func: Symbol, keep_old_name: bo
             name = old_func.name
         else:
             name = new_func.name
-        if self.exported_function_exists(old_func.name):
+        is_exported = self.exported_function_exists(old_func.name)
+        self.remove_function(old_func.name)
+        if is_exported:
             self.add_exported_symbol(new_func, symbol_name=name)
         else:
             self.add_function(new_func, func_name=name)
@@ -595,7 +597,7 @@ def fs_sym_validate(cls, data: Any, info: ValidationInfo) -> Any:
         # then recursively resolve symlinks
         nb_symlinks = len(res) + len(untreated_symlinks)
         while len(res) < nb_symlinks:
-            for path, content in untreated_symlinks.items():
+            for path, content in list(untreated_symlinks.items()):
                 target_path = content["target_path"]
                 if target_path in res:
                     content["target"] = res[target_path]
@@ -743,4 +745,4 @@ def resolve_symlink(self, symlink: Symlink) -> Binary | None:
             current_symlink = self.get_symlink_by_path(current_symlink.target.path)
         if not self.binary_exists(current_symlink.target.path):
             return None
-        return self.get_binary_by_path(current_symlink.target.path)
+        return self.get_binary_by_path(current_symlink.target.path)