Fix #97, Remove dead code checks: NULL bufptr and RunStatus_SYS_EXCEPTION

[thnkslprpt]

Checklist

• I reviewed the Contributing Guide.
• I signed and emailed the appropriate Contributor License Agreement to GSFC-SoftwareRelease@mail.nasa.gov and copied cfs-program@lists.nasa.gov.

Describe the contribution

• Fixes Dead code checking for CFE_ES_RunStatus_SYS_EXCEPTION and NULL bufptr #97

Dead Code removed (along with their accompanying tests):

• bufptr cannot be NULL after a successful return from CFE_SB_ReceiveBuffer
• status check for CFE_ES_RunStatus_SYS_EXCEPTION not necessary (and impossible to reach other than through forced test conditions) given that the code setting that status was removed at some point in the past

Minor changes to improve consistency and use the preferred symbol naming conventions:

• rename CmdCounter to ComandCounter
• rename CmdErrCounter to CommandErrorCounter
• rename CS_ResetCmd to CS_ResetCountersCmd

Testing performed
GitHub CI actions all passing successfully (incl. Build + Run, Unit/Functional Tests etc.).

Expected behavior changes
Remove dead code to reduce unnecessary clutter.

System(s) tested on
Debian 12 using the current main branch of cFS bundle.

Contributor Info
Avi Weiss   @thnkslprpt