diff --git a/Bigfoot/Tests/Utils/CMakeLists.txt b/Bigfoot/Tests/Utils/CMakeLists.txt index 6c98a7c..eb59da7 100644 --- a/Bigfoot/Tests/Utils/CMakeLists.txt +++ b/Bigfoot/Tests/Utils/CMakeLists.txt @@ -2,4 +2,6 @@ get_filename_component(PackageName ${CMAKE_CURRENT_SOURCE_DIR} NAME) project(${PackageName}Tests) bigfoot_create_package_tests( - "") \ No newline at end of file + "") + +bigfoot_create_logger() \ No newline at end of file diff --git a/Bigfoot/Tests/Utils/Include/UtilsTests/UtilsTestsLogger_generated.hpp b/Bigfoot/Tests/Utils/Include/UtilsTests/UtilsTestsLogger_generated.hpp new file mode 100644 index 0000000..956be17 --- /dev/null +++ b/Bigfoot/Tests/Utils/Include/UtilsTests/UtilsTestsLogger_generated.hpp @@ -0,0 +1,21 @@ +// AUTO-GENERATED DO NOT TOUCH + +/********************************************************************* + * \file UtilsTestsLogger.generated.hpp + * + *********************************************************************/ +#ifndef BIGFOOT_UTILSTESTSLOGGER_GENERATED_HPP +#define BIGFOOT_UTILSTESTSLOGGER_GENERATED_HPP +#include + +#if defined BIGFOOT_NOT_OPTIMIZED + +namespace Bigfoot +{ +/* + * Logger + */ +inline Log::LoggerInfo UTILSTESTS_LOGGER {"UTILSTESTS_LOGGER", Flat::LogLevel::Trace}; +} // namespace Bigfoot +#endif +#endif diff --git a/Bigfoot/Tests/Utils/Log/Log.cpp b/Bigfoot/Tests/Utils/Log/Log.cpp index 7183fe9..afd1a92 100644 --- a/Bigfoot/Tests/Utils/Log/Log.cpp +++ b/Bigfoot/Tests/Utils/Log/Log.cpp @@ -8,6 +8,8 @@ #include +#include + #if defined BIGFOOT_NOT_OPTIMIZED #include @@ -39,60 +41,57 @@ class LogFixture: public ::testing::Test return Flat::LogLevel::Trace; } - - void TearDown() override - { - Singleton::GetInstance().UnregisterLogger(m_logger); - } - - Log::LoggerInfo m_logger {"LOGGER", Flat::LogLevel::Trace}; }; /****************************************************************************************/ TEST_F(LogFixture, RegisterLogger_ShouldRegisterTheLogger) { - const quill::Logger* logger = Singleton::GetInstance().RegisterLogger(m_logger); + const quill::Logger* logger = Singleton::GetInstance().RegisterLogger(UTILSTESTS_LOGGER); EXPECT_TRUE(logger); - EXPECT_EQ(logger, Singleton::GetInstance().GetLogger(m_logger)); - EXPECT_EQ(logger->get_logger_name(), m_logger.m_name); - EXPECT_EQ(QuillLogLevelToLogLevel(logger->get_log_level()), m_logger.m_level); + EXPECT_EQ(logger, Singleton::GetInstance().GetLogger(UTILSTESTS_LOGGER)); + EXPECT_EQ(logger->get_logger_name(), UTILSTESTS_LOGGER.m_name); + EXPECT_EQ(QuillLogLevelToLogLevel(logger->get_log_level()), UTILSTESTS_LOGGER.m_level); } /****************************************************************************************/ TEST_F(LogFixture, UnregisterLogger_ShouldUnregisterTheLogger) { - std::ignore = Singleton::GetInstance().RegisterLogger(m_logger); - Singleton::GetInstance().UnregisterLogger(m_logger); + std::ignore = Singleton::GetInstance().RegisterLogger(UTILSTESTS_LOGGER); + Singleton::GetInstance().UnregisterLogger(UTILSTESTS_LOGGER); - EXPECT_EQ(Singleton::GetInstance().GetLogger(m_logger), nullptr); + EXPECT_EQ(Singleton::GetInstance().GetLogger(UTILSTESTS_LOGGER), nullptr); } /****************************************************************************************/ TEST_F(LogFixture, GetLogger_ShouldReturnNullptrIfTheLoggerDoesNotExist) { - EXPECT_FALSE(Singleton::GetInstance().GetLogger(m_logger)); + EXPECT_FALSE(Singleton::GetInstance().GetLogger(UTILSTESTS_LOGGER)); } /****************************************************************************************/ -TEST_F(LogFixture, GetLogger_ShouldReturnTheLoggerIfItExists) +TEST_F(LogFixture, GetLoger_ShouldReturnTheLoggerIfItExists) { [[maybe_unused]] - const quill::Logger* logger = Singleton::GetInstance().RegisterLogger(m_logger); - EXPECT_TRUE(Singleton::GetInstance().GetLogger(m_logger)); + const quill::Logger* logger = Singleton::GetInstance().RegisterLogger(UTILSTESTS_LOGGER); + EXPECT_TRUE(Singleton::GetInstance().GetLogger(UTILSTESTS_LOGGER)); } /****************************************************************************************/ TEST_F(LogFixture, ChangeLoggerLogLevel_ShouldChangeTheLoggerLogLevel) { - const quill::Logger* logger = Singleton::GetInstance().RegisterLogger(m_logger); + const Flat::LogLevel previous = UTILSTESTS_LOGGER.m_level; - Singleton::GetInstance().ChangeLoggerLogLevel(m_logger, Flat::LogLevel::Critical); + const quill::Logger* logger = Singleton::GetInstance().RegisterLogger(UTILSTESTS_LOGGER); + + Singleton::GetInstance().ChangeLoggerLogLevel(UTILSTESTS_LOGGER, Flat::LogLevel::Critical); EXPECT_EQ(QuillLogLevelToLogLevel(logger->get_log_level()), Flat::LogLevel::Critical); + Singleton::GetInstance().ChangeLoggerLogLevel(UTILSTESTS_LOGGER, previous); + EXPECT_EQ(QuillLogLevelToLogLevel(logger->get_log_level()), previous); } } // namespace Bigfoot #endif diff --git a/Bigfoot/Tests/Utils/Log/LogMacros.cpp b/Bigfoot/Tests/Utils/Log/LogMacros.cpp index 5c370d6..aa6e328 100644 --- a/Bigfoot/Tests/Utils/Log/LogMacros.cpp +++ b/Bigfoot/Tests/Utils/Log/LogMacros.cpp @@ -6,6 +6,8 @@ *********************************************************************/ #include +#include + #if defined BIGFOOT_NOT_OPTIMIZED #include @@ -15,54 +17,48 @@ namespace Bigfoot class LogMacrosFixture: public ::testing::Test { protected: - void TearDown() override - { - Singleton::GetInstance().UnregisterLogger(m_logger); - } - - Log::LoggerInfo m_logger {"LOGGER", Flat::LogLevel::Trace}; }; /****************************************************************************************/ TEST_F(LogMacrosFixture, LogDebug) { - BIGFOOT_LOG_DEBUG(m_logger, "Hello"); + BIGFOOT_LOG_DEBUG(UTILSTESTS_LOGGER, "Hello"); } /****************************************************************************************/ TEST_F(LogMacrosFixture, LogTrace) { - BIGFOOT_LOG_TRACE(m_logger, "Hello"); + BIGFOOT_LOG_TRACE(UTILSTESTS_LOGGER, "Hello"); } /****************************************************************************************/ TEST_F(LogMacrosFixture, LogInfo) { - BIGFOOT_LOG_INFO(m_logger, "Hello"); + BIGFOOT_LOG_INFO(UTILSTESTS_LOGGER, "Hello"); } /****************************************************************************************/ TEST_F(LogMacrosFixture, LogWarn) { - BIGFOOT_LOG_WARN(m_logger, "Hello"); + BIGFOOT_LOG_WARN(UTILSTESTS_LOGGER, "Hello"); } /****************************************************************************************/ TEST_F(LogMacrosFixture, LogError) { - BIGFOOT_LOG_ERROR(m_logger, "Hello"); + BIGFOOT_LOG_ERROR(UTILSTESTS_LOGGER, "Hello"); } /****************************************************************************************/ TEST_F(LogMacrosFixture, LogFatal) { - BIGFOOT_LOG_FATAL(m_logger, "Hello"); + BIGFOOT_LOG_FATAL(UTILSTESTS_LOGGER, "Hello"); } } // namespace Bigfoot #endif