F update

Summary

Implements update for a single entity

See https://gitlab.indiscale.com/caosdb/customers/lfpb/management/-/issues/392

Focus

• Updating entities works like this: Retrieve an entity, copy it and put it in the update transaction.
• ResultSet has an iterator now
• I privatated SetId and SetVersionId (nobody should ever need to change it, because Entities which are to be updated have to be retrieved first anyway.)
• More fine-grained TransactionStatus (INITIAL -> GO_ON -> READY -> EXECUTE -> SUCCESS/ERROR) and corresponding StatusCodes

Test Environment

Use with server branch f-grpc-f-update

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

@@ -374,6 +374,9 @@ private:
 class Entity {
 public:
   Entity();
+  inline Entity(const Entity &original) : Entity(CreateProtoEntity()) {
+    this->wrapped->CopyFrom(*original.wrapped);
+  };
   explicit Entity(IdResponse *idResponse);
   explicit inline Entity(ProtoEntity *wrapped) : wrapped(wrapped) {
     errors.wrapped = this->wrapped->mutable_errors();
@@ -383,9 +386,12 @@ public:
     parents.wrapped = this->wrapped->mutable_parents();
   };
 
-  [[nodiscard]] inline auto GetId() const -> const std::string & {
+  [[nodiscard]] inline auto GetId() const noexcept -> const std::string & {
     return wrapped->id();
   };
+  [[nodiscard]] inline auto HasId() const noexcept -> bool {
+    return !wrapped->id().empty();
+  }
   [[nodiscard]] inline auto GetVersionId() const -> const std::string & {
     return wrapped->version().id();
   };
@@ -435,9 +441,7 @@ public:
   }
 
   auto SetRole(const std::string &role) -> void;
-  auto SetId(const std::string &id) -> void;
   auto SetName(const std::string &name) -> void;
-  auto SetVersionId(const std::string &id) -> void;
 
   auto SetValue(const std::string &value) -> void;
   auto SetUnit(const std::string &unit) -> void;
@@ -451,8 +455,10 @@ public:
    */
   auto CopyTo(ProtoEntity *target) -> void;
 
-private:
+protected:
   static auto CreateProtoEntity() -> ProtoEntity *;
+  auto SetId(const std::string &id) -> void;
+  auto SetVersionId(const std::string &id) -> void;
   ProtoEntity *wrapped;
   Properties properties;
   Parents parents;