Enable running umfIpcTest tests when free() is not supported

Some memory providers (currently the devdax and the file providers)
do not support the free() operation (free() always returns
the UMF_RESULT_ERROR_NOT_SUPPORTED error).

Add the UMF_TEST_PROVIDER_FREE_NOT_SUPPORTED define
and the get_umf_result_of_free macro() to ipcFixtures.hpp
to enable running umfIpcTest tests when free() is not supported.

Signed-off-by: Lukasz Dorau <[email protected]>

Author: ldorau

test/ipcFixtures.hpp

@@ -19,6 +19,12 @@
 #include <numeric>
 #include <tuple>
 
+#ifdef UMF_TEST_PROVIDER_FREE_NOT_SUPPORTED
+#define get_umf_result_of_free(expected_result) UMF_RESULT_ERROR_NOT_SUPPORTED
+#else
+#define get_umf_result_of_free(expected_result) (expected_result)
+#endif
+
 class MemoryAccessor {
   public:
     virtual void fill(void *ptr, size_t size, const void *pattern,
@@ -163,7 +169,7 @@ TEST_P(umfIpcTest, GetIPCHandleInvalidArgs) {
     EXPECT_EQ(ret, UMF_RESULT_ERROR_INVALID_ARGUMENT);
 
     ret = umfFree(ptr);
-    EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+    EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
 }
 
 TEST_P(umfIpcTest, BasicFlow) {
@@ -218,7 +224,7 @@ TEST_P(umfIpcTest, BasicFlow) {
     EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
 
     ret = umfPoolFree(pool.get(), ptr);
-    EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+    EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
 
     // Test if umfCloseIPCHandle() did not corrupt the pool
     // and if umfPoolMalloc() works correctly after umfCloseIPCHandle().
@@ -230,7 +236,7 @@ TEST_P(umfIpcTest, BasicFlow) {
     memAccessor->fill(ptr, SIZE, &pattern, sizeof(pattern));
 
     ret = umfPoolFree(pool.get(), ptr);
-    EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+    EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
 
     pool.reset(nullptr);
     EXPECT_EQ(stat.getCount, 1);
@@ -294,7 +300,7 @@ TEST_P(umfIpcTest, GetPoolByOpenedHandle) {
 
     for (size_t i = 0; i < NUM_ALLOCS; ++i) {
         umf_result_t ret = umfFree(ptrs[i]);
-        EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+        EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
     }
 }
 
@@ -320,7 +326,7 @@ TEST_P(umfIpcTest, AllocFreeAllocTest) {
     EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
 
     ret = umfPoolFree(pool.get(), ptr);
-    EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+    EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
 
     ptr = umfPoolMalloc(pool.get(), SIZE);
     ASSERT_NE(ptr, nullptr);
@@ -338,7 +344,7 @@ TEST_P(umfIpcTest, AllocFreeAllocTest) {
     EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
 
     ret = umfPoolFree(pool.get(), ptr);
-    EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+    EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
 
     // Test if umfCloseIPCHandle() did not corrupt the pool
     // and if umfPoolMalloc() works correctly after umfCloseIPCHandle().
@@ -350,7 +356,7 @@ TEST_P(umfIpcTest, AllocFreeAllocTest) {
     memAccessor->fill(ptr, SIZE, &pattern, sizeof(pattern));
 
     ret = umfPoolFree(pool.get(), ptr);
-    EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+    EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
 
     pool.reset(nullptr);
     EXPECT_EQ(stat.getCount, stat.putCount);
@@ -401,7 +407,7 @@ TEST_P(umfIpcTest, openInTwoPools) {
     EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
 
     ret = umfPoolFree(pool1.get(), ptr);
-    EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+    EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
 
     pool1.reset(nullptr);
     pool2.reset(nullptr);
@@ -452,7 +458,7 @@ TEST_P(umfIpcTest, ConcurrentGetPutHandles) {
 
     for (void *ptr : ptrs) {
         umf_result_t ret = umfPoolFree(pool.get(), ptr);
-        EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+        EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
     }
 
     pool.reset(nullptr);
@@ -514,7 +520,7 @@ TEST_P(umfIpcTest, ConcurrentOpenCloseHandles) {
 
     for (void *ptr : ptrs) {
         umf_result_t ret = umfPoolFree(pool.get(), ptr);
-        EXPECT_EQ(ret, UMF_RESULT_SUCCESS);
+        EXPECT_EQ(ret, get_umf_result_of_free(UMF_RESULT_SUCCESS));
     }
 
     pool.reset(nullptr);