From bdfef98c34eecd6f3c2ceb1f8052c2e886444dd7 Mon Sep 17 00:00:00 2001
From: lids <1713278948@qq.com>
Date: Wed, 29 Apr 2026 15:16:29 +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 | 712 +++++++++++++++++++++++++----------
 2 files changed, 517 insertions(+), 203 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..f7c1ff7 100644
--- a/tests/test_alert_manager.cpp
+++ b/tests/test_alert_manager.cpp
@@ -3,265 +3,579 @@
 #include <vector>
 #include <string>
 
-// Mock for time to ensure deterministic tests
+using namespace std;
+
+// Mock time for deterministic testing
 static uint32_t mock_timestamp = 1640995200; // 2022-01-01 00:00:00 UTC
 
-class MockTime { 
-public:
-    static uint32_t getTimestamp() { return mock_timestamp; }
-    static void setTimestamp(uint32_t t) { mock_timestamp = t; }
-};
-
-// 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 function to replace std::time()
+uint32_t getMockTime() {
+    return mock_timestamp;
 }
 
-// Test fixture
+// Override the global time function in AlertManager
+void setMockTime(uint32_t time) {
+    mock_timestamp = time;
+}
+
+// Test fixture for AlertManager
 class AlertManagerTest : public ::testing::Test {
 protected:
-    AlertManager manager;
+    AlertManager alert_manager;
 
     void SetUp() override {
+        // Reset mock timestamp before each test
         mock_timestamp = 1640995200;
-        manager.initialize();
+        alert_manager.initialize();
     }
 
     void TearDown() override {
-        manager.clearHistory();
+        // Clear any remaining alerts
+        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 checkAltitudeAlert - Normal Input
+TEST_F(AlertManagerTest, testCheckAltitudeAlertNormalInput) {
+    // Given: Altitude within normal range
+    float altitude = 80.0f;
+    uint32_t timestamp = getMockTime();
+
+    // When: Check altitude alert
+    alert_manager.checkAltitudeAlert(altitude, timestamp);
+
+    // Then: No alert should be triggered
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
 }
 
-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 checkAltitudeAlert - Upper Threshold Exceeded
+TEST_F(AlertManagerTest, testCheckAltitudeAlertUpperThresholdExceeded) {
+    // Given: Altitude exceeds upper threshold
+    float altitude = 101.0f; // Above default 100.0f
+    uint32_t timestamp = getMockTime();
+
+    // When: Check altitude alert
+    alert_manager.checkAltitudeAlert(altitude, timestamp);
+
+    // Then: Upper altitude alert should be active
+    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);
 }
 
-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 checkAltitudeAlert - Lower Threshold Exceeded
+TEST_F(AlertManagerTest, testCheckAltitudeAlertLowerThresholdExceeded) {
+    // Given: Altitude below lower threshold
+    float altitude = -51.0f; // Below default -50.0f
+    uint32_t timestamp = getMockTime();
+
+    // When: Check altitude alert
+    alert_manager.checkAltitudeAlert(altitude, timestamp);
+
+    // Then: Lower altitude alert should be active
+    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);
 }
 
-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 checkAltitudeAlert - Recovery from Upper Threshold
+TEST_F(AlertManagerTest, testCheckAltitudeAlertRecoveryFromUpperThreshold) {
+    // Given: Initially exceed upper threshold
+    float altitude = 101.0f;
+    uint32_t timestamp = getMockTime();
+    alert_manager.checkAltitudeAlert(altitude, timestamp);
+
+    // When: Return to normal range
+    altitude = 99.0f;
+    alert_manager.checkAltitudeAlert(altitude, timestamp + 1);
+
+    // Then: Alert should be resolved
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
 }
 
-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 checkAltitudeAlert - Recovery from Lower Threshold
+TEST_F(AlertManagerTest, testCheckAltitudeAlertRecoveryFromLowerThreshold) {
+    // Given: Initially below lower threshold
+    float altitude = -51.0f;
+    uint32_t timestamp = getMockTime();
+    alert_manager.checkAltitudeAlert(altitude, timestamp);
+
+    // When: Return to normal range
+    altitude = -49.0f;
+    alert_manager.checkAltitudeAlert(altitude, timestamp + 1);
+
+    // Then: Alert should be resolved
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
 }
 
