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

Local stub helper update to match macro behavior #814

Closed
skliper opened this issue Feb 16, 2021 · 0 comments · Fixed by #815 or #830
Closed

Local stub helper update to match macro behavior #814

skliper opened this issue Feb 16, 2021 · 0 comments · Fixed by #815 or #830
Labels
bug unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Feb 16, 2021 •
edited
Loading

Is your feature request related to a problem? Please describe.
Custom call to UT_DefaultStubImpl don't match standard macro behavior (NULL terminate for functions not including va list).

Describe the solution you'd like
Match macro (avoids static analysis warning)

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman
@skliper skliper added the unit-test Tickets related to the OSAL unit testing (functional and/or coverage) label Feb 16, 2021
@skliper skliper added this to the 6.0.0 milestone Feb 16, 2021
@skliper skliper changed the title Simplify local UT stubs to use macros Local stub helper update to match macro behavior Feb 16, 2021
skliper added a commit to skliper/osal that referenced this issue Feb 16, 2021
@skliper
Fix nasa#814, UT Stub match macro behavior
1f60c85
@skliper skliper mentioned this issue Feb 16, 2021
Merged
@astrogeco astrogeco added the bug label Feb 24, 2021
astrogeco added a commit that referenced this issue Feb 24, 2021
@astrogeco
Merge pull request #815 from skliper/fix814-stub_helper
58823bb 
Fix #814, UT Stub match macro behavior
@astrogeco astrogeco mentioned this issue Feb 24, 2021
Merged
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@zanzaben
Fix nasa#814, add ES Misc Functional test
285cd12
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@zanzaben
Fix nasa#814, add additional vararg tests
04fbc9a
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
@astrogeco
Merge pull request #1586 from zanzaben/fix814_ES_Misc_Functional_Test
75a45fc 
Fix nasa#814, add ES Misc Functional test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Projects
None yet
Milestone
Caelum
2 participants
@skliper @astrogeco