Fix unit test failing

This commit is contained in:
Daniel Öster 2025-08-30 00:29:22 +03:00
parent aa9a4d429e
commit 5df7071999

View file

@ -159,14 +159,12 @@ void reset_all_events() {
}
events.level = EVENT_LEVEL_INFO;
update_bms_status();
logging.println("All events have been cleared.");
}
void set_event_MQTTpublished(EVENTS_ENUM_TYPE event) {
void set_event_MQTTpublished(EVENTS_ENUM_TYPE event) {
events.entries[event].MQTTpublished = true;
}
}
String get_event_message_string(EVENTS_ENUM_TYPE event) {
String get_event_message_string(EVENTS_ENUM_TYPE event) {
switch (event) {
case EVENT_CANMCP2517FD_INIT_FAILURE:
return "CAN-FD initialization failed. Check hardware or bitrate settings";
@ -380,32 +378,32 @@ String get_event_message_string(EVENTS_ENUM_TYPE event) {
default:
return "";
}
}
}
const char* get_event_enum_string(EVENTS_ENUM_TYPE event) {
const char* get_event_enum_string(EVENTS_ENUM_TYPE event) {
// Return the event name but skip "EVENT_" that should always be first
return EVENTS_ENUM_TYPE_STRING[event] + 6;
}
}
const char* get_event_level_string(EVENTS_ENUM_TYPE event) {
const char* get_event_level_string(EVENTS_ENUM_TYPE event) {
// Return the event level but skip "EVENT_LEVEL_" that should always be first
return EVENTS_LEVEL_TYPE_STRING[events.entries[event].level] + 12;
}
}
const char* get_event_level_string(EVENTS_LEVEL_TYPE event_level) {
const char* get_event_level_string(EVENTS_LEVEL_TYPE event_level) {
// Return the event level but skip "EVENT_LEVEL_TYPE_" that should always be first
return EVENTS_LEVEL_TYPE_STRING[event_level] + 17;
}
}
const EVENTS_STRUCT_TYPE* get_event_pointer(EVENTS_ENUM_TYPE event) {
const EVENTS_STRUCT_TYPE* get_event_pointer(EVENTS_ENUM_TYPE event) {
return &events.entries[event];
}
}
EVENTS_LEVEL_TYPE get_event_level(void) {
EVENTS_LEVEL_TYPE get_event_level(void) {
return events.level;
}
}
EMULATOR_STATUS get_emulator_status() {
EMULATOR_STATUS get_emulator_status() {
switch (events.level) {
case EVENT_LEVEL_DEBUG:
case EVENT_LEVEL_INFO:
@ -419,16 +417,16 @@ EMULATOR_STATUS get_emulator_status() {
default:
return EMULATOR_STATUS::STATUS_OK;
}
}
}
const char* get_emulator_status_string(EMULATOR_STATUS status) {
const char* get_emulator_status_string(EMULATOR_STATUS status) {
// Return the status string but skip "STATUS_" that should always be first
return EMULATOR_STATUS_STRING[status] + 7;
}
}
/* Local functions */
/* Local functions */
static void set_event(EVENTS_ENUM_TYPE event, uint8_t data, bool latched) {
static void set_event(EVENTS_ENUM_TYPE event, uint8_t data, bool latched) {
// Just some defensive stuff if someone sets an unknown event
if (event >= EVENT_NOF_EVENTS) {
event = EVENT_UNKNOWN_EVENT_SET;
@ -453,9 +451,9 @@ static void set_event(EVENTS_ENUM_TYPE event, uint8_t data, bool latched) {
events.level = (EVENTS_LEVEL_TYPE)max(events.level, events.entries[event].level);
update_bms_status();
}
}
static void update_bms_status(void) {
static void update_bms_status(void) {
switch (events.level) {
case EVENT_LEVEL_INFO:
case EVENT_LEVEL_WARNING:
@ -471,19 +469,19 @@ static void update_bms_status(void) {
default:
break;
}
}
}
// Function to compare events by timestamp descending
bool compareEventsByTimestampDesc(const EventData& a, const EventData& b) {
// Function to compare events by timestamp descending
bool compareEventsByTimestampDesc(const EventData& a, const EventData& b) {
return a.event_pointer->timestamp > b.event_pointer->timestamp;
}
}
// Function to compare events by timestamp ascending
bool compareEventsByTimestampAsc(const EventData& a, const EventData& b) {
// Function to compare events by timestamp ascending
bool compareEventsByTimestampAsc(const EventData& a, const EventData& b) {
return a.event_pointer->timestamp < b.event_pointer->timestamp;
}
}
static void update_event_level(void) {
static void update_event_level(void) {
EVENTS_LEVEL_TYPE temporary_level = EVENT_LEVEL_INFO;
for (uint8_t i = 0u; i < EVENT_NOF_EVENTS; i++) {
if ((events.entries[i].state == EVENT_STATE_ACTIVE) || (events.entries[i].state == EVENT_STATE_ACTIVE_LATCHED)) {
@ -491,4 +489,4 @@ static void update_event_level(void) {
}
}
events.level = temporary_level;
}
}