-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 setAltitudeThresholds - Valid Thresholds
+TEST_F(AlertManagerTest, testSetAltitudeThresholdsValidThresholds) {
+    // Given: Valid upper and lower thresholds
+    float upper = 150.0f;
+    float lower = -100.0f;
+
+    // When: Set thresholds
+    alert_manager.setAltitudeThresholds(upper, lower);
+
+    // Then: Thresholds should be updated
+    EXPECT_EQ(alert_manager.getAltitudeThresholds().first, upper);
+    EXPECT_EQ(alert_manager.getAltitudeThresholds().second, lower);
 }
 
-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 setAltitudeThresholds - Invalid Thresholds (upper <= lower)
+TEST_F(AlertManagerTest, testSetAltitudeThresholdsInvalidThresholds) {
+    // Given: Invalid thresholds
+    float upper = 50.0f;
+    float lower = 100.0f;
+
+    // When: Set thresholds
+    alert_manager.setAltitudeThresholds(upper, lower);
+
+    // Then: Should not update and print error
+    EXPECT_EQ(alert_manager.getAltitudeThresholds().first, 100.0f);
+    EXPECT_EQ(alert_manager.getAltitudeThresholds().second, -50.0f);
 }
 
-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 triggerSensorFailure - Normal Trigger
+TEST_F(AlertManagerTest, testTriggerSensorFailureNormalTrigger) {
+    // Given: Timestamp
+    uint32_t timestamp = getMockTime();
+
+    // When: Trigger sensor failure
+    alert_manager.triggerSensorFailure(timestamp);
+
+    // Then: Sensor failure alert should be active
+    EXPECT_TRUE(alert_manager.hasActiveAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 1);
+    EXPECT_EQ(active_alerts[0].type, AlertType::SENSOR_FAILURE);
+    EXPECT_EQ(active_alerts[0].status, AlertStatus::ACTIVE);
+    EXPECT_EQ(active_alerts[0].description, "传感器故障");
 }
 
-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 triggerLowBattery - Normal Trigger
+TEST_F(AlertManagerTest, testTriggerLowBatteryNormalTrigger) {
+    // Given: Timestamp and battery level
+    uint32_t timestamp = getMockTime();
+    float battery_level = 15.0f;
+
+    // When: Trigger low battery
+    alert_manager.triggerLowBattery(timestamp, battery_level);
+
+    // Then: Low battery alert should be active
+    EXPECT_TRUE(alert_manager.hasActiveAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 1);
+    EXPECT_EQ(active_alerts[0].type, AlertType::LOW_BATTERY);
+    EXPECT_EQ(active_alerts[0].status, AlertStatus::ACTIVE);
+    EXPECT_EQ(active_alerts[0].description, "低电量警告：当前电量15%") << "Description mismatch";
 }
 
-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 triggerCommunicationError - Normal Trigger
+TEST_F(AlertManagerTest, testTriggerCommunicationErrorNormalTrigger) {
+    // Given: Timestamp
+    uint32_t timestamp = getMockTime();
+
+    // When: Trigger communication error
+    alert_manager.triggerCommunicationError(timestamp);
+
+    // Then: Communication error alert should be active
+    EXPECT_TRUE(alert_manager.hasActiveAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 1);
+    EXPECT_EQ(active_alerts[0].type, AlertType::COMMUNICATION_ERROR);
+    EXPECT_EQ(active_alerts[0].status, AlertStatus::ACTIVE);
+    EXPECT_EQ(active_alerts[0].description, "通信错误");
 }
 
-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());
+// Test acknowledgeAllAlerts - Acknowledge All Active Alerts
+TEST_F(AlertManagerTest, testAcknowledgeAllAlertsAcknowledgeAllActiveAlerts) {
+    // Given: Multiple active alerts
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+    alert_manager.triggerLowBattery(timestamp + 1, 10.0f);
+
+    // When: Acknowledge all alerts
+    alert_manager.acknowledgeAllAlerts(timestamp + 2);
+
+    // Then: All alerts should be acknowledged
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 0);
+}
+
+// Test acknowledgeAlert - Acknowledge Specific Alert
+TEST_F(AlertManagerTest, testAcknowledgeAlertAcknowledgeSpecificAlert) {
+    // Given: One active alert
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+
+    // When: Acknowledge specific alert
+    alert_manager.acknowledgeAlert(AlertType::SENSOR_FAILURE, timestamp + 1);
+
+    // Then: Alert should be acknowledged
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 0);
+}
+
+// Test acknowledgeAlert - Try to Acknowledge Non-Existing Alert
+TEST_F(AlertManagerTest, testAcknowledgeAlertTryToAcknowledgeNonExistingAlert) {
+    // Given: No existing alert
+    uint32_t timestamp = getMockTime();
+
+    // When: Acknowledge non-existing alert
+    alert_manager.acknowledgeAlert(AlertType::ALTITUDE_UPPER, timestamp);
+
+    // Then: No change in state
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
+}
+
+// Test resetAlert - Reset Existing Alert
+TEST_F(AlertManagerTest, testResetAlertResetExistingAlert) {
+    // Given: An active alert
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+
+    // When: Reset the alert
+    alert_manager.resetAlert(AlertType::SENSOR_FAILURE);
+
+    // Then: Alert should be gone
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 0);
+}
+
+// Test resetAlert - Reset Non-Existing Alert
+TEST_F(AlertManagerTest, testResetAlertResetNonExistingAlert) {
+    // Given: No existing alert
+    // When: Reset non-existing alert
+    alert_manager.resetAlert(AlertType::ALTITUDE_UPPER);
+
+    // Then: No effect
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
+}
+
+// Test resetAllAlerts - Reset All Alerts
+TEST_F(AlertManagerTest, testResetAllAlertsResetAllAlerts) {
+    // Given: Multiple active alerts
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+    alert_manager.triggerLowBattery(timestamp + 1, 10.0f);
+
+    // When: Reset all alerts
+    alert_manager.resetAllAlerts();
+
+    // Then: All alerts should be cleared
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 0);
+}
+
+// Test hasActiveAlerts - No Active Alerts
+TEST_F(AlertManagerTest, testHasActiveAlertsNoActiveAlerts) {
+    // Given: No active alerts
+    // When: Check for active alerts
+    bool result = alert_manager.hasActiveAlerts();
+
+    // Then: Should return false
+    EXPECT_FALSE(result);
+}
+
+// Test hasActiveAlerts - With Active Alerts
+TEST_F(AlertManagerTest, testHasActiveAlertsWithActiveAlerts) {
+    // Given: One active alert
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+
+    // When: Check for active alerts
+    bool result = alert_manager.hasActiveAlerts();
+
+    // Then: Should return true
+    EXPECT_TRUE(result);
+}
+
+// Test hasUnacknowledgedAlerts - No Unacknowledged Alerts
+TEST_F(AlertManagerTest, testHasUnacknowledgedAlertsNoUnacknowledgedAlerts) {
+    // Given: No unacknowledged alerts
+    // When: Check for unacknowledged alerts
+    bool result = alert_manager.hasUnacknowledgedAlerts();
+
+    // Then: Should return false
+    EXPECT_FALSE(result);
+}
+
+// Test hasUnacknowledgedAlerts - With Unacknowledged Alerts
+TEST_F(AlertManagerTest, testHasUnacknowledgedAlertsWithUnacknowledgedAlerts) {
+    // Given: One active alert
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+
+    // When: Check for unacknowledged alerts
+    bool result = alert_manager.hasUnacknowledgedAlerts();
+
+    // Then: Should return true
+    EXPECT_TRUE(result);
+}
+
+// Test getActiveAlerts - Empty List
+TEST_F(AlertManagerTest, testGetActiveAlertsEmptyList) {
+    // Given: No active alerts
+    // When: Get active alerts
+    auto alerts = alert_manager.getActiveAlerts();
+
+    // Then: Should return empty vector
     EXPECT_EQ(alerts.size(), 0);
 }
 
