-
Notifications
You must be signed in to change notification settings - Fork 217
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
Unit test status code discrepancies in osapi-timebase.h #1022
Comments
Plan is to add these functions into the "ostimer-test" unit-test to verify the return codes of these routines. |
Correction/Update - after closer consideration, since many of these return values are actually checked in the "time-base-api-test" program, preferable approach is to avoid unnecessary duplication. Will update the cross reference set to include these test cases, rather than duplicating many of them in "unit-tests". |
Ensures correlation between the test cases and documented return values for the OSAL timebase API.
Ensures correlation between the test cases and documented return values for the OSAL timebase API.
Ensures correlation between the test cases and documented return values for the OSAL timebase API.
Ensures correlation between the test cases and documented return values for the OSAL timebase API.
Fix #1022, resolve discrepancies between timebase API and unit tests
Ensures correlation between the test cases and documented return values for the OSAL timebase API.
Fix nasa#1004, default stack size
Is your feature request related to a problem? Please describe.
The unit tests and API documentation should match, where all status codes in the API documentation are tested in a unit test, and likewise all codes being tested for are documented in the API.
Describe the solution you'd like
Resolve the following mismatches:
NOTE: This API does not have a "unit-tests" program to test these status codes - Hence why they are all flagged as being only in doxygen. The "time-base-api-test" program does exercise many of them, but isn't included in this test run.
File
src/os/inc/osapi-timebase.h
, functionOS_TimeBaseCreate()
:File
src/os/inc/osapi-timebase.h
, functionOS_TimeBaseSet()
:File
src/os/inc/osapi-timebase.h
, functionOS_TimeBaseDelete()
:File
src/os/inc/osapi-timebase.h
, functionOS_TimeBaseGetIdByName()
:File
src/os/inc/osapi-timebase.h
, functionOS_TimeBaseGetInfo()
:File
src/os/inc/osapi-timebase.h
, functionOS_TimeBaseGetFreeRun()
:Additional context
Originally noted in #331, split into separate work items.
Requester Info
Joseph Hickey, Vantage Systems, Inc.
The text was updated successfully, but these errors were encountered: