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.

Sign up for GitHub

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

Jump to bottom

Add Object Utility API functional tests #374

Closed
skliper opened this issue Mar 23, 2020 · 2 comments · Fixed by #439 or #482
Closed

Add Object Utility API functional tests #374

skliper opened this issue Mar 23, 2020 · 2 comments · Fixed by #439 or #482
Assignees
@yammajamma
Labels
enhancement unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Milestone
Bootes

Comments

@skliper
Copy link
Contributor

skliper commented Mar 23, 2020

Is your feature request related to a problem? Please describe.
OS_IdentifyObject, OS_ConvertToArrayIndex and OS_ForEachObject missing explicit functional tests

Describe the solution you'd like
Add tests

Describe alternatives you've considered
None.

Additional context
Certification violation

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 5.1.0 milestone Mar 23, 2020
@CDKnightNASA CDKnightNASA added the unit-test Tickets related to the OSAL unit testing (functional and/or coverage) label Mar 27, 2020
@yammajamma
Copy link
Contributor

I can take this one.

@skliper
Copy link
Contributor Author

skliper commented Apr 21, 2020

Recommend following the osal/src/tests model. Concept is to demonstrate these functionally work as intended/specified in the API (black box). You can find the API here: https://github.com/nasa/cFS/blob/gh-pages/OSAL_Users_Guide.pdf

yammajamma added a commit to yammajamma/osal that referenced this issue Apr 23, 2020
@yammajamma
Fix nasa#374, Added functional tests for OS_ConvertToArrayIndex and O…
df20ff9 
…S_IdentifyObject
yammajamma added a commit to yammajamma/osal that referenced this issue Apr 28, 2020
@yammajamma
Fix nasa#374, Created Object Utility API's missing functional tests
8afcf41
@yammajamma yammajamma mentioned this issue Apr 28, 2020
Merged
yammajamma added a commit to yammajamma/osal that referenced this issue May 6, 2020
@yammajamma
Fix nasa#374, created rigorous idmap-api-test functional tests and re…
8fbea56 
…moved the ReadMe
yammajamma added a commit to yammajamma/osal that referenced this issue May 6, 2020
@yammajamma
Fix nasa#374, Add idmap functional tests
1822026
skliper pushed a commit to yammajamma/osal that referenced this issue May 12, 2020
@yammajamma @skliper
Fix nasa#374, Add idmap functional tests
7d8d892
skliper pushed a commit to yammajamma/osal that referenced this issue May 13, 2020
@yammajamma @skliper
Fix nasa#374, Add idmap functional tests
8352d2f
yammajamma added a commit to yammajamma/osal that referenced this issue May 22, 2020
@yammajamma
Fix nasa#374, Updated OS_ConvertToArrayIndex to verify output is in r…
7466997 
…ange between 0 and <MAX and output array indices vary
astrogeco added a commit that referenced this issue May 26, 2020
@astrogeco
Merge pull request #439 from yammajamma/fix374-2-Object-Utilities-Mis…
35d8b73 
…sing-Tests

Fix #374, Add Object Utility API functional tests
@astrogeco astrogeco mentioned this issue May 26, 2020
Merged
@yammajamma yammajamma mentioned this issue May 26, 2020
Merged
astrogeco added a commit that referenced this issue May 26, 2020
@astrogeco
Merge pull request #483 from yammajamma/HOTFIX-374-Object-Utilitys-Mi…
5f910f7 
…ssing-Functional-API-Tests

Hotfix #374 object utilitys missing functional api tests
@astrogeco astrogeco changed the title Object Utility API's missing functional test Add Object Utility API functional tests Sep 22, 2020
jphickey added a commit to jphickey/osal that referenced this issue Aug 10, 2022
@jphickey @skliper
Fix nasa#374: CFE_ES_GetTaskInfo must be passed a task ID
198abc6 
Fix 2 areas where the CFE_ES_GetTaskInfo() function could
have been passed an App ID rather than a task ID, which
would create an incorrect result.
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@skliper
CCB 20191023/30: Merge nasa#362 nasa#375 nasa#376 nasa#382
e9f7382 
Fixes nasa#361, nasa#373, nasa#374, nasa#381
Code reviewed and approved at 20191023 and 30 CCBs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yammajamma yammajamma

Labels
enhancement unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Projects
None yet
Milestone
Bootes
3 participants
@CDKnightNASA @skliper @yammajamma