-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 getActiveAlerts - With Active Alerts
+TEST_F(AlertManagerTest, testGetActiveAlertsWithActiveAlerts) {
+    // Given: One active alert
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
 
-TEST_F(AlertManagerTest, testResetAlert) {
-    mock_timestamp = 1640995200;
-    manager.triggerSensorFailure(mock_timestamp);
-    manager.resetAlert(AlertType::SENSOR_FAILURE);
-    auto alerts = manager.getActiveAlerts();
-    EXPECT_TRUE(alerts.empty());
-}
+    // When: Get active alerts
+    auto alerts = alert_manager.getActiveAlerts();
 
-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_F(AlertManagerTest, testHasActiveAlerts) {
-    EXPECT_FALSE(manager.hasActiveAlerts());
-    mock_timestamp = 1640995200;
-    manager.triggerSensorFailure(mock_timestamp);
-    EXPECT_TRUE(manager.hasActiveAlerts());
-}
-
-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());
-}
-
-TEST_F(AlertManagerTest, testGetActiveAlerts) {
-    mock_timestamp = 1640995200;
-    manager.triggerSensorFailure(mock_timestamp);
-    auto alerts = manager.getActiveAlerts();
+    // Then: Should return one alert
     EXPECT_EQ(alerts.size(), 1);
     EXPECT_EQ(alerts[0].type, AlertType::SENSOR_FAILURE);
+    EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE);
 }
 
