F move entity

Summary

Introduction/Refactoring of move/copy constructors and assignment operators for several classes.

Focus

• Always mind the Rule of 5
• New move and copy constructors and assignment operators for DataType.
• New == and != operators for ProtoMessageWrapper and Entity (needed for testing).
• Simplified move constructor, copy/move assignment operators of Property.
• New move constructor, copy/move assignment operators for Entity.
• The new test cover all four things (move/copy constructors/assigment operators) for Entity, Property, and Parent.

Test Environment

Pipeline testing should be sufficient

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

• All automated tests pass
• Reference related Issues
• Up-to-date CHANGELOG.md
• 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 specifications? Are they satisfied?

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

include/caosdb/entity.h

@@ -51,8 +51,8 @@
 #include <random>                                      // for mt19937, rand...
 #include <stdexcept>                                   // for out_of_range
 #include <string>                                      // for string, basic...
-#include <vector>                                      // for vector
 #include <utility>                                     // for move
+#include <vector>                                      // for vector
 
 namespace caosdb::entity {
 using boost::filesystem::exists;
@@ -635,6 +635,9 @@ private:
  */
 class Entity : public ScalarProtoMessageWrapper<ProtoEntity> {
 public:
+  /**
+   * Copy constructor.
+   */
   inline Entity(const Entity &original)
     : Entity(ProtoMessageWrapper::CopyProtoMessage(original.wrapped)) {
     this->errors.wrapped->CopyFrom(*original.errors.wrapped);
@@ -671,6 +674,54 @@ public:
     parents.wrapped = this->wrapped->mutable_parents();
   };
 
+  /**
+   * Move constructor.
+   */
+  explicit inline Entity(Entity &&original)
+    : wrapped(std::move(original.wrapped)), value(Value(std::move(original.value.wrapped))),
+      data_type(DataType(std::move(original.data_type.wrapped))) {
+    this->properties.wrapped = std::move(original.properties.wrapped);
+    this->parents.wrapped = std::move(original.parents.wrapped);
+    this->errors.wrapped = std::move(original.errors.wrapped);
+    this->warnings.wrapped = std::move(original.warnings.wrapped);
+    this->infos.wrapped = std::move(original.infos.wrapped);
+  };
+
+  /**
+   * Move assignment operator.
+   */
+  auto operator=(Entity &&other) -> Entity & {
+    this->wrapped = std::move(other.wrapped);
+    this->data_type.wrapped = std::move(other.data_type.wrapped);
+    this->value.wrapped = std::move(other.value.wrapped);
+    this->properties.wrapped = std::move(other.properties.wrapped);
+    this->parents.wrapped = std::move(other.parents.wrapped);
+    this->file_descriptor = std::move(other.file_descriptor);
+    this->errors.wrapped = std::move(other.errors.wrapped);
+    this->warnings.wrapped = std::move(other.warnings.wrapped);
+    this->infos.wrapped = std::move(other.infos.wrapped);
+    return *this;
+  }
+
+  /**
+   * Copy assignment operator.
+   */
+  auto operator=(const Entity &other) -> Entity & {
+    this->wrapped->CopyFrom(*other.wrapped);
+    this->data_type.wrapped->CopyFrom(*other.data_type.wrapped);
+    this->value.wrapped->CopyFrom(*other.value.wrapped);
+    this->properties.wrapped->CopyFrom(*other.properties.wrapped);
+    this->parents.wrapped->CopyFrom(*other.parents.wrapped);
+    this->file_descriptor.local_path = boost::filesystem::path(other.file_descriptor.local_path);
+    this->file_descriptor.file_transmission_id->CopyFrom(
+      *other.file_descriptor.file_transmission_id);
+    this->file_descriptor.wrapped->CopyFrom(*other.file_descriptor.wrapped);
+    this->errors.wrapped->CopyFrom(*other.errors.wrapped);
+    this->warnings.wrapped->CopyFrom(*other.warnings.wrapped);
+    this->infos.wrapped->CopyFrom(*other.infos.wrapped);
+    return *this;
+  }
+
   [[nodiscard]] inline auto GetId() const noexcept -> const std::string & {
     return this->wrapped->id();
   };
@@ -787,6 +838,15 @@ public:
     infos.Clear();
   }
 
+  /**
+   * Return true if the other entity is equal to to this entity.
+   *
+   * This method ignores the errors, warnings and info messages.
+   */
+  inline auto operator==(const Entity &other) const noexcept -> bool {
+    return this->wrapped->SerializeAsString() == other.wrapped->SerializeAsString();
+  }
+
 private:
   inline auto GetNextFileId() -> std::string {
     std::string str = "0123456789abcdef";