diff --git a/docs/cFS_IdentifierNamingConvention.md b/docs/cFS_IdentifierNamingConvention.md index 79ca8c6f6..4d3ecc92c 100644 --- a/docs/cFS_IdentifierNamingConvention.md +++ b/docs/cFS_IdentifierNamingConvention.md @@ -114,6 +114,24 @@ Some general rules/recommendations to follow for cFS identifiers: __NOTE__: a sufficient number of terms must always be used in order to ensure uniqueness and avoid ambiguity. +8. Event IDs in cFE (and across the other cFS components/apps) should be capitalized and use + a consistent abbreviated naming pattern in the following order: + <APPLICATION\_NAME>\_<EVENT>\_<EVENT\_TYPE>\_<EID> +
+ For example: + | Event ID | Application | Command/Event Description | Event Type | Event ID Suffix | + | :---: | :---: | :---: | :---: | :---: | + | CFE_EVS_NOOP_INF_EID | CFE Event Service | NOOP Command | Informational | Event ID | + | HS_CR_PIPE_ERR_EID | Health & Safety | Create Pipe Error | Error | Event ID | + + Common events should be named consistently across applications for ease of recognition. +
+ Common command/event descriptions include: + \_MID\_ERR\_EID: Invalid Message ID Received + \_NOOP\_INF\_EID: No-op Command Success + \_INIT\_INF\_EID: Applicaiton Initialization Success + \_RESET\_INF\_EID: Reset Command Counters Command Success + ## Applicability to Actions / Commands diff --git a/docs/src/cfe_es.dox b/docs/src/cfe_es.dox index 2c45114af..339a365e8 100644 --- a/docs/src/cfe_es.dox +++ b/docs/src/cfe_es.dox @@ -728,7 +728,7 @@ message length embedded within the header (from `CFE_MSG_GetSize()`) matches the expected length of that message, based on the size of the C structure defining that command. If there is any discrepancy between the expected and actual message size, ES will generate - the #CFE_ES_LEN_ERR_EID event, increment the command error counter (\ES_CMDEC), and the + the #CFE_ES_CMD_LEN_ERR_EID event, increment the command error counter (\ES_CMDEC), and the command will _not_ be accepted for processing. The following is a list of commands that are processed by the cFE Executive Services Task. diff --git a/docs/src/cfe_evs.dox b/docs/src/cfe_evs.dox index dbdce7248..599d24b00 100644 --- a/docs/src/cfe_evs.dox +++ b/docs/src/cfe_evs.dox @@ -492,7 +492,7 @@ ** message length embedded within the header (from `CFE_MSG_GetSize()`) matches the expected ** length of that message, based on the size of the C structure defining that command. ** If there is any discrepancy between the expected and actual message size, EVS will generate -** the #CFE_EVS_LEN_ERR_EID event, increment the command error counter (\EVS_CMDEC), and the +** the #CFE_EVS_CMD_LEN_ERR_EID event, increment the command error counter (\EVS_CMDEC), and the ** command will _not_ be accepted for processing. ** ** The following is a list of commands that are processed by the cFE Event Services Task. diff --git a/docs/src/cfe_sb.dox b/docs/src/cfe_sb.dox index f97aeb2f7..eac7ecb64 100644 --- a/docs/src/cfe_sb.dox +++ b/docs/src/cfe_sb.dox @@ -530,7 +530,7 @@ ** message length embedded within the header (from `CFE_MSG_GetSize()`) matches the expected ** length of that message, based on the size of the C structure defining that command. ** If there is any discrepancy between the expected and actual message size, SB will generate -** the #CFE_SB_LEN_ERR_EID event, increment the command error counter (\SB_CMDEC), and the +** the #CFE_SB_CMD_LEN_ERR_EID event, increment the command error counter (\SB_CMDEC), and the ** command will _not_ be accepted for processing. ** ** The following is a list of commands that are processed by the cFE Software Bus Task. diff --git a/docs/src/cfe_tbl.dox b/docs/src/cfe_tbl.dox index c8a9d86cd..54e9608a4 100644 --- a/docs/src/cfe_tbl.dox +++ b/docs/src/cfe_tbl.dox @@ -377,7 +377,7 @@ ** message length embedded within the header (from `CFE_MSG_GetSize()`) matches the expected ** length of that message, based on the size of the C structure defining that command. ** If there is any discrepancy between the expected and actual message size, TBL will generate -** the #CFE_TBL_LEN_ERR_EID event, increment the command error counter (\TBL_CMDEC), and the +** the #CFE_TBL_CMD_LEN_ERR_EID event, increment the command error counter (\TBL_CMDEC), and the ** command will _not_ be accepted for processing. ** ** The following is a list of commands that are processed by the cFE Table Services Task. diff --git a/docs/src/cfe_time.dox b/docs/src/cfe_time.dox index 1f16a05cb..ce46ad997 100644 --- a/docs/src/cfe_time.dox +++ b/docs/src/cfe_time.dox @@ -706,7 +706,7 @@ ** message length embedded within the header (from `CFE_MSG_GetSize()`) matches the expected ** length of that message, based on the size of the C structure defining that command. ** If there is any discrepancy between the expected and actual message size, TIME will generate -** the #CFE_TIME_LEN_ERR_EID event, increment the command error counter (\TIME_CMDEC), and the +** the #CFE_TIME_CMD_LEN_ERR_EID event, increment the command error counter (\TIME_CMDEC), and the ** command will _not_ be accepted for processing. ** ** The following is a list of commands that are processed by the cFE Time Services Task. diff --git a/modules/es/eds/cfe_es.xml b/modules/es/eds/cfe_es.xml index 8548c4f3c..0cf41e95a 100644 --- a/modules/es/eds/cfe_es.xml +++ b/modules/es/eds/cfe_es.xml @@ -822,7 +822,7 @@ Evidence of failure may be found in the following telemetry: - \b \c \ES_CMDEC - command error counter will increment - - the #CFE_ES_LEN_ERR_EID error event message will be generated + - the #CFE_ES_CMD_LEN_ERR_EID error event message will be generated \par Criticality None @@ -863,7 +863,7 @@ Evidence of failure may be found in the following telemetry: - \b \c \ES_CMDEC - command error counter will increment - - the #CFE_ES_LEN_ERR_EID error event message will be generated + - the #CFE_ES_CMD_LEN_ERR_EID error event message will be generated \par Criticality diff --git a/modules/es/fsw/inc/cfe_es_events.h b/modules/es/fsw/inc/cfe_es_events.h index 91958291d..10b10692a 100644 --- a/modules/es/fsw/inc/cfe_es_events.h +++ b/modules/es/fsw/inc/cfe_es_events.h @@ -269,7 +269,7 @@ * * Invalid command code for message ID #CFE_ES_CMD_MID received on the ES message pipe. */ -#define CFE_ES_CC1_ERR_EID 22 +#define CFE_ES_CC_ERR_EID 22 /** * \brief ES Invalid Command Length Event ID @@ -280,7 +280,7 @@ * * Invalid length for the command code in message ID #CFE_ES_CMD_MID received on the ES message pipe. */ -#define CFE_ES_LEN_ERR_EID 23 +#define CFE_ES_CMD_LEN_ERR_EID 23 /** * \brief ES Restart Command Invalid Restart Type Event ID diff --git a/modules/es/fsw/inc/cfe_es_msg.h b/modules/es/fsw/inc/cfe_es_msg.h index b320df61e..19c5f4498 100644 --- a/modules/es/fsw/inc/cfe_es_msg.h +++ b/modules/es/fsw/inc/cfe_es_msg.h @@ -73,7 +73,7 @@ ** ** Evidence of failure may be found in the following telemetry: ** - \b \c \ES_CMDEC - command error counter will increment -** - the #CFE_ES_LEN_ERR_EID error event message will be generated +** - the #CFE_ES_CMD_LEN_ERR_EID error event message will be generated ** ** \par Criticality ** None diff --git a/modules/es/fsw/src/cfe_es_dispatch.c b/modules/es/fsw/src/cfe_es_dispatch.c index 2d0df7172..fb6828822 100644 --- a/modules/es/fsw/src/cfe_es_dispatch.c +++ b/modules/es/fsw/src/cfe_es_dispatch.c @@ -58,7 +58,7 @@ bool CFE_ES_VerifyCmdLength(const CFE_MSG_Message_t *MsgPtr, size_t ExpectedLeng CFE_MSG_GetMsgId(MsgPtr, &MsgId); CFE_MSG_GetFcnCode(MsgPtr, &FcnCode); - CFE_EVS_SendEvent(CFE_ES_LEN_ERR_EID, CFE_EVS_EventType_ERROR, + CFE_EVS_SendEvent(CFE_ES_CMD_LEN_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid msg length: ID = 0x%X, CC = %u, Len = %u, Expected = %u", (unsigned int)CFE_SB_MsgIdToValue(MsgId), (unsigned int)FcnCode, (unsigned int)ActualLength, (unsigned int)ExpectedLength); @@ -267,7 +267,7 @@ void CFE_ES_TaskPipe(const CFE_SB_Buffer_t *SBBufPtr) break; default: - CFE_EVS_SendEvent(CFE_ES_CC1_ERR_EID, CFE_EVS_EventType_ERROR, + CFE_EVS_SendEvent(CFE_ES_CC_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid ground command code: ID = 0x%X, CC = %d", (unsigned int)CFE_SB_MsgIdToValue(MessageID), (int)CommandCode); CFE_ES_Global.TaskData.CommandErrorCounter++; diff --git a/modules/es/ut-coverage/es_UT.c b/modules/es/ut-coverage/es_UT.c index 68ae5b4e4..bbba44b5d 100644 --- a/modules/es/ut-coverage/es_UT.c +++ b/modules/es/ut-coverage/es_UT.c @@ -3217,7 +3217,7 @@ void TestTask(void) */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_CLEAR_ER_LOG_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test resetting and setting the max for the processor reset count */ ES_ResetUnitTest(); @@ -3332,22 +3332,22 @@ void TestTask(void) /* Test the command pipe message process with an invalid command */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, sizeof(CmdBuf.NoopCmd), UT_TPID_CFE_ES_CMD_INVALID_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_CC1_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CC_ERR_EID); /* Test sending a no-op command with an invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_NOOP_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a reset counters command with an invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_RESET_COUNTERS_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a cFE restart command with an invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_RESTART_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test cFE restart with a power on reset */ ES_ResetUnitTest(); @@ -3361,7 +3361,7 @@ void TestTask(void) */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_START_APP_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test start application command with a processor restart on application * exception @@ -3386,63 +3386,63 @@ void TestTask(void) */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_STOP_APP_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a restart application command with an invalid command * length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_RESTART_APP_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a reload application command with an invalid command * length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_RELOAD_APP_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a write request for a single application with an * invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_QUERY_ONE_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a write request for all applications with an * invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_QUERY_ALL_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a write request for all tasks with an * invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_QUERY_ALL_TASKS_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a request to clear the system log with an * invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_CLEAR_SYSLOG_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a request to overwrite the system log with an * invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_OVER_WRITE_SYSLOG_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a request to write the system log with an * invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_WRITE_SYSLOG_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test successful overwriting of the system log using overwrite mode */ ES_ResetUnitTest(); @@ -3457,35 +3457,35 @@ void TestTask(void) */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_WRITE_ER_LOG_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a request to reset the processor reset count with an * invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_RESET_PR_COUNT_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a request to set the maximum processor reset count with * an invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_SET_MAX_PR_COUNT_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a request to delete the CDS with an invalid command * length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_DELETE_CDS_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test sending a telemetry pool statistics retrieval command with an * invalid command length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_SEND_MEM_POOL_STATS_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test successful dump of CDS to file using a specified dump file name */ ES_ResetUnitTest(); @@ -3502,7 +3502,7 @@ void TestTask(void) /* Dump CDS command with invalid length */ ES_ResetUnitTest(); UT_CallTaskPipe(CFE_ES_TaskPipe, &CmdBuf.Msg, 0, UT_TPID_CFE_ES_CMD_DUMP_CDS_REGISTRY_CC); - CFE_UtAssert_EVENTSENT(CFE_ES_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_ES_CMD_LEN_ERR_EID); /* Test error when sending Build Info event */ ES_ResetUnitTest(); diff --git a/modules/evs/eds/cfe_evs.xml b/modules/evs/eds/cfe_evs.xml index 5a3146555..41c45f36e 100644 --- a/modules/evs/eds/cfe_evs.xml +++ b/modules/evs/eds/cfe_evs.xml @@ -344,7 +344,7 @@ \par Command Verification Successful execution of this command may be verified with the following telemetry: - \b \c \EVS_CMDPC - command execution counter will increment - - The #CFE_EVS_NOOP_EID informational event message will be generated + - The #CFE_EVS_NOOP_INF_EID informational event message will be generated \par Error Conditions There are no error conditions for this command. If the Event @@ -379,7 +379,7 @@ the following telemetry: - \b \c \EVS_CMDPC - command execution counter will increment - - The #CFE_EVS_RSTCNT_EID debug event message will be + - The #CFE_EVS_RESET_INF_EID debug event message will be generated \par Error Conditions diff --git a/modules/evs/fsw/inc/cfe_evs_events.h b/modules/evs/fsw/inc/cfe_evs_events.h index 72003ed3e..ecd46c3c4 100644 --- a/modules/evs/fsw/inc/cfe_evs_events.h +++ b/modules/evs/fsw/inc/cfe_evs_events.h @@ -39,7 +39,7 @@ * * \link #CFE_EVS_NOOP_CC EVS NO-OP command \endlink success. */ -#define CFE_EVS_NOOP_EID 0 +#define CFE_EVS_NOOP_INF_EID 0 /** * \brief EVS Initialization Event ID @@ -50,7 +50,7 @@ * * Event Services Task initialization complete. */ -#define CFE_EVS_STARTUP_EID 1 +#define CFE_EVS_INIT_INF_EID 1 /** * \brief EVS Write Event Log Command File Write Entry Failed Event ID @@ -85,7 +85,7 @@ * * Invalid message ID received on the EVS message pipe. */ -#define CFE_EVS_ERR_MSGID_EID 5 +#define CFE_EVS_MID_ERR_EID 5 /** * \brief EVS Command Event Not Registered For Filtering Event ID @@ -188,7 +188,7 @@ * * Invalid command code for message ID #CFE_EVS_CMD_MID received on the EVS message pipe. */ -#define CFE_EVS_ERR_CC_EID 15 +#define CFE_EVS_CC_ERR_EID 15 /** * \brief EVS Reset Counters Command Success Event ID @@ -199,7 +199,7 @@ * * \link #CFE_EVS_RESET_COUNTERS_CC EVS Reset Counters Command \endlink success. */ -#define CFE_EVS_RSTCNT_EID 16 +#define CFE_EVS_RESET_INF_EID 16 /** * \brief EVS Set Filter Command Success Event ID @@ -465,7 +465,7 @@ * * Invalid length for the command code in message ID #CFE_EVS_CMD_MID received on the EVS message pipe. */ -#define CFE_EVS_LEN_ERR_EID 43 +#define CFE_EVS_CMD_LEN_ERR_EID 43 /** * \brief EVS Events Squelched Error Event ID diff --git a/modules/evs/fsw/inc/cfe_evs_msg.h b/modules/evs/fsw/inc/cfe_evs_msg.h index 1a106fcd3..e0948d5dc 100644 --- a/modules/evs/fsw/inc/cfe_evs_msg.h +++ b/modules/evs/fsw/inc/cfe_evs_msg.h @@ -60,7 +60,7 @@ ** following telemetry: ** - \b \c \EVS_CMDPC - command execution counter will ** increment -** - The #CFE_EVS_NOOP_EID informational event message will +** - The #CFE_EVS_NOOP_INF_EID informational event message will ** be generated ** ** \par Error Conditions @@ -96,7 +96,7 @@ ** will be reset to 0 ** - \b \c \EVS_CMDEC - command error counter ** will be reset to 0 -** - The #CFE_EVS_RSTCNT_EID debug event message will be +** - The #CFE_EVS_RESET_INF_EID debug event message will be ** generated ** ** \par Error Conditions diff --git a/modules/evs/fsw/src/cfe_evs_dispatch.c b/modules/evs/fsw/src/cfe_evs_dispatch.c index 53ee28f71..d29b35262 100644 --- a/modules/evs/fsw/src/cfe_evs_dispatch.c +++ b/modules/evs/fsw/src/cfe_evs_dispatch.c @@ -61,7 +61,7 @@ void CFE_EVS_ProcessCommandPacket(const CFE_SB_Buffer_t *SBBufPtr) default: /* Unknown command -- should never occur */ CFE_EVS_Global.EVS_TlmPkt.Payload.CommandErrorCounter++; - EVS_SendEvent(CFE_EVS_ERR_MSGID_EID, CFE_EVS_EventType_ERROR, "Invalid command packet, Message ID = 0x%08X", + EVS_SendEvent(CFE_EVS_MID_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid command packet, Message ID = 0x%08X", (unsigned int)CFE_SB_MsgIdToValue(MessageID)); break; } @@ -257,7 +257,7 @@ void CFE_EVS_ProcessGroundCommand(const CFE_SB_Buffer_t *SBBufPtr, CFE_SB_MsgId_ /* default is a bad command code as it was not found above */ default: - EVS_SendEvent(CFE_EVS_ERR_CC_EID, CFE_EVS_EventType_ERROR, "Invalid command code -- ID = 0x%08x, CC = %u", + EVS_SendEvent(CFE_EVS_CC_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid command code -- ID = 0x%08x, CC = %u", (unsigned int)CFE_SB_MsgIdToValue(MsgId), (unsigned int)FcnCode); Status = CFE_STATUS_BAD_COMMAND_CODE; @@ -299,7 +299,7 @@ bool CFE_EVS_VerifyCmdLength(const CFE_MSG_Message_t *MsgPtr, size_t ExpectedLen CFE_MSG_GetMsgId(MsgPtr, &MsgId); CFE_MSG_GetFcnCode(MsgPtr, &FcnCode); - EVS_SendEvent(CFE_EVS_LEN_ERR_EID, CFE_EVS_EventType_ERROR, + EVS_SendEvent(CFE_EVS_CMD_LEN_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid msg length: ID = 0x%X, CC = %u, Len = %u, Expected = %u", (unsigned int)CFE_SB_MsgIdToValue(MsgId), (unsigned int)FcnCode, (unsigned int)ActualLength, (unsigned int)ExpectedLength); diff --git a/modules/evs/fsw/src/cfe_evs_task.c b/modules/evs/fsw/src/cfe_evs_task.c index 782a297c5..14ee6ca91 100644 --- a/modules/evs/fsw/src/cfe_evs_task.c +++ b/modules/evs/fsw/src/cfe_evs_task.c @@ -302,7 +302,7 @@ int32 CFE_EVS_TaskInit(void) /* Write the AppID to the global location, now that the rest of initialization is done */ CFE_EVS_Global.EVS_AppID = AppID; - EVS_SendEvent(CFE_EVS_STARTUP_EID, CFE_EVS_EventType_INFORMATION, "cFE EVS Initialized: %s", CFE_VERSION_STRING); + EVS_SendEvent(CFE_EVS_INIT_INF_EID, CFE_EVS_EventType_INFORMATION, "cFE EVS Initialized: %s", CFE_VERSION_STRING); return CFE_SUCCESS; } @@ -315,7 +315,7 @@ int32 CFE_EVS_TaskInit(void) *-----------------------------------------------------------------*/ int32 CFE_EVS_NoopCmd(const CFE_EVS_NoopCmd_t *data) { - EVS_SendEvent(CFE_EVS_NOOP_EID, CFE_EVS_EventType_INFORMATION, "No-op Cmd Rcvd: %s", CFE_VERSION_STRING); + EVS_SendEvent(CFE_EVS_NOOP_INF_EID, CFE_EVS_EventType_INFORMATION, "No-op Cmd Rcvd: %s", CFE_VERSION_STRING); return CFE_SUCCESS; } @@ -399,7 +399,7 @@ int32 CFE_EVS_ResetCountersCmd(const CFE_EVS_ResetCountersCmd_t *data) CFE_EVS_Global.EVS_TlmPkt.Payload.MessageTruncCounter = 0; CFE_EVS_Global.EVS_TlmPkt.Payload.UnregisteredAppCounter = 0; - EVS_SendEvent(CFE_EVS_RSTCNT_EID, CFE_EVS_EventType_DEBUG, "Reset Counters Command Received"); + EVS_SendEvent(CFE_EVS_RESET_INF_EID, CFE_EVS_EventType_DEBUG, "Reset Counters Command Received"); /* NOTE: Historically the reset counters command does _NOT_ increment the command counter */ diff --git a/modules/evs/ut-coverage/evs_UT.c b/modules/evs/ut-coverage/evs_UT.c index 534743970..350decb70 100644 --- a/modules/evs/ut-coverage/evs_UT.c +++ b/modules/evs/ut-coverage/evs_UT.c @@ -297,7 +297,7 @@ void Test_Init(void) UT_EVS_DoGenericCheckEvents(CFE_EVS_TaskMain, &UT_EVS_EventBuf); CFE_UtAssert_SYSLOG(EVS_SYSLOG_MSGS[8]); - UtAssert_INT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_ERR_MSGID_EID); + UtAssert_INT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_MID_ERR_EID); /* Test early initialization with a get reset area failure */ UT_InitData(); @@ -979,7 +979,7 @@ void Test_Logging(void) memset(&CmdBuf, 0, sizeof(CmdBuf)); UT_EVS_DoDispatchCheckEvents(&CmdBuf.noopcmd, sizeof(CmdBuf.noopcmd), UT_TPID_CFE_EVS_CMD_NOOP_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_NOOP_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_NOOP_INF_EID); /* Clear log for next test */ UT_InitData(); @@ -1083,7 +1083,7 @@ void Test_WriteApp(void) UT_InitData(); UT_EVS_DoDispatchCheckEvents(&CmdBuf.ResetCountersCmd, sizeof(CmdBuf.ResetCountersCmd), UT_TPID_CFE_EVS_CMD_RESET_COUNTERS_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_RSTCNT_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_RESET_INF_EID); /* Test writing application data with a create failure using default * file name @@ -1750,122 +1750,122 @@ void Test_InvalidCmd(void) /* Test invalid msg id event */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, sizeof(cmd), UT_TPID_CFE_EVS_INVALID_MID, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_ERR_MSGID_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_MID_ERR_EID); /* Test invalid command code event */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, sizeof(cmd), UT_TPID_CFE_EVS_CMD_INVALID_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_ERR_CC_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CC_ERR_EID); /* Test invalid command length event */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_NOOP_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with reset counters command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_RESET_COUNTERS_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with enable event type command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_ENABLE_EVENT_TYPE_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with disable event type command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_DISABLE_EVENT_TYPE_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with set event format mode command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_SET_EVENT_FORMAT_MODE_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with enable application event * type command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_ENABLE_APP_EVENT_TYPE_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with disable application event * type command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_DISABLE_APP_EVENT_TYPE_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with enable application events command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_ENABLE_APP_EVENTS_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with disable application events command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_DISABLE_APP_EVENTS_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with reset application counter command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_RESET_APP_COUNTER_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with set filter command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_SET_FILTER_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with enable ports command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_ENABLE_PORTS_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with disable ports command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_DISABLE_PORTS_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with reset filter command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_RESET_FILTER_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with reset all filters command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_RESET_ALL_FILTERS_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with add event filter command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_ADD_EVENT_FILTER_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with delete event filter command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_DELETE_EVENT_FILTER_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with write application data command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_WRITE_APP_DATA_FILE_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with write log data command */ UT_InitData(); CFE_EVS_Global.EVS_TlmPkt.Payload.LogEnabled = true; UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_WRITE_LOG_DATA_FILE_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with set log mode command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_SET_LOG_MODE_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); /* Test invalid command length with clear log command */ UT_InitData(); UT_EVS_DoDispatchCheckEvents(&cmd, 0, UT_TPID_CFE_EVS_CMD_CLEAR_LOG_CC, &UT_EVS_EventBuf); - UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_LEN_ERR_EID); + UtAssert_UINT32_EQ(UT_EVS_EventBuf.EventID, CFE_EVS_CMD_LEN_ERR_EID); } /* diff --git a/modules/sb/eds/cfe_sb.xml b/modules/sb/eds/cfe_sb.xml index 7fff096e8..19da4ca12 100644 --- a/modules/sb/eds/cfe_sb.xml +++ b/modules/sb/eds/cfe_sb.xml @@ -474,7 +474,7 @@ Successful execution of this command may be verified with the following telemetry: - \b \c \SB_CMDPC - command execution counter will increment - - The #CFE_SB_CMD0_RCVD_EID informational event message will be generated + - The #CFE_SB_NOOP_INF_EID informational event message will be generated \par Error Conditions There are no error conditions for this command. If the Software @@ -508,7 +508,7 @@ Successful execution of this command may be verified with the following telemetry: - \b \c \SB_CMDPC - command execution counter will increment - - The #CFE_SB_CMD1_RCVD_EID informational event message will be generated + - The #CFE_SB_RESET_INF_EID informational event message will be generated \par Error Conditions There are no error conditions for this command. If the Software diff --git a/modules/sb/fsw/inc/cfe_sb_events.h b/modules/sb/fsw/inc/cfe_sb_events.h index 326542f9c..2460bc50d 100644 --- a/modules/sb/fsw/inc/cfe_sb_events.h +++ b/modules/sb/fsw/inc/cfe_sb_events.h @@ -39,7 +39,7 @@ * * Software Bus Services Task initialization complete. */ -#define CFE_SB_INIT_EID 1 +#define CFE_SB_INIT_INF_EID 1 /** * \brief SB Create Pipe API Bad Argument Event ID @@ -332,7 +332,7 @@ * * \link #CFE_SB_NOOP_CC SB NO-OP Command \endlink success. */ -#define CFE_SB_CMD0_RCVD_EID 28 +#define CFE_SB_NOOP_INF_EID 28 /** * \brief SB Reset Counters Command Success Event ID @@ -343,7 +343,7 @@ * * \link #CFE_SB_RESET_COUNTERS_CC SB Reset Counters Command \endlink success. */ -#define CFE_SB_CMD1_RCVD_EID 29 +#define CFE_SB_RESET_INF_EID 29 /** * \brief SB Send Statistics Command Success Event ID @@ -458,7 +458,7 @@ * Invalid command code for message ID #CFE_SB_CMD_MID or #CFE_SB_SUB_RPT_CTRL_MID received * on the SB message pipe. OVERLOADED */ -#define CFE_SB_BAD_CMD_CODE_EID 42 +#define CFE_SB_CC_ERR_EID 42 /** * \brief SB Invalid Message ID Received Event ID @@ -469,7 +469,7 @@ * * Invalid message ID received on the SB message pipe. */ -#define CFE_SB_BAD_MSGID_EID 43 +#define CFE_SB_MID_ERR_EID 43 /** * \brief SB Send Previous Subscriptions Command Full Packet Sent Event ID @@ -736,7 +736,7 @@ * Invalid length for the command code in message ID #CFE_SB_CMD_MID or #CFE_SB_SUB_RPT_CTRL_MID * received on the SB message pipe. */ -#define CFE_SB_LEN_ERR_EID 68 +#define CFE_SB_CMD_LEN_ERR_EID 68 /** * \brief SB Create Pipe API Name Taken Event ID diff --git a/modules/sb/fsw/inc/cfe_sb_msg.h b/modules/sb/fsw/inc/cfe_sb_msg.h index 87a7c1dc6..30b2f8638 100644 --- a/modules/sb/fsw/inc/cfe_sb_msg.h +++ b/modules/sb/fsw/inc/cfe_sb_msg.h @@ -59,7 +59,7 @@ ** following telemetry: ** - \b \c \SB_CMDPC - command execution counter will ** increment -** - The #CFE_SB_CMD0_RCVD_EID informational event message will +** - The #CFE_SB_NOOP_INF_EID informational event message will ** be generated ** ** \par Error Conditions @@ -103,7 +103,7 @@ ** - \b \c \SB_CMDPC - command execution counter will ** be reset to 0 ** - All other counters listed in description will be reset to 0 -** - The #CFE_SB_CMD1_RCVD_EID informational event message will +** - The #CFE_SB_RESET_INF_EID informational event message will ** be generated ** ** \par Error Conditions diff --git a/modules/sb/fsw/src/cfe_sb_dispatch.c b/modules/sb/fsw/src/cfe_sb_dispatch.c index 9db5f9770..d1d819276 100644 --- a/modules/sb/fsw/src/cfe_sb_dispatch.c +++ b/modules/sb/fsw/src/cfe_sb_dispatch.c @@ -56,7 +56,7 @@ bool CFE_SB_VerifyCmdLength(const CFE_MSG_Message_t *MsgPtr, size_t ExpectedLeng CFE_MSG_GetMsgId(MsgPtr, &MsgId); CFE_MSG_GetFcnCode(MsgPtr, &FcnCode); - CFE_EVS_SendEvent(CFE_SB_LEN_ERR_EID, CFE_EVS_EventType_ERROR, + CFE_EVS_SendEvent(CFE_SB_CMD_LEN_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid msg length: ID = 0x%X, CC = %u, Len = %u, Expected = %u", (unsigned int)CFE_SB_MsgIdToValue(MsgId), (unsigned int)FcnCode, (unsigned int)ActualLength, (unsigned int)ExpectedLength); @@ -114,7 +114,7 @@ void CFE_SB_ProcessCmdPipePkt(const CFE_SB_Buffer_t *SBBufPtr) break; default: - CFE_EVS_SendEvent(CFE_SB_BAD_CMD_CODE_EID, CFE_EVS_EventType_ERROR, + CFE_EVS_SendEvent(CFE_SB_CC_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid Cmd, Unexpected Command Code %u", (unsigned int)FcnCode); CFE_SB_Global.HKTlmMsg.Payload.CommandErrorCounter++; break; @@ -183,7 +183,7 @@ void CFE_SB_ProcessCmdPipePkt(const CFE_SB_Buffer_t *SBBufPtr) break; default: - CFE_EVS_SendEvent(CFE_SB_BAD_CMD_CODE_EID, CFE_EVS_EventType_ERROR, + CFE_EVS_SendEvent(CFE_SB_CC_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid Cmd, Unexpected Command Code %u", FcnCode); CFE_SB_Global.HKTlmMsg.Payload.CommandErrorCounter++; break; @@ -191,7 +191,7 @@ void CFE_SB_ProcessCmdPipePkt(const CFE_SB_Buffer_t *SBBufPtr) break; default: - CFE_EVS_SendEvent(CFE_SB_BAD_MSGID_EID, CFE_EVS_EventType_ERROR, "Invalid Cmd, Unexpected Msg Id: 0x%x", + CFE_EVS_SendEvent(CFE_SB_MID_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid Cmd, Unexpected Msg Id: 0x%x", (unsigned int)CFE_SB_MsgIdToValue(MessageID)); CFE_SB_Global.HKTlmMsg.Payload.CommandErrorCounter++; break; diff --git a/modules/sb/fsw/src/cfe_sb_task.c b/modules/sb/fsw/src/cfe_sb_task.c index 2f99e48f0..3fb91781b 100644 --- a/modules/sb/fsw/src/cfe_sb_task.c +++ b/modules/sb/fsw/src/cfe_sb_task.c @@ -266,8 +266,8 @@ int32 CFE_SB_AppInit(void) return Status; } - Status = - CFE_EVS_SendEvent(CFE_SB_INIT_EID, CFE_EVS_EventType_INFORMATION, "cFE SB Initialized: %s", CFE_VERSION_STRING); + Status = CFE_EVS_SendEvent(CFE_SB_INIT_INF_EID, CFE_EVS_EventType_INFORMATION, "cFE SB Initialized: %s", + CFE_VERSION_STRING); if (Status != CFE_SUCCESS) { CFE_ES_WriteToSysLog("%s: Error sending init event:RC=0x%08X\n", __func__, (unsigned int)Status); @@ -285,7 +285,7 @@ int32 CFE_SB_AppInit(void) *-----------------------------------------------------------------*/ int32 CFE_SB_NoopCmd(const CFE_SB_NoopCmd_t *data) { - CFE_EVS_SendEvent(CFE_SB_CMD0_RCVD_EID, CFE_EVS_EventType_INFORMATION, "No-op Cmd Rcvd: %s", CFE_VERSION_STRING); + CFE_EVS_SendEvent(CFE_SB_NOOP_INF_EID, CFE_EVS_EventType_INFORMATION, "No-op Cmd Rcvd: %s", CFE_VERSION_STRING); CFE_SB_Global.HKTlmMsg.Payload.CommandCounter++; return CFE_SUCCESS; @@ -299,7 +299,7 @@ int32 CFE_SB_NoopCmd(const CFE_SB_NoopCmd_t *data) *-----------------------------------------------------------------*/ int32 CFE_SB_ResetCountersCmd(const CFE_SB_ResetCountersCmd_t *data) { - CFE_EVS_SendEvent(CFE_SB_CMD1_RCVD_EID, CFE_EVS_EventType_DEBUG, "Reset Counters Cmd Rcvd"); + CFE_EVS_SendEvent(CFE_SB_RESET_INF_EID, CFE_EVS_EventType_DEBUG, "Reset Counters Cmd Rcvd"); CFE_SB_ResetCounters(); diff --git a/modules/sb/ut-coverage/sb_UT.c b/modules/sb/ut-coverage/sb_UT.c index ba6a3dded..834f58a21 100644 --- a/modules/sb/ut-coverage/sb_UT.c +++ b/modules/sb/ut-coverage/sb_UT.c @@ -392,7 +392,7 @@ void Test_SB_Main_RcvErr(void) CFE_UtAssert_EVENTCOUNT(6); - CFE_UtAssert_EVENTSENT(CFE_SB_INIT_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_INIT_INF_EID); CFE_UtAssert_EVENTSENT(CFE_SB_Q_RD_ERR_EID); @@ -443,8 +443,8 @@ void Test_SB_Main_Nominal(void) CFE_UtAssert_EVENTCOUNT(6); - CFE_UtAssert_EVENTSENT(CFE_SB_INIT_EID); - CFE_UtAssert_EVENTSENT(CFE_SB_CMD0_RCVD_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_INIT_INF_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_NOOP_INF_EID); /* remove the handler so the pipe can be deleted */ UT_SetHandlerFunction(UT_KEY(OS_QueueGet), NULL, NULL); @@ -518,10 +518,10 @@ void Test_SB_Cmds_Noop(void) CFE_UtAssert_EVENTCOUNT(1); - CFE_UtAssert_EVENTSENT(CFE_SB_CMD0_RCVD_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_NOOP_INF_EID); UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &Noop.SBBuf.Msg, 0, UT_TPID_CFE_SB_CMD_NOOP_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); } /* @@ -542,10 +542,10 @@ void Test_SB_Cmds_RstCtrs(void) CFE_UtAssert_EVENTCOUNT(1); - CFE_UtAssert_EVENTSENT(CFE_SB_CMD1_RCVD_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_RESET_INF_EID); UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &ResetCounters.SBBuf.Msg, 0, UT_TPID_CFE_SB_CMD_RESET_COUNTERS_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); } /* Test_SB_Cmds_RstCtrs */ /* @@ -588,7 +588,7 @@ void Test_SB_Cmds_Stats(void) CFE_UtAssert_EVENTSENT(CFE_SB_SND_STATS_EID); UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &SendSbStats.SBBuf.Msg, 0, UT_TPID_CFE_SB_CMD_SEND_SB_STATS_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId1)); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId2)); @@ -616,7 +616,7 @@ void Test_SB_Cmds_RoutingInfoDef(void) CFE_UtAssert_EVENTCOUNT(5); - CFE_UtAssert_EVENTSENT(CFE_SB_INIT_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_INIT_INF_EID); CFE_UtAssert_EVENTSENT(CFE_SB_SUBSCRIPTION_RCVD_EID); @@ -628,7 +628,7 @@ void Test_SB_Cmds_RoutingInfoDef(void) CFE_UtAssert_EVENTSENT(CFE_SB_SND_RTG_ERR1_EID); UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &WriteRoutingInfo.SBBuf.Msg, 0, UT_TPID_CFE_SB_CMD_WRITE_ROUTING_INFO_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(CFE_SB_Global.CmdPipe)); } @@ -772,7 +772,7 @@ void Test_SB_Cmds_PipeInfoDef(void) CFE_UtAssert_EVENTSENT(CFE_SB_SND_RTG_ERR1_EID); UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &WritePipeInfo.SBBuf.Msg, 0, UT_TPID_CFE_SB_CMD_WRITE_PIPE_INFO_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId1)); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId2)); @@ -977,7 +977,7 @@ void Test_SB_Cmds_MapInfoDef(void) /* Bad Size */ UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &WriteMapInfo.SBBuf.Msg, 0, UT_TPID_CFE_SB_CMD_WRITE_MAP_INFO_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId1)); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId2)); @@ -1044,7 +1044,7 @@ void Test_SB_Cmds_EnRouteValParam(void) /* Bad Size */ UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &EnableRoute.SBBuf.Msg, 0, UT_TPID_CFE_SB_CMD_ENABLE_ROUTE_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId)); } @@ -1192,7 +1192,7 @@ void Test_SB_Cmds_DisRouteValParam(void) /* Bad Size */ UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &DisableRoute.SBBuf.Msg, 0, UT_TPID_CFE_SB_CMD_DISABLE_ROUTE_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId)); } @@ -1461,7 +1461,7 @@ void Test_SB_Cmds_SendPrevSubs(void) /* Bad Size */ UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &SendPrevSubs.SBBuf.Msg, 0, UT_TPID_CFE_SB_SUB_RPT_CTL_SEND_PREV_SUBS_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId1)); CFE_UtAssert_TEARDOWN(CFE_SB_DeletePipe(PipeId2)); @@ -1492,7 +1492,7 @@ void Test_SB_Cmds_SubRptOn(void) UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &EnableSubReporting.SBBuf.Msg, 0, UT_TPID_CFE_SB_SUB_RPT_CTL_ENABLE_SUB_REPORTING_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); } /* @@ -1520,7 +1520,7 @@ void Test_SB_Cmds_SubRptOff(void) UT_CallTaskPipe(CFE_SB_ProcessCmdPipePkt, &DisableSubReporting.SBBuf.Msg, 0, UT_TPID_CFE_SB_SUB_RPT_CTL_DISABLE_SUB_REPORTING_CC); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); } /* @@ -1531,7 +1531,7 @@ void Test_SB_Cmds_CmdUnexpCmdCode(void) UT_SetupBasicMsgDispatch(&UT_TPID_CFE_SB_CMD_BAD_FCNCODE, sizeof(CFE_MSG_CommandHeader_t), true); CFE_SB_ProcessCmdPipePkt((CFE_SB_Buffer_t *)NULL); CFE_UtAssert_EVENTCOUNT(1); - CFE_UtAssert_EVENTSENT(CFE_SB_BAD_CMD_CODE_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CC_ERR_EID); UT_SetupBasicMsgDispatch(&UT_TPID_CFE_SB_SUB_RPT_CTRL_BAD_FCNCODE, sizeof(CFE_MSG_CommandHeader_t), true); CFE_SB_ProcessCmdPipePkt((CFE_SB_Buffer_t *)NULL); @@ -1556,7 +1556,7 @@ void Test_SB_Cmds_BadCmdLength(void) CFE_UtAssert_EVENTCOUNT(1); - CFE_UtAssert_EVENTSENT(CFE_SB_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_CMD_LEN_ERR_EID); } /* @@ -1569,7 +1569,7 @@ void Test_SB_Cmds_UnexpMsgId(void) CFE_UtAssert_EVENTCOUNT(1); - CFE_UtAssert_EVENTSENT(CFE_SB_BAD_MSGID_EID); + CFE_UtAssert_EVENTSENT(CFE_SB_MID_ERR_EID); } /* diff --git a/modules/tbl/fsw/inc/cfe_tbl_events.h b/modules/tbl/fsw/inc/cfe_tbl_events.h index 83c2205ad..79ea02541 100644 --- a/modules/tbl/fsw/inc/cfe_tbl_events.h +++ b/modules/tbl/fsw/inc/cfe_tbl_events.h @@ -243,7 +243,7 @@ * * Invalid command code for message ID #CFE_TBL_CMD_MID received on the TBL message pipe. */ -#define CFE_TBL_CC1_ERR_EID 51 +#define CFE_TBL_CC_ERR_EID 51 /** * \brief TBL Invalid Command Length Event ID @@ -254,7 +254,7 @@ * * Invalid length for the message ID and command code received on the TBL message pipe. */ -#define CFE_TBL_LEN_ERR_EID 52 +#define CFE_TBL_CMD_LEN_ERR_EID 52 /** * \brief TBL Load Table File Open Failure Event ID diff --git a/modules/tbl/fsw/src/cfe_tbl_dispatch.c b/modules/tbl/fsw/src/cfe_tbl_dispatch.c index b9d8a2484..b16f55623 100644 --- a/modules/tbl/fsw/src/cfe_tbl_dispatch.c +++ b/modules/tbl/fsw/src/cfe_tbl_dispatch.c @@ -117,7 +117,7 @@ void CFE_TBL_TaskPipe(const CFE_SB_Buffer_t *SBBufPtr) } else /* Bad Message Length */ { - CFE_EVS_SendEvent(CFE_TBL_LEN_ERR_EID, CFE_EVS_EventType_ERROR, + CFE_EVS_SendEvent(CFE_TBL_CMD_LEN_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid msg length -- ID = 0x%X, CC = %u, Len = %u, Expected = %u", (unsigned int)CFE_SB_MsgIdToValue(MessageID), (unsigned int)CommandCode, (unsigned int)ActualLength, (unsigned int)CFE_TBL_CmdHandlerTbl[CmdIndx].ExpectedLength); @@ -142,8 +142,7 @@ void CFE_TBL_TaskPipe(const CFE_SB_Buffer_t *SBBufPtr) /* "Bad Command Code" or "Bad Message ID" */ if (CmdIndx == CFE_TBL_BAD_CMD_CODE) { - CFE_EVS_SendEvent(CFE_TBL_CC1_ERR_EID, CFE_EVS_EventType_ERROR, - "Invalid command code -- ID = 0x%X, CC = %u", + CFE_EVS_SendEvent(CFE_TBL_CC_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid command code -- ID = 0x%X, CC = %u", (unsigned int)CFE_SB_MsgIdToValue(MessageID), (unsigned int)CommandCode); /* Update the command error counter */ diff --git a/modules/tbl/ut-coverage/tbl_UT.c b/modules/tbl/ut-coverage/tbl_UT.c index 77db831f9..c5b9a874a 100644 --- a/modules/tbl/ut-coverage/tbl_UT.c +++ b/modules/tbl/ut-coverage/tbl_UT.c @@ -308,14 +308,14 @@ void Test_CFE_TBL_TaskInit(void) */ UT_InitData(); UT_CallTaskPipe(CFE_TBL_TaskPipe, &CmdBuf.Msg, sizeof(CmdBuf.NoopCmd) - 1, UT_TPID_CFE_TBL_CMD_NOOP_CC); - CFE_UtAssert_EVENTSENT(CFE_TBL_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TBL_CMD_LEN_ERR_EID); /* Test command pipe messages handler response to an invalid * command code */ UT_InitData(); UT_CallTaskPipe(CFE_TBL_TaskPipe, &CmdBuf.Msg, sizeof(CmdBuf.NoopCmd), UT_TPID_CFE_TBL_CMD_INVALID_CC); - CFE_UtAssert_EVENTSENT(CFE_TBL_CC1_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TBL_CC_ERR_EID); /* Test command pipe messages handler response to other errors */ UT_InitData(); diff --git a/modules/time/eds/cfe_time.xml b/modules/time/eds/cfe_time.xml index 0e58d9605..7b0d23d08 100644 --- a/modules/time/eds/cfe_time.xml +++ b/modules/time/eds/cfe_time.xml @@ -495,7 +495,7 @@ Successful execution of this command may be verified with the following telemetry: - \b \c \TIME_CMDPC - command execution counter will increment - - The #CFE_TIME_NOOP_EID informational event message will be generated + - The #CFE_TIME_NOOP_INF_EID informational event message will be generated \par Error Conditions There are no error conditions for this command. If the Time @@ -541,7 +541,7 @@ Successful execution of this command may be verified with the following telemetry: - \b \c \TIME_CMDPC - command execution counter will increment - - The #CFE_TIME_RESET_EID informational event message will be generated + - The #CFE_TIME_RESET_INF_EID informational event message will be generated \par Error Conditions There are no error conditions for this command. If the Time diff --git a/modules/time/fsw/inc/cfe_time_events.h b/modules/time/fsw/inc/cfe_time_events.h index 9ea5eb239..d3d8cfff5 100644 --- a/modules/time/fsw/inc/cfe_time_events.h +++ b/modules/time/fsw/inc/cfe_time_events.h @@ -39,7 +39,7 @@ * * Time Services Task Initialization complete. */ -#define CFE_TIME_INIT_EID 1 +#define CFE_TIME_INIT_INF_EID 1 /** * \brief TIME No-op Command Success Event ID @@ -50,7 +50,7 @@ * * \link #CFE_TIME_NOOP_CC TIME NO-OP Command \endlink success. */ -#define CFE_TIME_NOOP_EID 4 +#define CFE_TIME_NOOP_INF_EID 4 /** * \brief TIME Reset Counters Command Success Event ID @@ -61,7 +61,7 @@ * * \link #CFE_TIME_RESET_COUNTERS_CC TIME Reset Counters Command \endlink success. */ -#define CFE_TIME_RESET_EID 5 +#define CFE_TIME_RESET_INF_EID 5 /** * \brief TIME Request Diagnostics Command Success Event ID @@ -218,7 +218,7 @@ * * Invalid message ID received on the TIME message pipe. */ -#define CFE_TIME_ID_ERR_EID 26 +#define CFE_TIME_MID_ERR_EID 26 /** * \brief TIME Invalid Command Code Received Event ID @@ -447,7 +447,7 @@ * Invalid length for the command code in message ID #CFE_TIME_CMD_MID received on the TIME * message pipe. */ -#define CFE_TIME_LEN_ERR_EID 49 +#define CFE_TIME_CMD_LEN_ERR_EID 49 /**\}*/ #endif /* CFE_TIME_EVENTS_H */ diff --git a/modules/time/fsw/inc/cfe_time_msg.h b/modules/time/fsw/inc/cfe_time_msg.h index 216821790..debee4d2a 100644 --- a/modules/time/fsw/inc/cfe_time_msg.h +++ b/modules/time/fsw/inc/cfe_time_msg.h @@ -61,7 +61,7 @@ ** Successful execution of this command may be verified with the ** following telemetry: ** - \b \c \TIME_CMDPC - command execution counter will increment -** - The #CFE_TIME_NOOP_EID informational event message will be generated +** - The #CFE_TIME_NOOP_INF_EID informational event message will be generated ** ** \par Error Conditions ** There are no error conditions for this command. If the Time @@ -106,7 +106,7 @@ ** following telemetry: ** - \b \c \TIME_CMDPC - command execution counter will reset to 0 ** - \b \c \TIME_CMDEC - command error counter will reset to 0 -** - The #CFE_TIME_RESET_EID informational event message will be generated +** - The #CFE_TIME_RESET_INF_EID informational event message will be generated ** ** \par Error Conditions ** There are no error conditions for this command. If the Time diff --git a/modules/time/fsw/src/cfe_time_dispatch.c b/modules/time/fsw/src/cfe_time_dispatch.c index fcd6e08fd..72ae76894 100644 --- a/modules/time/fsw/src/cfe_time_dispatch.c +++ b/modules/time/fsw/src/cfe_time_dispatch.c @@ -51,7 +51,7 @@ bool CFE_TIME_VerifyCmdLength(const CFE_MSG_Message_t *MsgPtr, size_t ExpectedLe CFE_MSG_GetMsgId(MsgPtr, &MsgId); CFE_MSG_GetFcnCode(MsgPtr, &FcnCode); - CFE_EVS_SendEvent(CFE_TIME_LEN_ERR_EID, CFE_EVS_EventType_ERROR, + CFE_EVS_SendEvent(CFE_TIME_CMD_LEN_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid msg length: ID = 0x%X, CC = %u, Len = %u, Expected = %u", (unsigned int)CFE_SB_MsgIdToValue(MsgId), (unsigned int)FcnCode, (unsigned int)ActualLength, (unsigned int)ExpectedLength); @@ -256,7 +256,7 @@ void CFE_TIME_TaskPipe(const CFE_SB_Buffer_t *SBBufPtr) ** Note: we only increment the command error counter when ** processing CFE_TIME_CMD_MID commands... */ - CFE_EVS_SendEvent(CFE_TIME_ID_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid message ID -- ID = 0x%X", + CFE_EVS_SendEvent(CFE_TIME_MID_ERR_EID, CFE_EVS_EventType_ERROR, "Invalid message ID -- ID = 0x%X", (unsigned int)CFE_SB_MsgIdToValue(MessageID)); break; diff --git a/modules/time/fsw/src/cfe_time_task.c b/modules/time/fsw/src/cfe_time_task.c index f392e6627..3f2f7860b 100644 --- a/modules/time/fsw/src/cfe_time_task.c +++ b/modules/time/fsw/src/cfe_time_task.c @@ -259,7 +259,7 @@ int32 CFE_TIME_TaskInit(void) return Status; } - Status = CFE_EVS_SendEvent(CFE_TIME_INIT_EID, CFE_EVS_EventType_INFORMATION, "cFE TIME Initialized: %s", + Status = CFE_EVS_SendEvent(CFE_TIME_INIT_INF_EID, CFE_EVS_EventType_INFORMATION, "cFE TIME Initialized: %s", CFE_VERSION_STRING); if (Status != CFE_SUCCESS) { @@ -445,7 +445,7 @@ int32 CFE_TIME_NoopCmd(const CFE_TIME_NoopCmd_t *data) { CFE_TIME_Global.CommandCounter++; - CFE_EVS_SendEvent(CFE_TIME_NOOP_EID, CFE_EVS_EventType_INFORMATION, "No-op Cmd Rcvd: %s", CFE_VERSION_STRING); + CFE_EVS_SendEvent(CFE_TIME_NOOP_INF_EID, CFE_EVS_EventType_INFORMATION, "No-op Cmd Rcvd: %s", CFE_VERSION_STRING); return CFE_SUCCESS; } @@ -483,7 +483,7 @@ int32 CFE_TIME_ResetCountersCmd(const CFE_TIME_ResetCountersCmd_t *data) CFE_TIME_Global.InternalCount = 0; CFE_TIME_Global.ExternalCount = 0; - CFE_EVS_SendEvent(CFE_TIME_RESET_EID, CFE_EVS_EventType_DEBUG, "Reset Counters command"); + CFE_EVS_SendEvent(CFE_TIME_RESET_INF_EID, CFE_EVS_EventType_DEBUG, "Reset Counters command"); return CFE_SUCCESS; } diff --git a/modules/time/ut-coverage/time_UT.c b/modules/time/ut-coverage/time_UT.c index 3a4961ac1..6831eda00 100644 --- a/modules/time/ut-coverage/time_UT.c +++ b/modules/time/ut-coverage/time_UT.c @@ -1386,16 +1386,16 @@ void Test_PipeCmds(void) #if (CFE_PLATFORM_TIME_CFG_SERVER == true) UtAssert_UINT32_EQ(CFE_TIME_Global.InternalCount, count + 1); - CFE_UtAssert_EVENTNOTSENT(CFE_TIME_ID_ERR_EID); + CFE_UtAssert_EVENTNOTSENT(CFE_TIME_MID_ERR_EID); #else - CFE_UtAssert_EVENTSENT(CFE_TIME_ID_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_MID_ERR_EID); #endif /* Test sending the no-op command */ UT_InitData(); memset(&CmdBuf, 0, sizeof(CmdBuf)); UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, sizeof(CmdBuf.noopcmd), UT_TPID_CFE_TIME_CMD_NOOP_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_NOOP_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_NOOP_INF_EID); /* Noop with bad size */ UT_InitData(); @@ -1403,7 +1403,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_NOOP_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1418,10 +1418,9 @@ void Test_PipeCmds(void) CFE_TIME_Global.ToneTaskCounter = 1; CFE_TIME_Global.LocalIntCounter = 1; CFE_TIME_Global.LocalTaskCounter = 1; - memset(&CmdBuf, 0, sizeof(CmdBuf)); UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, sizeof(CmdBuf.resetcounterscmd), UT_TPID_CFE_TIME_CMD_RESET_COUNTERS_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_RESET_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_RESET_INF_EID); /* Confirm error counters get reset to help cover requirements that are difficult operationally */ UtAssert_ZERO(CFE_TIME_Global.ToneMatchCounter); @@ -1440,7 +1439,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_RESET_COUNTERS_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1457,7 +1456,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SEND_DIAGNOSTIC_TLM_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1497,7 +1496,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SET_STATE_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1540,7 +1539,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SET_SOURCE_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1597,7 +1596,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SET_SIGNAL_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1626,7 +1625,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_ADD_DELAY_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1653,7 +1652,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SUB_DELAY_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1680,7 +1679,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SET_TIME_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1707,7 +1706,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SET_MET_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1734,7 +1733,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SET_STCF_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1761,7 +1760,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_ADD_ADJUST_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1788,7 +1787,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SUB_ADJUST_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1816,7 +1815,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_ADD_1HZ_ADJUSTMENT_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1844,7 +1843,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SUB_1HZ_ADJUSTMENT_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1908,7 +1907,7 @@ void Test_PipeCmds(void) CFE_TIME_Global.CommandCounter = 0; CFE_TIME_Global.CommandErrorCounter = 0; UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, 0, UT_TPID_CFE_TIME_CMD_SET_LEAP_SECONDS_CC); - CFE_UtAssert_EVENTSENT(CFE_TIME_LEN_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_CMD_LEN_ERR_EID); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandCounter, 0); UtAssert_UINT32_EQ(CFE_TIME_Global.CommandErrorCounter, 1); @@ -1922,7 +1921,7 @@ void Test_PipeCmds(void) UT_InitData(); memset(&CmdBuf, 0, sizeof(CmdBuf)); UT_CallTaskPipe(CFE_TIME_TaskPipe, &CmdBuf.message, sizeof(CmdBuf.noopcmd), UT_TPID_CFE_TIME_INVALID_MID); - CFE_UtAssert_EVENTSENT(CFE_TIME_ID_ERR_EID); + CFE_UtAssert_EVENTSENT(CFE_TIME_MID_ERR_EID); /* Call the Task Pipe with the 1Hz command. */ /* In the 1Hz state machine it should call PSP GetTime as part,