-TEST_F(AlertManagerTest, testGetAlertHistoryMaxCount) {
-    mock_timestamp = 1640995200;
-    for (int i = 0; i < 15; ++i) {
-        manager.triggerSensorFailure(mock_timestamp + i);
+// Test getAlertHistory - Max Count Greater Than History Size
+TEST_F(AlertManagerTest, testGetAlertHistoryMaxCountGreaterThanHistorySize) {
+    // Given: Add multiple alerts
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+    alert_manager.triggerLowBattery(timestamp + 1, 10.0f);
+    alert_manager.triggerCommunicationError(timestamp + 2);
+
+    // When: Get history with max count larger than size
+    auto history = alert_manager.getAlertHistory(10);
+
+    // Then: Should return all history entries
+    EXPECT_EQ(history.size(), 3);
+}
+
+// Test getAlertHistory - Max Count Less Than History Size
+TEST_F(AlertManagerTest, testGetAlertHistoryMaxCountLessThanHistorySize) {
+    // Given: Add more alerts than max count
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+    alert_manager.triggerLowBattery(timestamp + 1, 10.0f);
+    alert_manager.triggerCommunicationError(timestamp + 2);
+
+    // When: Get history with smaller max count
+    auto history = alert_manager.getAlertHistory(2);
+
+    // Then: Should return only last 2 entries
+    EXPECT_EQ(history.size(), 2);
+    EXPECT_EQ(history[0].type, AlertType::LOW_BATTERY);
+    EXPECT_EQ(history[1].type, AlertType::COMMUNICATION_ERROR);
+}
+
+// Test getAlertHistory - Empty History
+TEST_F(AlertManagerTest, testGetAlertHistoryEmptyHistory) {
+    // Given: No history entries
+    // When: Get history
+    auto history = alert_manager.getAlertHistory(5);
+
+    // Then: Should return empty vector
+    EXPECT_EQ(history.size(), 0);
+}
+
+// Test getStatusDescription - No Active Alerts
+TEST_F(AlertManagerTest, testGetStatusDescriptionNoActiveAlerts) {
+    // Given: No active alerts
+    // When: Get status description
+    string description = alert_manager.getStatusDescription();
+
+    // Then: Should indicate system is normal
+    EXPECT_EQ(description, "系统状态：正常");
+}
+
+// Test getStatusDescription - With Active Alerts
+TEST_F(AlertManagerTest, testGetStatusDescriptionWithActiveAlerts) {
+    // Given: One active alert
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+
+    // When: Get status description
+    string description = alert_manager.getStatusDescription();
+
+    // Then: Should include alert details
+    EXPECT_NE(description.find("系统状态：有1个活动预警"), string::npos);
+    EXPECT_NE(description.find("传感器故障"), string::npos);
+}
+
+// Test clearHistory - Clear All History
+TEST_F(AlertManagerTest, testClearHistoryClearAllHistory) {
+    // Given: Add some history entries
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+    alert_manager.triggerLowBattery(timestamp + 1, 10.0f);
+
+    // When: Clear history
+    alert_manager.clearHistory();
+
+    // Then: History should be empty
+    EXPECT_EQ(alert_manager.getAlertHistory(10).size(), 0);
+}
+
+// Test addToHistory - Add Entry
+TEST_F(AlertManagerTest, testAddToHistoryAddEntry) {
+    // Given: Create a new alert
+    AlertInfo alert;
+    alert.type = AlertType::SENSOR_FAILURE;
+    alert.status = AlertStatus::ACTIVE;
+    alert.trigger_time = getMockTime();
+    alert.description = "Test alert";
+
+    // When: Add to history
+    alert_manager.addToHistory(alert);
+
+    // Then: History should contain the entry
+    auto history = alert_manager.getAlertHistory(1);
+    EXPECT_EQ(history.size(), 1);
+    EXPECT_EQ(history[0].type, AlertType::SENSOR_FAILURE);
+}
+
+// Test addToHistory - Limit History Size
+TEST_F(AlertManagerTest, testAddToHistoryLimitHistorySize) {
+    // Given: Add more entries than MAX_HISTORY
+    const int MAX_HISTORY = 5;
+    for (int i = 0; i < MAX_HISTORY + 1; ++i) {
+        AlertInfo alert;
+        alert.type = AlertType::SENSOR_FAILURE;
+        alert.status = AlertStatus::ACTIVE;
+        alert.trigger_time = getMockTime() + i;
+        alert.description = "Test alert";
+        alert_manager.addToHistory(alert);
     }
-    auto history = manager.getAlertHistory(10);
-    EXPECT_EQ(history.size(), 10);
+
+    // When: Check history size
+    auto history = alert_manager.getAlertHistory(10);
+
+    // Then: Only MAX_HISTORY entries should remain
+    EXPECT_EQ(history.size(), MAX_HISTORY);
 }
 
-TEST_F(AlertManagerTest, testGetStatusDescriptionNoAlerts) {
-    std::string desc = manager.getStatusDescription();
-    EXPECT_EQ(desc, "系统状态：正常");
+// Test triggerAlert - Normal Trigger
+TEST_F(AlertManagerTest, testTriggerAlertNormalTrigger) {
+    // Given: Valid parameters
+    uint32_t timestamp = getMockTime();
+    string description = "Test alert";
+
+    // When: Trigger alert
+    alert_manager.triggerAlert(AlertType::SENSOR_FAILURE, timestamp, description);
+
+    // Then: Alert should be active
+    EXPECT_TRUE(alert_manager.hasActiveAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 1);
+    EXPECT_EQ(active_alerts[0].type, AlertType::SENSOR_FAILURE);
+    EXPECT_EQ(active_alerts[0].description, description);
 }
 
-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 updateAlertStatus - Acknowledge Alert
+TEST_F(AlertManagerTest, testUpdateAlertStatusAcknowledgeAlert) {
+    // Given: An active alert
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+
+    // When: Update status to acknowledged
+    alert_manager.updateAlertStatus(AlertType::SENSOR_FAILURE, AlertStatus::ACKNOWLEDGED, timestamp + 1);
+
+    // Then: Alert should be acknowledged
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 0);
 }
 
-TEST_F(AlertManagerTest, testClearHistory) {
-    mock_timestamp = 1640995200;
-    manager.triggerSensorFailure(mock_timestamp);
-    manager.clearHistory();
-    EXPECT_TRUE(manager.getAlertHistory(10).empty());
+// Test updateAlertStatus - Resolve Alert
+TEST_F(AlertManagerTest, testUpdateAlertStatusResolveAlert) {
+    // Given: An active alert
+    uint32_t timestamp = getMockTime();
+    alert_manager.triggerSensorFailure(timestamp);
+
+    // When: Update status to resolved
+    alert_manager.updateAlertStatus(AlertType::SENSOR_FAILURE, AlertStatus::RESOLVED, timestamp + 1);
+
+    // Then: Alert should be resolved
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
+    EXPECT_FALSE(alert_manager.hasUnacknowledgedAlerts());
+    auto active_alerts = alert_manager.getActiveAlerts();
+    EXPECT_EQ(active_alerts.size(), 0);
 }
 
-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 updateAlertStatus - Update Non-Existing Alert
+TEST_F(AlertManagerTest, testUpdateAlertStatusUpdateNonExistingAlert) {
+    // Given: No existing alert
+    // When: Update non-existing alert
+    alert_manager.updateAlertStatus(AlertType::SENSOR_FAILURE, AlertStatus::ACTIVE, getMockTime());
+
+    // Then: No change
+    EXPECT_FALSE(alert_manager.hasActiveAlerts());
 }
 
-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 getAlertTypeName - Valid Type
+TEST_F(AlertManagerTest, testGetAlertTypeNameValidType) {
+    // Given: Valid alert type
+    AlertType type = AlertType::ALTITUDE_UPPER;
+
+    // When: Get type name
+    string name = alert_manager.getAlertTypeName(type);
+
+    // Then: Should return correct name
+    EXPECT_EQ(name, "高度上限预警");
 }
 
-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 getAlertTypeName - Unknown Type
+TEST_F(AlertManagerTest, testGetAlertTypeNameUnknownType) {
+    // Given: Unknown alert type
+    AlertType type = static_cast<AlertType>(999);
+
+    // When: Get type name
+    string name = alert_manager.getAlertTypeName(type);
+
+    // Then: Should return "未知预警"
+    EXPECT_EQ(name, "未知预警");
 }
 
-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 getAlertStatusName - Valid Status
+TEST_F(AlertManagerTest, testGetAlertStatusNameValidStatus) {
+    // Given: Valid alert status
+    AlertStatus status = AlertStatus::ACTIVE;
+
+    // When: Get status name
+    string name = alert_manager.getAlertStatusName(status);
+
+    // Then: Should return correct name
+    EXPECT_EQ(name, "预警中");
 }
 
-TEST_F(AlertManagerTest, testGetAlertTypeName) {
-    EXPECT_EQ(manager.getAlertTypeName(AlertType::ALTITUDE_UPPER), "高度上限预警");
-    EXPECT_EQ(manager.getAlertTypeName(AlertType::NONE), "无");
-    EXPECT_EQ(manager.getAlertTypeName(static_cast<AlertType>(99)), "未知预警");
-}
+// Test getAlertStatusName - Unknown Status
+TEST_F(AlertManagerTest, testGetAlertStatusNameUnknownStatus) {
+    // Given: Unknown alert status
+    AlertStatus status = static_cast<AlertStatus>(999);
 
-TEST_F(AlertManagerTest, testGetAlertStatusName) {
-    EXPECT_EQ(manager.getAlertStatusName(AlertStatus::ACTIVE), "预警中");
-    EXPECT_EQ(manager.getAlertStatusName(AlertStatus::INACTIVE), "未激活");
-    EXPECT_EQ(manager.getAlertStatusName(static_cast<AlertStatus>(99)), "未知状态");
-}
+    // When: Get status name
+    string name = alert_manager.getAlertStatusName(status);
 
-TEST_F(AlertManagerTest, testDestructorDoesNotCrash) {
-    // Verify destructor doesn't throw or crash
-    EXPECT_NO_FATAL_FAILURE({
-        AlertManager* mgr = new AlertManager();
-        delete mgr;
-    });
-}
\ No newline at end of file
+    // Then: Should return "未知状态"
+    EXPECT_EQ(name, "未知状态");
+}