Fix nasa#50, Apply consistent Event ID names to common events in MM

Author: thnkslprpt

fsw/inc/mm_events.h

@@ -228,7 +228,7 @@
  *  This event message is issued when a software bus message is received
  *  with an invalid command code.
  */
-#define MM_CC1_ERR_EID 17
+#define MM_CC_ERR_EID 17
 
 /**
  * \brief MM Command Message Length Invalid Event ID
@@ -240,7 +240,7 @@
  *  This event message is issued when command message is received with a message
  *  length that doesn't match the expected value.
  */
-#define MM_LEN_ERR_EID 18
+#define MM_CMD_LEN_ERR_EID 18
 
 /**
  * \brief MM Commanded Memory Type Invalid Event ID

fsw/src/mm_app.c

@@ -348,7 +348,7 @@ void MM_AppPipe(const CFE_SB_Buffer_t *BufPtr)
 
                 default:
                     CmdResult = false;
-                    CFE_EVS_SendEvent(MM_CC1_ERR_EID, CFE_EVS_EventType_ERROR,
+                    CFE_EVS_SendEvent(MM_CC_ERR_EID, CFE_EVS_EventType_ERROR,
                                       "Invalid ground command code: ID = 0x%08lX, CC = %d",
                                       (unsigned long)CFE_SB_MsgIdToValue(MessageID), CommandCode);
                     break;

fsw/src/mm_utils.c

@@ -117,7 +117,7 @@ bool MM_VerifyCmdLength(const CFE_MSG_Message_t *MsgPtr, size_t ExpectedLength)
         }
         else
         {
-            CFE_EVS_SendEvent(MM_LEN_ERR_EID, CFE_EVS_EventType_ERROR,
+            CFE_EVS_SendEvent(MM_CMD_LEN_ERR_EID, CFE_EVS_EventType_ERROR,
                               "Invalid msg length: ID = 0x%08lX, CC = %d, Len = %d, Expected = %d",
                               (unsigned long)CFE_SB_MsgIdToValue(MessageID), CommandCode, (int)ActualLength,
                               (int)ExpectedLength);

fsw/src/mm_utils.h

@@ -79,7 +79,7 @@ void MM_SegmentBreak(void);
  *  \retval true  Length matches expected
  *  \retval false Length does not match expected
  *
- *  \sa #MM_LEN_ERR_EID
+ *  \sa #MM_CMD_LEN_ERR_EID
  */
 bool MM_VerifyCmdLength(const CFE_MSG_Message_t *MsgPtr, size_t ExpectedLength);
 

unit-test/mm_app_tests.c

@@ -1100,7 +1100,7 @@ void MM_AppPipe_Test_InvalidCommandCode(void)
     UtAssert_INT32_EQ(MM_AppData.HkPacket.Payload.CmdCounter, 0);
     UtAssert_INT32_EQ(MM_AppData.HkPacket.Payload.ErrCounter, 1);
 
-    UtAssert_INT32_EQ(context_CFE_EVS_SendEvent[0].EventID, MM_CC1_ERR_EID);
+    UtAssert_INT32_EQ(context_CFE_EVS_SendEvent[0].EventID, MM_CC_ERR_EID);
     UtAssert_INT32_EQ(context_CFE_EVS_SendEvent[0].EventType, CFE_EVS_EventType_ERROR);
 
     strCmpResult = strncmp(ExpectedEventString, context_CFE_EVS_SendEvent[0].Spec, CFE_MISSION_EVS_MAX_MESSAGE_LENGTH);

unit-test/mm_utils_tests.c

@@ -180,7 +180,7 @@ void MM_VerifyCmdLength_Test_LengthError(void)
     /* Verify results */
     UtAssert_True(Result == false, "Result == false");
 
-    UtAssert_INT32_EQ(context_CFE_EVS_SendEvent[0].EventID, MM_LEN_ERR_EID);
+    UtAssert_INT32_EQ(context_CFE_EVS_SendEvent[0].EventID, MM_CMD_LEN_ERR_EID);
     UtAssert_INT32_EQ(context_CFE_EVS_SendEvent[0].EventType, CFE_EVS_EventType_ERROR);
 
     strCmpResult = strncmp(ExpectedEventString, context_CFE_EVS_SendEvent[0].Spec, CFE_MISSION_EVS_MAX_MESSAGE_LENGTH);