From 02da7768a6e48635d0db2394a79639906aa78f52 Mon Sep 17 00:00:00 2001 From: lids <1713278948@qq.com> Date: Wed, 29 Apr 2026 15:31:33 +0800 Subject: [PATCH] =?UTF-8?q?AI=20=E8=87=AA=E5=8A=A8=E7=94=9F=E6=88=90?= =?UTF-8?q?=E6=B5=8B=E8=AF=95=E7=94=A8=E4=BE=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- tests/CMakeLists.txt | 8 +- tests/test_alert_manager.cpp | 449 ++++++++++++++++++----------------- 2 files changed, 236 insertions(+), 221 deletions(-) diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt index 59052db..d247ec0 100644 --- a/tests/CMakeLists.txt +++ b/tests/CMakeLists.txt @@ -1,5 +1,5 @@ cmake_minimum_required(VERSION 3.10.0) -project(CppGenerate_test) +project(test_CppGenerate) include(FetchContent) if (MSVC) add_compile_options(/utf-8) @@ -18,8 +18,8 @@ include_directories(${CMAKE_CURRENT_SOURCE_DIR}/../include) include(CTest) enable_testing() -add_executable(CppGenerate_test test_alert_manager.cpp ../src/alert_manager.cpp) +add_executable(test_CppGenerate test_alert_manager.cpp ../src/alert_manager.cpp) -target_link_libraries(CppGenerate_test gtest gmock gtest_main) +target_link_libraries(test_CppGenerate gtest gmock gtest_main) include(GoogleTest) -gtest_discover_tests(CppGenerate_test) \ No newline at end of file +gtest_discover_tests(test_CppGenerate) \ No newline at end of file diff --git a/tests/test_alert_manager.cpp b/tests/test_alert_manager.cpp index 5bc247a..35a8b8d 100644 --- a/tests/test_alert_manager.cpp +++ b/tests/test_alert_manager.cpp @@ -1,267 +1,282 @@ #include "gtest/gtest.h" #include "alert_manager.hpp" +#include #include -#include -// Mock for time to ensure deterministic tests -static uint32_t mock_timestamp = 1640995200; // 2022-01-01 00:00:00 UTC +using namespace std; -class MockTime { -public: - static uint32_t getTimestamp() { return mock_timestamp; } - static void setTimestamp(uint32_t t) { mock_timestamp = t; } -}; +// Mock time function for deterministic testing +uint32_t mock_timestamp = 1640995200; // 2022-01-01 00:00:00 UTC -// Helper function to compare AlertInfo -bool operator==(const AlertInfo& a, const AlertInfo& b) { - return a.type == b.type && - a.status == b.status && - a.trigger_time == b.trigger_time && - a.acknowledge_time == b.acknowledge_time && - a.resolve_time == b.resolve_time && - a.description == b.description; +// Mock the time function to return a fixed value +extern "C" uint32_t time(nullptr) { + return mock_timestamp; } -// Test fixture +// Helper function to get current timestamp +uint32_t getCurrentTimestamp() { + return mock_timestamp; +} + +// Test fixture for AlertManager class AlertManagerTest : public ::testing::Test { protected: - AlertManager manager; + AlertManager alert_manager; void SetUp() override { mock_timestamp = 1640995200; - manager.initialize(); + alert_manager.initialize(); } void TearDown() override { - manager.clearHistory(); + // Reset state + alert_manager.resetAllAlerts(); } }; -TEST_F(AlertManagerTest, testAlertManagerDefaultInitialization) { - EXPECT_FLOAT_EQ(manager.getAltitudeThresholds().first, 100.0f); - EXPECT_FLOAT_EQ(manager.getAltitudeThresholds().second, -50.0f); - EXPECT_TRUE(manager.getActiveAlerts().empty()); - EXPECT_TRUE(manager.getAlertHistory(10).empty()); +// Test case: checkAltitudeAlert - Normal input test +TEST_F(AlertManagerTest, testCheckAltitudeAlertNormalInput) { + // Arrange + float altitude = 80.0f; // Within normal range + uint32_t timestamp = getCurrentTimestamp(); + + // Act + alert_manager.checkAltitudeAlert(altitude, timestamp); + + // Assert + EXPECT_FALSE(alert_manager.hasActiveAlerts()); + EXPECT_EQ(alert_manager.getActiveAlerts().size(), 0); } -TEST_F(AlertManagerTest, testInitialize) { - // No direct output assertion, but verify initialization logic - // This is tested implicitly via constructor - EXPECT_NO_FATAL_FAILURE(manager.initialize()); +// Test case: checkAltitudeAlert - Upper threshold exceeded +TEST_F(AlertManagerTest, testCheckAltitudeAlertUpperThresholdExceeded) { + // Arrange + float altitude = 150.0f; // Above upper threshold (100.0f) + uint32_t timestamp = getCurrentTimestamp(); + + // Act + alert_manager.checkAltitudeAlert(altitude, timestamp); + + // Assert + EXPECT_TRUE(alert_manager.hasActiveAlerts()); + auto active_alerts = alert_manager.getActiveAlerts(); + EXPECT_EQ(active_alerts.size(), 1); + EXPECT_EQ(active_alerts[0].type, AlertType::ALTITUDE_UPPER); + EXPECT_EQ(active_alerts[0].status, AlertStatus::ACTIVE); + EXPECT_EQ(active_alerts[0].trigger_time, timestamp); } -TEST_F(AlertManagerTest, testCheckAltitudeAlertNormal) { - mock_timestamp = 1640995200; - manager.checkAltitudeAlert(105.0f, mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_EQ(alerts.size(), 1); - EXPECT_EQ(alerts[0].type, AlertType::ALTITUDE_UPPER); - EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE); - EXPECT_EQ(alerts[0].trigger_time, mock_timestamp); +// Test case: checkAltitudeAlert - Lower threshold exceeded +TEST_F(AlertManagerTest, testCheckAltitudeAlertLowerThresholdExceeded) { + // Arrange + float altitude = -60.0f; // Below lower threshold (-50.0f) + uint32_t timestamp = getCurrentTimestamp(); + + // Act + alert_manager.checkAltitudeAlert(altitude, timestamp); + + // Assert + EXPECT_TRUE(alert_manager.hasActiveAlerts()); + auto active_alerts = alert_manager.getActiveAlerts(); + EXPECT_EQ(active_alerts.size(), 1); + EXPECT_EQ(active_alerts[0].type, AlertType::ALTITUDE_LOWER); + EXPECT_EQ(active_alerts[0].status, AlertStatus::ACTIVE); + EXPECT_EQ(active_alerts[0].trigger_time, timestamp); } -TEST_F(AlertManagerTest, testCheckAltitudeAlertBelowLower) { - mock_timestamp = 1640995200; - manager.checkAltitudeAlert(-55.0f, mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_EQ(alerts.size(), 1); - EXPECT_EQ(alerts[0].type, AlertType::ALTITUDE_LOWER); - EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE); +// Test case: checkAltitudeAlert - Recovery from upper threshold +TEST_F(AlertManagerTest, testCheckAltitudeAlertUpperThresholdRecovery) { + // Arrange + float altitude_above = 150.0f; // Trigger alert + float altitude_normal = 80.0f; // Recover + uint32_t timestamp1 = getCurrentTimestamp(); + uint32_t timestamp2 = getCurrentTimestamp() + 1; + + // Act + alert_manager.checkAltitudeAlert(altitude_above, timestamp1); + alert_manager.checkAltitudeAlert(altitude_normal, timestamp2); + + // Assert + EXPECT_FALSE(alert_manager.hasActiveAlerts()); + auto active_alerts = alert_manager.getActiveAlerts(); + EXPECT_EQ(active_alerts.size(), 0); } -TEST_F(AlertManagerTest, testCheckAltitudeAlertResetWhenNormal) { - mock_timestamp = 1640995200; - manager.checkAltitudeAlert(105.0f, mock_timestamp); // Trigger upper - mock_timestamp = 1640995201; - manager.checkAltitudeAlert(90.0f, mock_timestamp); // Reset - auto alerts = manager.getActiveAlerts(); - EXPECT_TRUE(alerts.empty()); +// Test case: checkAltitudeAlert - Recovery from lower threshold +TEST_F(AlertManagerTest, testCheckAltitudeAlertLowerThresholdRecovery) { + // Arrange + float altitude_below = -60.0f; // Trigger alert + float altitude_normal = -40.0f; // Recover + uint32_t timestamp1 = getCurrentTimestamp(); + uint32_t timestamp2 = getCurrentTimestamp() + 1; + + // Act + alert_manager.checkAltitudeAlert(altitude_below, timestamp1); + alert_manager.checkAltitudeAlert(altitude_normal, timestamp2); + + // Assert + EXPECT_FALSE(alert_manager.hasActiveAlerts()); + auto active_alerts = alert_manager.getActiveAlerts(); + EXPECT_EQ(active_alerts.size(), 0); } -TEST_F(AlertManagerTest, testSetAltitudeThresholdsValid) { - manager.setAltitudeThresholds(200.0f, -100.0f); - auto thresholds = manager.getAltitudeThresholds(); - EXPECT_FLOAT_EQ(thresholds.first, 200.0f); - EXPECT_FLOAT_EQ(thresholds.second, -100.0f); +// Test case: setAltitudeThresholds - Normal input test +TEST_F(AlertManagerTest, testSetAltitudeThresholdsNormalInput) { + // Arrange + float upper = 200.0f; + float lower = -100.0f; + + // Act + alert_manager.setAltitudeThresholds(upper, lower); + + // Assert + EXPECT_EQ(alert_manager.getAltitudeThresholds().first, upper); + EXPECT_EQ(alert_manager.getAltitudeThresholds().second, lower); + EXPECT_FALSE(alert_manager.hasActiveAlerts()); } -TEST_F(AlertManagerTest, testSetAltitudeThresholdsInvalid) { - EXPECT_NO_FATAL_FAILURE(manager.setAltitudeThresholds(50.0f, 100.0f)); - // Should print error, but no crash - EXPECT_FLOAT_EQ(manager.getAltitudeThresholds().first, 100.0f); - EXPECT_FLOAT_EQ(manager.getAltitudeThresholds().second, -50.0f); +// Test case: setAltitudeThresholds - Invalid input (upper <= lower) +TEST_F(AlertManagerTest, testSetAltitudeThresholdsInvalidInput) { + // Arrange + float upper = 50.0f; + float lower = 100.0f; + + // Act & Assert + EXPECT_NO_THROW(alert_manager.setAltitudeThresholds(upper, lower)); + // Note: The function prints error but does not throw exception + // So we cannot use EXPECT_THROW here. + // Instead, verify that thresholds were not updated + EXPECT_EQ(alert_manager.getAltitudeThresholds().first, 100.0f); // Still default + EXPECT_EQ(alert_manager.getAltitudeThresholds().second, -50.0f); // Still default } -TEST_F(AlertManagerTest, testTriggerSensorFailure) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_EQ(alerts.size(), 1); - EXPECT_EQ(alerts[0].type, AlertType::SENSOR_FAILURE); - EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE); - EXPECT_EQ(alerts[0].trigger_time, mock_timestamp); +// Test case: setAltitudeThresholds - Edge case: upper == lower +TEST_F(AlertManagerTest, testSetAltitudeThresholdsEdgeCaseEqual) { + // Arrange + float upper = 100.0f; + float lower = 100.0f; + + // Act + alert_manager.setAltitudeThresholds(upper, lower); + + // Assert + EXPECT_EQ(alert_manager.getAltitudeThresholds().first, 100.0f); + EXPECT_EQ(alert_manager.getAltitudeThresholds().second, 100.0f); + // But since upper <= lower, it should not be accepted + // Verify no change in actual thresholds + EXPECT_EQ(alert_manager.getAltitudeThresholds().first, 100.0f); + EXPECT_EQ(alert_manager.getAltitudeThresholds().second, -50.0f); } -TEST_F(AlertManagerTest, testTriggerLowBattery) { - mock_timestamp = 1640995200; - manager.triggerLowBattery(mock_timestamp, 15.0f); - auto alerts = manager.getActiveAlerts(); - EXPECT_EQ(alerts.size(), 1); - EXPECT_EQ(alerts[0].type, AlertType::LOW_BATTERY); - EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE); - EXPECT_NE(alerts[0].description.find("15.0%"), std::string::npos); +// Test case: setAltitudeThresholds - Reset alerts after threshold change +TEST_F(AlertManagerTest, testSetAltitudeThresholdsResetAlerts) { + // Arrange + float altitude = 150.0f; + uint32_t timestamp = getCurrentTimestamp(); + + // Trigger an alert first + alert_manager.checkAltitudeAlert(altitude, timestamp); + EXPECT_TRUE(alert_manager.hasActiveAlerts()); + + // Change thresholds + float new_upper = 200.0f; + float new_lower = -100.0f; + + // Act + alert_manager.setAltitudeThresholds(new_upper, new_lower); + + // Assert + EXPECT_FALSE(alert_manager.hasActiveAlerts()); + EXPECT_EQ(alert_manager.getActiveAlerts().size(), 0); } -TEST_F(AlertManagerTest, testTriggerCommunicationError) { - mock_timestamp = 1640995200; - manager.triggerCommunicationError(mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_EQ(alerts.size(), 1); - EXPECT_EQ(alerts[0].type, AlertType::COMMUNICATION_ERROR); - EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE); +// Test case: checkAltitudeAlert - Multiple consecutive calls +TEST_F(AlertManagerTest, testCheckAltitudeAlertMultipleCalls) { + // Arrange + float altitude = 150.0f; + uint32_t timestamp = getCurrentTimestamp(); + + // Act + alert_manager.checkAltitudeAlert(altitude, timestamp); + alert_manager.checkAltitudeAlert(altitude, timestamp + 1); + + // Assert + EXPECT_TRUE(alert_manager.hasActiveAlerts()); + auto active_alerts = alert_manager.getActiveAlerts(); + EXPECT_EQ(active_alerts.size(), 1); + EXPECT_EQ(active_alerts[0].type, AlertType::ALTITUDE_UPPER); + EXPECT_EQ(active_alerts[0].status, AlertStatus::ACTIVE); + EXPECT_EQ(active_alerts[0].trigger_time, timestamp); } -TEST_F(AlertManagerTest, testAcknowledgeAllAlerts) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - manager.triggerLowBattery(mock_timestamp + 1, 10.0f); - mock_timestamp = 1640995202; - manager.acknowledgeAllAlerts(mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_TRUE(alerts.empty()); - EXPECT_EQ(alerts.size(), 0); +// Test case: setAltitudeThresholds - Valid input with negative values +TEST_F(AlertManagerTest, testSetAltitudeThresholdsValidNegative) { + // Arrange + float upper = -10.0f; + float lower = -50.0f; + + // Act + alert_manager.setAltitudeThresholds(upper, lower); + + // Assert + EXPECT_EQ(alert_manager.getAltitudeThresholds().first, upper); + EXPECT_EQ(alert_manager.getAltitudeThresholds().second, lower); + EXPECT_FALSE(alert_manager.hasActiveAlerts()); } -TEST_F(AlertManagerTest, testAcknowledgeAlert) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - mock_timestamp = 1640995201; - manager.acknowledgeAlert(AlertType::SENSOR_FAILURE, mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_TRUE(alerts.empty()); +// Test case: checkAltitudeAlert - Zero altitude +TEST_F(AlertManagerTest, testCheckAltitudeAlertZeroAltitude) { + // Arrange + float altitude = 0.0f; + uint32_t timestamp = getCurrentTimestamp(); + + // Act + alert_manager.checkAltitudeAlert(altitude, timestamp); + + // Assert + EXPECT_FALSE(alert_manager.hasActiveAlerts()); + EXPECT_EQ(alert_manager.getActiveAlerts().size(), 0); } -TEST_F(AlertManagerTest, testResetAlert) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - manager.resetAlert(AlertType::SENSOR_FAILURE); - auto alerts = manager.getActiveAlerts(); - EXPECT_TRUE(alerts.empty()); +// Test case: setAltitudeThresholds - Max float values +TEST_F(AlertManagerTest, testSetAltitudeThresholdsMaxFloat) { + // Arrange + float upper = std::numeric_limits::max(); + float lower = std::numeric_limits::lowest(); + + // Act + alert_manager.setAltitudeThresholds(upper, lower); + + // Assert + EXPECT_EQ(alert_manager.getAltitudeThresholds().first, upper); + EXPECT_EQ(alert_manager.getAltitudeThresholds().second, lower); + EXPECT_FALSE(alert_manager.hasActiveAlerts()); } -TEST_F(AlertManagerTest, testResetAllAlerts) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - manager.triggerLowBattery(mock_timestamp + 1, 10.0f); - manager.resetAllAlerts(); - auto alerts = manager.getActiveAlerts(); - EXPECT_TRUE(alerts.empty()); +// Test case: checkAltitudeAlert - Altitude exactly at upper threshold +TEST_F(AlertManagerTest, testCheckAltitudeAlertAtUpperThreshold) { + // Arrange + float altitude = 100.0f; // Exactly equal to upper threshold + uint32_t timestamp = getCurrentTimestamp(); + + // Act + alert_manager.checkAltitudeAlert(altitude, timestamp); + + // Assert + EXPECT_FALSE(alert_manager.hasActiveAlerts()); + EXPECT_EQ(alert_manager.getActiveAlerts().size(), 0); } -TEST_F(AlertManagerTest, testHasActiveAlerts) { - EXPECT_FALSE(manager.hasActiveAlerts()); - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - EXPECT_TRUE(manager.hasActiveAlerts()); -} +// Test case: checkAltitudeAlert - Altitude exactly at lower threshold +TEST_F(AlertManagerTest, testCheckAltitudeAlertAtLowerThreshold) { + // Arrange + float altitude = -50.0f; // Exactly equal to lower threshold + uint32_t timestamp = getCurrentTimestamp(); -TEST_F(AlertManagerTest, testHasUnacknowledgedAlerts) { - EXPECT_FALSE(manager.hasUnacknowledgedAlerts()); - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - EXPECT_TRUE(manager.hasUnacknowledgedAlerts()); - manager.acknowledgeAllAlerts(mock_timestamp + 1); - EXPECT_FALSE(manager.hasUnacknowledgedAlerts()); -} + // Act + alert_manager.checkAltitudeAlert(altitude, timestamp); -TEST_F(AlertManagerTest, testGetActiveAlerts) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_EQ(alerts.size(), 1); - EXPECT_EQ(alerts[0].type, AlertType::SENSOR_FAILURE); -} - -TEST_F(AlertManagerTest, testGetAlertHistoryMaxCount) { - mock_timestamp = 1640995200; - for (int i = 0; i < 15; ++i) { - manager.triggerSensorFailure(mock_timestamp + i); - } - auto history = manager.getAlertHistory(10); - EXPECT_EQ(history.size(), 10); -} - -TEST_F(AlertManagerTest, testGetStatusDescriptionNoAlerts) { - std::string desc = manager.getStatusDescription(); - EXPECT_EQ(desc, "系统状态：正常"); -} - -TEST_F(AlertManagerTest, testGetStatusDescriptionWithAlerts) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - std::string desc = manager.getStatusDescription(); - EXPECT_NE(desc.find("有1个活动预警"), std::string::npos); - EXPECT_NE(desc.find("传感器故障"), std::string::npos); -} - -TEST_F(AlertManagerTest, testClearHistory) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - manager.clearHistory(); - EXPECT_TRUE(manager.getAlertHistory(10).empty()); -} - -TEST_F(AlertManagerTest, testAddToHistoryLimit) { - mock_timestamp = 1640995200; - for (int i = 0; i < 100; ++i) { - manager.triggerSensorFailure(mock_timestamp + i); - } - auto history = manager.getAlertHistory(10); - EXPECT_EQ(history.size(), 10); -} - -TEST_F(AlertManagerTest, testTriggerAlert) { - mock_timestamp = 1640995200; - manager.triggerAlert(AlertType::SENSOR_FAILURE, mock_timestamp, "Test alert"); - auto alerts = manager.getActiveAlerts(); - EXPECT_EQ(alerts.size(), 1); - EXPECT_EQ(alerts[0].description, "Test alert"); -} - -TEST_F(AlertManagerTest, testUpdateAlertStatusAcknowledged) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - mock_timestamp = 1640995201; - manager.updateAlertStatus(AlertType::SENSOR_FAILURE, AlertStatus::ACKNOWLEDGED, mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_TRUE(alerts.empty()); -} - -TEST_F(AlertManagerTest, testUpdateAlertStatusResolved) { - mock_timestamp = 1640995200; - manager.triggerSensorFailure(mock_timestamp); - mock_timestamp = 1640995201; - manager.updateAlertStatus(AlertType::SENSOR_FAILURE, AlertStatus::RESOLVED, mock_timestamp); - auto alerts = manager.getActiveAlerts(); - EXPECT_TRUE(alerts.empty()); -} - -TEST_F(AlertManagerTest, testGetAlertTypeName) { - EXPECT_EQ(manager.getAlertTypeName(AlertType::ALTITUDE_UPPER), "高度上限预警"); - EXPECT_EQ(manager.getAlertTypeName(AlertType::NONE), "无"); - EXPECT_EQ(manager.getAlertTypeName(static_cast(99)), "未知预警"); -} - -TEST_F(AlertManagerTest, testGetAlertStatusName) { - EXPECT_EQ(manager.getAlertStatusName(AlertStatus::ACTIVE), "预警中"); - EXPECT_EQ(manager.getAlertStatusName(AlertStatus::INACTIVE), "未激活"); - EXPECT_EQ(manager.getAlertStatusName(static_cast(99)), "未知状态"); -} - -TEST_F(AlertManagerTest, testDestructorDoesNotCrash) { - // Verify destructor doesn't throw or crash - EXPECT_NO_FATAL_FAILURE({ - AlertManager* mgr = new AlertManager(); - delete mgr; - }); + // Assert + EXPECT_FALSE(alert_manager.hasActiveAlerts()); + EXPECT_EQ(alert_manager.getActiveAlerts().size(), 0); } \ No newline at end of file