Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Incomplete coverage test for cfe_es_apps.c #469

Closed
avan989 opened this issue Jan 9, 2020 · 0 comments · Fixed by #1695 or #1701
Closed

Incomplete coverage test for cfe_es_apps.c #469

avan989 opened this issue Jan 9, 2020 · 0 comments · Fixed by #1695 or #1701
Assignees
Milestone

Comments

@avan989
Copy link
Contributor

avan989 commented Jan 9, 2020

Is your feature request related to a problem? Please describe.
Current coverage for cfe_es_apps.c = 96.9 %
Missing coverage test for the following:

CFE_ES_AppCreate

    646           0 :                 CFE_ES_WriteToSysLog("ES Startup: Error removing temp RAM disk file, EC = 0x%08X\n",
     647             :                                      (unsigned int) ReturnCode);

CFE_ES_CleanupObjectCallback

    1363           0 :     case OS_OBJECT_TYPE_OS_MODULE:
    1364           0 :         Status = OS_ModuleUnload(ObjectId);
    1365           0 :         break;
    1366           0 :     default:
    1367           0 :         ObjIsValid = false;
    1368           0 :         Status = OS_ERROR;
    1369           0 :         break;

CFE_ES_CleanupTaskResources
1467 0 : Result = CFE_ES_APP_CLEANUP_ERR;

CFE_ES_CountObjectCallback

  1495           0 : void CFE_ES_CountObjectCallback(uint32 ObjectId, void *arg)
   1496             : {
   1497             :     uint32                 *CountState;
   1498             :     uint32                 idtype;
   1499             : 
   1500           0 :     CountState = (uint32 *)arg;
   1501           0 :     idtype = OS_IdentifyObject(ObjectId);
   1502           0 :     if (idtype < OS_OBJECT_TYPE_USER)
   1503             :     {
   1504           0 :         ++CountState[idtype];
   1505             :     }
   1506           0 : }

Requester Info
Anh Van, NASA Goddard

@skliper skliper added this to the 6.8.0 milestone Feb 25, 2020
@skliper skliper modified the milestones: 6.8.0, 7.0.0 Jun 5, 2020
@skliper skliper changed the title Incomplete unit test for cfe_es_apps.c Incomplete coverage test for cfe_es_apps.c Jan 11, 2021
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jul 20, 2021
Added tests to complete coverage on the following functions:
* CFE_ES_RunAppTableScan
* CFE_ES_ParseFileEntry
* CFE_ES_LoadModule
* CFE_ES_CleanUpApp
* CFE_ES_CleanupObjectCallback
pepepr08 added a commit to pepepr08/cFE that referenced this issue Jul 20, 2021
Added tests to complete coverage on the following functions:
* CFE_ES_RunAppTableScan
* CFE_ES_ParseFileEntry
* CFE_ES_LoadModule
* CFE_ES_CleanUpApp
* CFE_ES_CleanupObjectCallback
astrogeco added a commit that referenced this issue Jul 21, 2021
Fix #469, Adding coverage tests for cfe_es_apps.c
paulober pushed a commit to paulober/cFE that referenced this issue Aug 1, 2021
Added tests to complete coverage on the following functions:
* CFE_ES_RunAppTableScan
* CFE_ES_ParseFileEntry
* CFE_ES_LoadModule
* CFE_ES_CleanUpApp
* CFE_ES_CleanupObjectCallback
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants