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 functional test to confirm OS_CountSemTimedWait waits up to the requested amount of time #991

Closed
skliper opened this issue May 7, 2021 · 0 comments · Fixed by #1026 or #1050
Closed

Add functional test to confirm OS_CountSemTimedWait waits up to the requested amount of time #991

skliper opened this issue May 7, 2021 · 0 comments · Fixed by #1026 or #1050
Assignees
@zanzaben
Labels
unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented May 7, 2021

Is your feature request related to a problem? Please describe.
There's only 2 checks, one for the OS_SEM_TIMEOUT return and one immediate success case. Nothing to confirm the passed in wait value is being honored.

Describe the solution you'd like
Add a time check (maybe as part of count-sem-test)

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added the unit-test Tickets related to the OSAL unit testing (functional and/or coverage) label May 7, 2021
@skliper skliper added this to the 6.0.0 milestone May 7, 2021
zanzaben added a commit to zanzaben/osal that referenced this issue May 12, 2021
@zanzaben
Fix nasa#991, Add count sem timeout test
edcd01b
zanzaben added a commit to zanzaben/osal that referenced this issue May 18, 2021
@zanzaben
Fix nasa#991, Add count sem timeout test
cb84b5e
@zanzaben zanzaben mentioned this issue May 18, 2021
Merged
zanzaben added a commit to zanzaben/osal that referenced this issue May 19, 2021
@zanzaben
Fix nasa#991, Add count sem timeout test
076164e
astrogeco added a commit that referenced this issue May 24, 2021
@astrogeco
Merge pull request #1026 from zanzaben/fix991_CountSemTimedWait_Test
8643a9c 
Fix #991, Add count sem timeout test
@astrogeco astrogeco mentioned this issue May 24, 2021
Merged
pepepr08 pushed a commit to pepepr08/osal that referenced this issue Jun 9, 2021
@zanzaben @pepepr08
Fix nasa#991, Add count sem timeout test
5d1c6d7
jphickey added a commit to jphickey/osal that referenced this issue Aug 10, 2022
@jphickey
HOTFIX nasa#991, correct doxygen reference
8935e89
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@astrogeco
Merge pull request nasa#991 from nasa/integration-candidate
9804b59 
Integration Candidate: 2020-11-03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zanzaben zanzaben

Labels
unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Projects
None yet
Milestone
Caelum
2 participants
@zanzaben @skliper