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/transaction.h

@@ -31,7 +31,6 @@
 #include "caosdb/transaction_status.h"           // for TransactionStatus
 #include "caosdb/status_code.h"                  // for StatusCode
 #include "google/protobuf/util/json_util.h" // for MessageToJsonString, Jso...
-#include <cstddef>                          // for size_t
 #include <iterator>
 // IWYU pragma: no_include <ext/alloc_traits.h>
 #include <memory> // for shared_ptr, unique_ptr
@@ -180,7 +179,7 @@ class ResultSet {
 
 public:
   virtual ~ResultSet() = default;
-  [[nodiscard]] virtual auto Size() const noexcept -> size_t = 0;
+  [[nodiscard]] virtual auto Size() const noexcept -> int = 0;
   [[nodiscard]] virtual auto At(const int index) const -> const Entity & = 0;
   auto begin() const -> iterator;
   auto end() const -> iterator;
@@ -188,14 +187,14 @@ public:
 private:
   class iterator : public std::iterator<std::output_iterator_tag, Entity> {
   public:
-    explicit iterator(const ResultSet *result_set, size_t index = 0);
+    explicit iterator(const ResultSet *result_set, int index = 0);
     auto operator*() const -> const Entity &;
     auto operator++() -> iterator &;
     auto operator++(int) -> iterator;
     auto operator!=(const iterator &rhs) const -> bool;
 
   private:
-    size_t current_index = 0;
+    int current_index = 0;
     const ResultSet *result_set;
   };
 };
@@ -210,7 +209,7 @@ class MultiResultSet : public ResultSet {
 public:
   ~MultiResultSet() = default;
   explicit MultiResultSet(MultiTransactionResponse *response);
-  [[nodiscard]] inline auto Size() const noexcept -> size_t override {
+  [[nodiscard]] inline auto Size() const noexcept -> int override {
     return this->entities.size();
   }
   [[nodiscard]] inline auto At(const int index) const
@@ -234,9 +233,7 @@ public:
   explicit inline UniqueResult(IdResponse *idResponse)
     : entity(new Entity(idResponse)){};
   [[nodiscard]] auto GetEntity() const -> const Entity &;
-  [[nodiscard]] inline auto Size() const noexcept -> size_t override {
-    return 1;
-  }
+  [[nodiscard]] inline auto Size() const noexcept -> int override { return 1; }
   [[nodiscard]] inline auto At(const int index) const
     -> const Entity & override {
     if (index != 0) {