Before cleanup

Author: EuphoricThinking

test/memspaces/memspace_fixtures.hpp

@@ -74,11 +74,11 @@ struct memspaceGetTest : ::numaNodesTest,
         ASSERT_NE(hMemspace, nullptr);
     }
 
-    public:
-    static void SetUpTestSuite() {
-        //ASSERT_TRUE(false) << "YOU SHALL NOT PASS";
-        GTEST_SKIP() << "SKIP ME BABY";
-    }
+    // public:
+    // static void SetUpTestSuite() {
+    //     //ASSERT_TRUE(false) << "YOU SHALL NOT PASS";
+    //     GTEST_SKIP() << "SKIP ME BABY";
+    // }
 
     umf_const_memspace_handle_t hMemspace = nullptr;
 };
@@ -87,15 +87,15 @@ struct memspaceProviderTest : ::memspaceGetTest {
     void SetUp() override {
         ::memspaceGetTest::SetUp();
 
-        // auto [isQuerySupported, memspaceGet] = ::memspaceGetTest::GetParam();
-        // bool restinpiss = isQuerySupported(nodeIds.front());
+        auto [isQuerySupported, memspaceGet] = ::memspaceGetTest::GetParam();
+        bool restinpiss = isQuerySupported(nodeIds.front());
 
-        // if (!restinpiss || numa_available() == -1 || numa_all_nodes_ptr == nullptr) {
-        //     GTEST_SKIP() << "Dkippig 2222";
-        // }
-        if (::memspaceGetTest::IsSkipped()) {
-            GTEST_SKIP() << "SIPPING I WAS WROOONG";
+        if (!restinpiss || numa_available() == -1 || numa_all_nodes_ptr == nullptr) {
+            GTEST_SKIP() << "Dkippig 2222";
         }
+        // if (::memspaceGetTest::IsSkipped()) {
+        //     GTEST_SKIP() << "SIPPING I WAS WROOONG";
+        // }
 
         umf_result_t ret =
             umfMemoryProviderCreateFromMemspace(hMemspace, nullptr, &hProvider);

test/memspaces/memspace_lowest_latency.cpp

@@ -37,7 +37,7 @@ static bool canQueryLatency(size_t nodeId) {
     }
     ret = hwloc_topology_load(topology);
     //UT_ASSERTeq(ret, 0);
-    if (!GTEST_OUT_NE(ret, 0)) {
+    if (!GTEST_OUT_EQ(ret, 0)) {
         return false;
     }