Tests for LogAndAbortTracer

Author: heinwessels-philips

services/tracer/CMakeLists.txt

@@ -17,6 +17,8 @@ target_link_libraries(services.tracer PUBLIC
 target_sources(services.tracer PRIVATE
     GlobalTracer.cpp
     GlobalTracer.hpp
+    LogAndAbortTracer.cpp
+    LogAndAbortTracer.hpp
     StreamWriterOnSerialCommunication.cpp
     StreamWriterOnSerialCommunication.hpp
     StreamWriterOnSynchronousSerialCommunication.cpp

services/util/LogAndAbortTracer.cpp services/tracer/LogAndAbortTracer.cppservices/util/LogAndAbortTracer.cpp renamed to services/tracer/LogAndAbortTracer.cpp

@@ -1,4 +1,4 @@
-#include "services/util/LogAndAbortTracer.hpp"
+#include "services/tracer/LogAndAbortTracer.hpp"
 #include "services/tracer/TracerAdapterPrintf.hpp"
 
 namespace services

services/util/LogAndAbortTracer.hpp services/tracer/LogAndAbortTracer.hppservices/util/LogAndAbortTracer.hpp renamed to services/tracer/LogAndAbortTracer.hpp

@@ -11,6 +11,7 @@ target_link_libraries(services.tracer_test PUBLIC
 )
 
 target_sources(services.tracer_test PRIVATE
+    TestLogAndAbortTracer.cpp
     TestStreamWriterOnSerialCommunication.cpp
     TestStreamWriterOnSynchronousSerialCommunication.cpp
     TestTracer.cpp

services/tracer/test/CMakeLists.txt

@@ -0,0 +1,48 @@
+#include "services/tracer/LogAndAbortTracer.hpp"
+#include "services/tracer/TracerOnIoOutputInfrastructure.hpp"
+#include "gmock/gmock.h"
+#include "gtest/gtest.h"
+
+namespace
+{
+    struct StdOutTracer
+    {
+        // Basically TracerOnIoOutputInfrastructure, but does not set global tracer
+
+        infra::IoOutputStreamWriter writer;
+        infra::TextOutputStream::WithErrorPolicy stream{ writer };
+        services::TracerWithDateTime tracer{ stream };
+    };
+
+    class LogAndAbortTracerTest
+        : public testing::Test
+    {
+    public:
+        StdOutTracer tracer;
+    };
+}
+
+TEST_F(LogAndAbortTracerTest, log_and_abort_without_registered_tracer_doesnt_call_tracer)
+{
+    testing::internal::CaptureStdout();
+    // Calling hook directly to avoid aborting the test process.
+    infra::ExecuteLogAndAbortHook("speak %s and enter", "elf");
+    std::string output = testing::internal::GetCapturedStdout();
+
+    EXPECT_THAT(output, testing::HasSubstr(""));
+}
+
+TEST_F(LogAndAbortTracerTest, log_and_abort_with_registered_tracer_calls_tracer)
+{
+    services::RegisterLogAndAbortTracerProvider([this]() -> services::Tracer&
+        {
+            return tracer.tracer;
+        });
+
+    testing::internal::CaptureStdout();
+    // Calling hook directly to avoid aborting the test process.
+    infra::ExecuteLogAndAbortHook("speak %s and enter", "mellon");
+    std::string output = testing::internal::GetCapturedStdout();
+
+    EXPECT_THAT(output, testing::HasSubstr("speak mellon and enter"));
+}

services/tracer/test/TestLogAndAbortTracer.cpp

@@ -51,8 +51,6 @@ target_sources(services.util PRIVATE
     GpioPinInverted.hpp
     I2cMultipleAccess.cpp
     I2cMultipleAccess.hpp
-    LogAndAbortTracer.cpp
-    LogAndAbortTracer.hpp
     LowPowerSerialCommunication.cpp
     LowPowerSerialCommunication.hpp
     LowPowerSpiMaster.cpp