You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Many cases in cfe_es_task.c where code can't be exercised based on the definition of GLOBAL_CONFIGDATA in the psp. Example:
Describe the solution you'd like
Provide a mechanism for overriding or defining GLOBAL_CONFIGDATA from the test code. Some branches require both NULL and non-NULL values for coverage, so likely would need multiple builds.
Describe alternatives you've considered
Coverage could be improved by just exposing a few of the internal routines and/or splitting up code and providing stubs. With 2 versions available from the PSP likely could achieve full coverage, but clunky to cross repos just for coverage...
Additional context
None
Requester Info
Jacob Hageman - NASA/GSFC
The text was updated successfully, but these errors were encountered:
Is your feature request related to a problem? Please describe.
Many cases in cfe_es_task.c where code can't be exercised based on the definition of GLOBAL_CONFIGDATA in the psp. Example:
here:
cFE/modules/es/fsw/src/cfe_es_task.c
Lines 194 to 206 in e5d4ed9
Impacts CFE_ES_FindConfigKeyValue, CFE_ES_GenerateSingleVersionEvent, CFE_ES_GenerateVersionEvents, CFE_ES_GenerateBuildInfoEvents
Describe the solution you'd like
Provide a mechanism for overriding or defining GLOBAL_CONFIGDATA from the test code. Some branches require both NULL and non-NULL values for coverage, so likely would need multiple builds.
Describe alternatives you've considered
Coverage could be improved by just exposing a few of the internal routines and/or splitting up code and providing stubs. With 2 versions available from the PSP likely could achieve full coverage, but clunky to cross repos just for coverage...
Additional context
None
Requester Info
Jacob Hageman - NASA/GSFC
The text was updated successfully, but these errors were encountered: