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

ut_sb_stubs.c needs a stub for CFE_SB_DeletePipe #704

Closed
asgibson opened this issue May 14, 2020 · 1 comment · Fixed by #714 or #729
Closed

ut_sb_stubs.c needs a stub for CFE_SB_DeletePipe #704

asgibson opened this issue May 14, 2020 · 1 comment · Fixed by #714 or #729
Labels
enhancement unit-test
Milestone
Bootes

Comments

@asgibson
Copy link
Contributor

Is your feature request related to a problem? Please describe.
CFE_SB_DeletePipe does not have a stub in ut_sb_stubs.c causing undefined reference error when building unit tests for an app that uses this call.

Describe the solution you'd like
Add the stub for CFE_SB_DeletePipe with full context tracking.

Describe alternatives you've considered
Create my own locally in the app's unit test, but this does not help everyone.

Requester Info
Alan Gibson, NASA
@skliper skliper added this to the 6.8.0 milestone May 14, 2020
@skliper
Copy link
Contributor

skliper commented May 14, 2020

@AlanSGibson - we encourage submissions... can you add your implementation to ut_sb_stubs.c and submit a pull request?

asgibson added a commit to asgibson/cFE that referenced this issue May 21, 2020
@asgibson
Fix nasa#704. Added stub for CFE_SB_DeletePipe in ut_sb_stubs.c
91aacd8
@asgibson asgibson mentioned this issue May 21, 2020
Merged
astrogeco added a commit that referenced this issue Jun 2, 2020
@astrogeco
Merge pull request #714 from asgibson/fix_#704
086bf1b 
Fix #704, Added stub for CFE_SB_DeletePipe in ut_sb_stubs.c
@astrogeco astrogeco mentioned this issue Jun 9, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement unit-test
Projects
None yet
Milestone
Bootes
2 participants
@skliper @asgibson