BUG: Fix missing virtual destructor

include/caosdb/protobuf_helper.h

@@ -44,6 +44,7 @@ auto get_arena() -> Arena *;
 
 template <typename P> class ProtoMessageWrapper {
 public:
+  virtual ~ProtoMessageWrapper() = 0;
   ProtoMessageWrapper(const ProtoMessageWrapper<P> &other) = default;
   inline auto CopyFrom(const ProtoMessageWrapper<P> &other) noexcept -> StatusCode {
     this->wrapped->CopyFrom(*other.wrapped);
@@ -86,8 +87,11 @@ protected:
   P *wrapped;
 };
 
+template <typename P> ProtoMessageWrapper<P>::~ProtoMessageWrapper() = default;
+
 template <typename P> class ScalarFieldWrapper : public ProtoMessageWrapper<P> {
 public:
+  inline virtual ~ScalarFieldWrapper() = default;
   inline ScalarFieldWrapper() = default;
   inline ScalarFieldWrapper(P *wrapped) : ProtoMessageWrapper<P>(wrapped) {}
   /**

test/test_entity.cpp

@@ -439,10 +439,7 @@ TEST(test_entity, test_remove_property) {
   // P0,P1,P2,P3,P4,P5,P6,P7,P8,P9
   //          ^
   // P0,P1,P2,   P4,P5,P6,P7,P8,P9
-  CAOSDB_LOG_WARN(logger_name) << "REMOVE(3)!!!!!!!!!!!!!!!!!!!!!";
-  CAOSDB_LOG_WARN(logger_name) << entity.GetProperties().ToString();
   entity.RemoveProperty(3);
-  CAOSDB_LOG_WARN(logger_name) << entity.GetProperties().ToString();
 
   ASSERT_EQ(entity.GetProperties().size(), 9);
   for (int i = 0; i < 3; i++) {