F fix comparison

Summary

This is a fix for #94 (closed). These comparisons might be needed in the new crawler framework. Please review whether the suggested amendment could potentially break other things.

Test Environment

There are unit tests supplied with this merge request.

Check List for the Author

• All automated tests pass
• Reference related Issues
• Up-to-date CHANGELOG.md
• Add type hints in created/changed code
• Annotations in code (Gitlab comments)
  • Intent of new code
  • Problems with old code
  • Why this implementation?

Check List for the Reviewer

• I understand the intent of this MR
• All automated tests pass
• Up-to-date CHANGELOG.md
• The test environment setup works and the intended behavior is reproducible in the test environment
• In-code documentation and comments are up-to-date.
• Check: Are there spezifications? Are they satisfied?

For further good practices have a look at our review guidelines.

src/caosdb/apiutils.py

@@ -558,8 +558,8 @@ def getCommitIn(folder):
         return t.readline().strip()
 
 
-COMPARED = ["name", "role", "datatype", "description", "importance"]
-
+COMPARED = ["name", "role", "datatype", "description", "importance",
+            "id", "path", "checksum", "size"]
 
 def compare_entities(old_entity: Entity, new_entity: Entity):
     """
@@ -613,7 +613,7 @@ def compare_entities(old_entity: Entity, new_entity: Entity):
 
             if new_entity_attr_exists:
                 newdiff[attr] = newattr
-
+                
     # properties
 
     for prop in old_entity.properties:
@@ -653,7 +653,7 @@ def compare_entities(old_entity: Entity, new_entity: Entity):
                 olddiff["properties"].pop(prop.name)
 
         else:
-            raise NotImplementedError()
+            raise NotImplementedError("Comparison not implemented for multi-properties.")
 
     for prop in new_entity.properties:
         if len([0 for p in old_entity.properties if p.name == prop.name]) == 0: