Use value instead

Author: Hind-M

tests/CMakeLists.txt

@@ -7,7 +7,7 @@ set(test_target "test_sparrow_ipc_lib")
 set(
     SPARROW_IPC_TESTS_SRC
     include/sparrow_ipc_tests_helpers.hpp
-    # TODO move all these below under src?
+    # TODO move all the files below under src?
     main.cpp
     test_utils.cpp
     test_primitive_array_serialization.cpp

tests/include/sparrow_ipc_tests_helpers.hpp

@@ -17,8 +17,8 @@ namespace sparrow_ipc
         }
 
         CHECK(arr2.metadata().has_value());
-        sp::key_value_view kvs1_view = *(arr1.metadata()); // TODO use value() instead
-        sp::key_value_view kvs2_view = *(arr2.metadata());
+        sp::key_value_view kvs1_view = arr1.metadata().value();
+        sp::key_value_view kvs2_view = arr2.metadata().value();
 
         CHECK_EQ(kvs1_view.size(), kvs2_view.size());
         auto kvs1_it = kvs1_view.cbegin();