AI 自动生成测试用例

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)
+gtest_discover_tests(test_CppGenerate)

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 { 
+// Mock function to replace std::time()
-public:
+uint32_t getMockTime() {
-    static uint32_t getTimestamp() { return mock_timestamp; }
+    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;
 }
 
-// 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) {
+// Test checkAltitudeAlert - Normal Input
-    EXPECT_FLOAT_EQ(manager.getAltitudeThresholds().first, 100.0f);
+TEST_F(AlertManagerTest, testCheckAltitudeAlertNormalInput) {
-    EXPECT_FLOAT_EQ(manager.getAltitudeThresholds().second, -50.0f);
+    // Given: Altitude within normal range
-    EXPECT_TRUE(manager.getActiveAlerts().empty());
+    float altitude = 80.0f;
-    EXPECT_TRUE(manager.getAlertHistory(10).empty());
+    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) {
+// Test checkAltitudeAlert - Upper Threshold Exceeded
-    // No direct output assertion, but verify initialization logic
+TEST_F(AlertManagerTest, testCheckAltitudeAlertUpperThresholdExceeded) {
-    // This is tested implicitly via constructor
+    // Given: Altitude exceeds upper threshold
-    EXPECT_NO_FATAL_FAILURE(manager.initialize());
+    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) {
+// Test checkAltitudeAlert - Lower Threshold Exceeded
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testCheckAltitudeAlertLowerThresholdExceeded) {
-    manager.checkAltitudeAlert(105.0f, mock_timestamp);
+    // Given: Altitude below lower threshold
-    auto alerts = manager.getActiveAlerts();
+    float altitude = -51.0f; // Below default -50.0f
-    EXPECT_EQ(alerts.size(), 1);
+    uint32_t timestamp = getMockTime();
-    EXPECT_EQ(alerts[0].type, AlertType::ALTITUDE_UPPER);
+
-    EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE);
+    // When: Check altitude alert
-    EXPECT_EQ(alerts[0].trigger_time, mock_timestamp);
+    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) {
+// Test checkAltitudeAlert - Recovery from Upper Threshold
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testCheckAltitudeAlertRecoveryFromUpperThreshold) {
-    manager.checkAltitudeAlert(-55.0f, mock_timestamp);
+    // Given: Initially exceed upper threshold
-    auto alerts = manager.getActiveAlerts();
+    float altitude = 101.0f;
-    EXPECT_EQ(alerts.size(), 1);
+    uint32_t timestamp = getMockTime();
-    EXPECT_EQ(alerts[0].type, AlertType::ALTITUDE_LOWER);
+    alert_manager.checkAltitudeAlert(altitude, timestamp);
-    EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE);
+
+    // 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) {
+// Test checkAltitudeAlert - Recovery from Lower Threshold
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testCheckAltitudeAlertRecoveryFromLowerThreshold) {
-    manager.checkAltitudeAlert(105.0f, mock_timestamp); // Trigger upper
+    // Given: Initially below lower threshold
-    mock_timestamp = 1640995201;
+    float altitude = -51.0f;
-    manager.checkAltitudeAlert(90.0f, mock_timestamp); // Reset
+    uint32_t timestamp = getMockTime();
-    auto alerts = manager.getActiveAlerts();
+    alert_manager.checkAltitudeAlert(altitude, timestamp);
-    EXPECT_TRUE(alerts.empty());
+
+    // 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) {
+// Test setAltitudeThresholds - Valid Thresholds
-    manager.setAltitudeThresholds(200.0f, -100.0f);
+TEST_F(AlertManagerTest, testSetAltitudeThresholdsValidThresholds) {
-    auto thresholds = manager.getAltitudeThresholds();
+    // Given: Valid upper and lower thresholds
-    EXPECT_FLOAT_EQ(thresholds.first, 200.0f);
+    float upper = 150.0f;
-    EXPECT_FLOAT_EQ(thresholds.second, -100.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) {
+// Test setAltitudeThresholds - Invalid Thresholds (upper <= lower)
-    EXPECT_NO_FATAL_FAILURE(manager.setAltitudeThresholds(50.0f, 100.0f));
+TEST_F(AlertManagerTest, testSetAltitudeThresholdsInvalidThresholds) {
-    // Should print error, but no crash
+    // Given: Invalid thresholds
-    EXPECT_FLOAT_EQ(manager.getAltitudeThresholds().first, 100.0f);
+    float upper = 50.0f;
-    EXPECT_FLOAT_EQ(manager.getAltitudeThresholds().second, -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) {
+// Test triggerSensorFailure - Normal Trigger
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testTriggerSensorFailureNormalTrigger) {
-    manager.triggerSensorFailure(mock_timestamp);
+    // Given: Timestamp
-    auto alerts = manager.getActiveAlerts();
+    uint32_t timestamp = getMockTime();
-    EXPECT_EQ(alerts.size(), 1);
+
-    EXPECT_EQ(alerts[0].type, AlertType::SENSOR_FAILURE);
+    // When: Trigger sensor failure
-    EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE);
+    alert_manager.triggerSensorFailure(timestamp);
-    EXPECT_EQ(alerts[0].trigger_time, mock_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) {
+// Test triggerLowBattery - Normal Trigger
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testTriggerLowBatteryNormalTrigger) {
-    manager.triggerLowBattery(mock_timestamp, 15.0f);
+    // Given: Timestamp and battery level
-    auto alerts = manager.getActiveAlerts();
+    uint32_t timestamp = getMockTime();
-    EXPECT_EQ(alerts.size(), 1);
+    float battery_level = 15.0f;
-    EXPECT_EQ(alerts[0].type, AlertType::LOW_BATTERY);
+
-    EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE);
+    // When: Trigger low battery
-    EXPECT_NE(alerts[0].description.find("15.0%"), std::string::npos);
+    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) {
+// Test triggerCommunicationError - Normal Trigger
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testTriggerCommunicationErrorNormalTrigger) {
-    manager.triggerCommunicationError(mock_timestamp);
+    // Given: Timestamp
-    auto alerts = manager.getActiveAlerts();
+    uint32_t timestamp = getMockTime();
-    EXPECT_EQ(alerts.size(), 1);
+
-    EXPECT_EQ(alerts[0].type, AlertType::COMMUNICATION_ERROR);
+    // When: Trigger communication error
-    EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE);
+    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) {
+// Test acknowledgeAllAlerts - Acknowledge All Active Alerts
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testAcknowledgeAllAlertsAcknowledgeAllActiveAlerts) {
-    manager.triggerSensorFailure(mock_timestamp);
+    // Given: Multiple active alerts
-    manager.triggerLowBattery(mock_timestamp + 1, 10.0f);
+    uint32_t timestamp = getMockTime();
-    mock_timestamp = 1640995202;
+    alert_manager.triggerSensorFailure(timestamp);
-    manager.acknowledgeAllAlerts(mock_timestamp);
+    alert_manager.triggerLowBattery(timestamp + 1, 10.0f);
-    auto alerts = manager.getActiveAlerts();
+
-    EXPECT_TRUE(alerts.empty());
+    // 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) {
+// Test getActiveAlerts - With Active Alerts
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testGetActiveAlertsWithActiveAlerts) {
-    manager.triggerSensorFailure(mock_timestamp);
+    // Given: One active alert
-    mock_timestamp = 1640995201;
+    uint32_t timestamp = getMockTime();
-    manager.acknowledgeAlert(AlertType::SENSOR_FAILURE, mock_timestamp);
+    alert_manager.triggerSensorFailure(timestamp);
-    auto alerts = manager.getActiveAlerts();
-    EXPECT_TRUE(alerts.empty());
-}
 
-TEST_F(AlertManagerTest, testResetAlert) {
+    // When: Get active alerts
-    mock_timestamp = 1640995200;
+    auto alerts = alert_manager.getActiveAlerts();
-    manager.triggerSensorFailure(mock_timestamp);
-    manager.resetAlert(AlertType::SENSOR_FAILURE);
-    auto alerts = manager.getActiveAlerts();
-    EXPECT_TRUE(alerts.empty());
-}
 
-TEST_F(AlertManagerTest, testResetAllAlerts) {
+    // Then: Should return one alert
-    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();
     EXPECT_EQ(alerts.size(), 1);
     EXPECT_EQ(alerts[0].type, AlertType::SENSOR_FAILURE);
+    EXPECT_EQ(alerts[0].status, AlertStatus::ACTIVE);
 }
 
-TEST_F(AlertManagerTest, testGetAlertHistoryMaxCount) {
+// Test getAlertHistory - Max Count Greater Than History Size
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testGetAlertHistoryMaxCountGreaterThanHistorySize) {
-    for (int i = 0; i < 15; ++i) {
+    // Given: Add multiple alerts
-        manager.triggerSensorFailure(mock_timestamp + i);
+    uint32_t timestamp = getMockTime();
-    }
+    alert_manager.triggerSensorFailure(timestamp);
-    auto history = manager.getAlertHistory(10);
+    alert_manager.triggerLowBattery(timestamp + 1, 10.0f);
-    EXPECT_EQ(history.size(), 10);
+    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_F(AlertManagerTest, testGetStatusDescriptionNoAlerts) {
+// Test getAlertHistory - Max Count Less Than History Size
-    std::string desc = manager.getStatusDescription();
+TEST_F(AlertManagerTest, testGetAlertHistoryMaxCountLessThanHistorySize) {
-    EXPECT_EQ(desc, "系统状态：正常");
+    // 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_F(AlertManagerTest, testGetStatusDescriptionWithAlerts) {
+// Test getAlertHistory - Empty History
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testGetAlertHistoryEmptyHistory) {
-    manager.triggerSensorFailure(mock_timestamp);
+    // Given: No history entries
-    std::string desc = manager.getStatusDescription();
+    // When: Get history
-    EXPECT_NE(desc.find("有1个活动预警"), std::string::npos);
+    auto history = alert_manager.getAlertHistory(5);
-    EXPECT_NE(desc.find("传感器故障"), std::string::npos);
+
+    // Then: Should return empty vector
+    EXPECT_EQ(history.size(), 0);
 }
 
-TEST_F(AlertManagerTest, testClearHistory) {
+// Test getStatusDescription - No Active Alerts
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testGetStatusDescriptionNoActiveAlerts) {
-    manager.triggerSensorFailure(mock_timestamp);
+    // Given: No active alerts
-    manager.clearHistory();
+    // When: Get status description
-    EXPECT_TRUE(manager.getAlertHistory(10).empty());
+    string description = alert_manager.getStatusDescription();
+
+    // Then: Should indicate system is normal
+    EXPECT_EQ(description, "系统状态：正常");
 }
 
-TEST_F(AlertManagerTest, testAddToHistoryLimit) {
+// Test getStatusDescription - With Active Alerts
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testGetStatusDescriptionWithActiveAlerts) {
-    for (int i = 0; i < 100; ++i) {
+    // Given: One active alert
-        manager.triggerSensorFailure(mock_timestamp + i);
+    uint32_t timestamp = getMockTime();
-    }
+    alert_manager.triggerSensorFailure(timestamp);
-    auto history = manager.getAlertHistory(10);
+
-    EXPECT_EQ(history.size(), 10);
+    // 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_F(AlertManagerTest, testTriggerAlert) {
+// Test clearHistory - Clear All History
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testClearHistoryClearAllHistory) {
-    manager.triggerAlert(AlertType::SENSOR_FAILURE, mock_timestamp, "Test alert");
+    // Given: Add some history entries
-    auto alerts = manager.getActiveAlerts();
+    uint32_t timestamp = getMockTime();
-    EXPECT_EQ(alerts.size(), 1);
+    alert_manager.triggerSensorFailure(timestamp);
-    EXPECT_EQ(alerts[0].description, "Test alert");
+    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_F(AlertManagerTest, testUpdateAlertStatusAcknowledged) {
+// Test addToHistory - Add Entry
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testAddToHistoryAddEntry) {
-    manager.triggerSensorFailure(mock_timestamp);
+    // Given: Create a new alert
-    mock_timestamp = 1640995201;
+    AlertInfo alert;
-    manager.updateAlertStatus(AlertType::SENSOR_FAILURE, AlertStatus::ACKNOWLEDGED, mock_timestamp);
+    alert.type = AlertType::SENSOR_FAILURE;
-    auto alerts = manager.getActiveAlerts();
+    alert.status = AlertStatus::ACTIVE;
-    EXPECT_TRUE(alerts.empty());
+    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_F(AlertManagerTest, testUpdateAlertStatusResolved) {
+// Test addToHistory - Limit History Size
-    mock_timestamp = 1640995200;
+TEST_F(AlertManagerTest, testAddToHistoryLimitHistorySize) {
-    manager.triggerSensorFailure(mock_timestamp);
+    // Given: Add more entries than MAX_HISTORY
-    mock_timestamp = 1640995201;
+    const int MAX_HISTORY = 5;
-    manager.updateAlertStatus(AlertType::SENSOR_FAILURE, AlertStatus::RESOLVED, mock_timestamp);
+    for (int i = 0; i < MAX_HISTORY + 1; ++i) {
-    auto alerts = manager.getActiveAlerts();
+        AlertInfo alert;
-    EXPECT_TRUE(alerts.empty());
+        alert.type = AlertType::SENSOR_FAILURE;
+        alert.status = AlertStatus::ACTIVE;
+        alert.trigger_time = getMockTime() + i;
+        alert.description = "Test alert";
+        alert_manager.addToHistory(alert);
     }
 
-TEST_F(AlertManagerTest, testGetAlertTypeName) {
+    // When: Check history size
-    EXPECT_EQ(manager.getAlertTypeName(AlertType::ALTITUDE_UPPER), "高度上限预警");
+    auto history = alert_manager.getAlertHistory(10);
-    EXPECT_EQ(manager.getAlertTypeName(AlertType::NONE), "无");
+
-    EXPECT_EQ(manager.getAlertTypeName(static_cast<AlertType>(99)), "未知预警");
+    // Then: Only MAX_HISTORY entries should remain
+    EXPECT_EQ(history.size(), MAX_HISTORY);
 }
 
-TEST_F(AlertManagerTest, testGetAlertStatusName) {
+// Test triggerAlert - Normal Trigger
-    EXPECT_EQ(manager.getAlertStatusName(AlertStatus::ACTIVE), "预警中");
+TEST_F(AlertManagerTest, testTriggerAlertNormalTrigger) {
-    EXPECT_EQ(manager.getAlertStatusName(AlertStatus::INACTIVE), "未激活");
+    // Given: Valid parameters
-    EXPECT_EQ(manager.getAlertStatusName(static_cast<AlertStatus>(99)), "未知状态");
+    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, testDestructorDoesNotCrash) {
+// Test updateAlertStatus - Acknowledge Alert
-    // Verify destructor doesn't throw or crash
+TEST_F(AlertManagerTest, testUpdateAlertStatusAcknowledgeAlert) {
-    EXPECT_NO_FATAL_FAILURE({
+    // Given: An active alert
-        AlertManager* mgr = new AlertManager();
+    uint32_t timestamp = getMockTime();
-        delete mgr;
+    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 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 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 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 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 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 getAlertStatusName - Unknown Status
+TEST_F(AlertManagerTest, testGetAlertStatusNameUnknownStatus) {
+    // Given: Unknown alert status
+    AlertStatus status = static_cast<AlertStatus>(999);
+
+    // When: Get status name
+    string name = alert_manager.getAlertStatusName(status);
+
+    // Then: Should return "未知状态"
+    EXPECT_EQ(name, "未知状态");
 }