Refs #15841: Fix tests that set monitors ad-hoc and then they fail in…

… destruction Signed-off-by: jparisu <javierparis@eprosima.com>
eProsima · Oct 7, 2022 · b1744c5 · b1744c5
1 parent bb84d67
commit b1744c5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/test/unittest/Database/DatabaseStatusTests.cpp b/test/unittest/Database/DatabaseStatusTests.cpp
@@ -43,6 +43,7 @@ class database_status_tests : public ::testing::Test
         // Simulate that the backend is monitorizing the domain
         // NOTE: This is so F*** dangerous, please do not do it again (1)
         std::shared_ptr<details::Monitor> monitor = std::make_shared<details::Monitor>();
+        monitor->id = domain->id;
         details::StatisticsBackendData::get_instance()->monitors_by_entity_[domain->id] = monitor;
 
         // Simulate the discover of the entitiy
@@ -81,11 +82,11 @@ class database_status_tests : public ::testing::Test
     }
 
 
-    void TearDown()
-    {
-        // NOTE: This is thanks to (1) brilliant idea
-        details::StatisticsBackendData::get_instance()->monitors_by_entity_.erase(domain->id);
-    }
+    // void TearDown()
+    // {
+    //     // NOTE: This is thanks to (1) brilliant idea
+    //     details::StatisticsBackendData::get_instance()->monitors_by_entity_.erase(domain->id);
+    // }
 
     std::shared_ptr<Host> host;
     std::shared_ptr<User> user;

diff --git a/test/unittest/StatisticsBackend/IsActiveTests.cpp b/test/unittest/StatisticsBackend/IsActiveTests.cpp
@@ -60,6 +60,7 @@ class is_active_tests : public ::testing::Test
 
         // Simulate that the backend is monitorizing the domain
         std::shared_ptr<details::Monitor> monitor = std::make_shared<details::Monitor>();
+        monitor->id = domain->id;
         details::StatisticsBackendData::get_instance()->monitors_by_entity_[domain->id] = monitor;
 
         // Simulate the discover of the entitiy