Fix tests with Log: wait for Log background thread initialization on …

…the first queued entry (#3604) * Fix test Signed-off-by: Irene Bandera <[email protected]> * Uncrustifiy Signed-off-by: Irene Bandera <[email protected]> --------- Signed-off-by: Irene Bandera <[email protected]> (cherry picked from commit 5ffc8a4) Co-authored-by: Irene Bandera Moreno <[email protected]>
eProsima · Jul 14, 2023 · 45e6cee · 45e6cee
1 parent e69fed8
commit 45e6cee
Showing 1 changed file with 14 additions and 6 deletions.
diff --git a/src/cpp/fastdds/log/Log.cpp b/src/cpp/fastdds/log/Log.cpp
@@ -209,8 +209,14 @@ struct LogResources
         {
             std::unique_lock<std::mutex> guard(cv_mutex_);
             work_ = true;
+            // pessimization
+            cv_.notify_all();
+            // wait till the thread is initialized
+            cv_.wait(guard, [&]
+                    {
+                        return current_loop_;
+                    });
         }
-        cv_.notify_all();
     }
 
     //! Stops the logging_ thread. It will re-launch on the next call to QueueLog.
@@ -269,14 +275,16 @@ struct LogResources
                 logs_.Swap();
                 while (!logs_.Empty())
                 {
-                    std::unique_lock<std::mutex> configGuard(config_mutex_);
-
                     Log::Entry& entry = logs_.Front();
-                    if (preprocess(entry))
                     {
-                        for (auto& consumer : consumers_)
+                        std::unique_lock<std::mutex> configGuard(config_mutex_);
+
+                        if (preprocess(entry))
                         {
-                            consumer->Consume(entry);
+                            for (auto& consumer : consumers_)
+                            {
+                                consumer->Consume(entry);
+                            }
                         }
                     }
                     // This Pop() is also a barrier for Log::Flush wait condition