Fix ASAN errors in tests (#6)

* Fix Owner Ptr leak int test

Signed-off-by: jparisu <javierparis@eprosima.com>

* Fix SlotThreadPool asan error in test

Signed-off-by: jparisu <javierparis@eprosima.com>

Signed-off-by: jparisu <javierparis@eprosima.com>

Author: jparisu

cpp_utils/test/unittest/memory/OwnerPtrTest.cpp

@@ -143,10 +143,11 @@ TEST(OwnerPtrTest, owner_ptr_custom_deleter)
 {
     // Variable to store deleter calls
     int deleter_calls = 0;
+    int* internal_value = new int(42);
 
     // Create Ptr object from int with custom deleter
     OwnerPtr<int> ptr(
-        new int(42),
+        internal_value,
         [&deleter_calls](int*)
         {
             deleter_calls++;
@@ -163,6 +164,9 @@ TEST(OwnerPtrTest, owner_ptr_custom_deleter)
 
     // As deleter did not destroy object, it should still be accessible
     ASSERT_EQ(actual_value_inside, 42);
+
+    // Actually remove object as the deleter does not
+    delete internal_value;
 }
 
 /**

cpp_utils/test/unittest/thread_pool/slot_thread_pool_test.cpp

@@ -99,12 +99,12 @@ TEST(slot_thread_pool_test, pool_one_thread_n_slots)
     eprosima::utils::Timer timer;
 
     // Create slot
-    for (uint32_t i = 0; i < test::N_EXECUTIONS_IN_TEST; ++i)
+    for (uint32_t i = 1; i <= test::N_EXECUTIONS_IN_TEST; ++i)
     {
         TaskId task_id(i);
         thread_pool.slot(
             task_id,
-            [&waiter, &i]
+            [&waiter, i]
                 ()
             {
                 test::test_lambda_increase_waiter(waiter, i);
@@ -113,7 +113,7 @@ TEST(slot_thread_pool_test, pool_one_thread_n_slots)
     }
 
     // Emit every task 1 time
-    for (uint32_t i = 0; i < test::N_EXECUTIONS_IN_TEST; ++i)
+    for (uint32_t i = 1; i <= test::N_EXECUTIONS_IN_TEST; ++i)
     {
         thread_pool.emit(TaskId(i));
     }