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

Remove UT_CheckForOpenSockets #753

Closed
skliper opened this issue Jan 11, 2021 · 0 comments · Fixed by #754 or #767
Closed

Remove UT_CheckForOpenSockets #753

skliper opened this issue Jan 11, 2021 · 0 comments · Fixed by #754 or #767
Labels
removed unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Jan 11, 2021

Is your feature request related to a problem? Please describe.
See also nasa/cFE#901, was used to check that unit tests closed sockets but is no longer applicable.

Describe the solution you'd like
Remove

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added enhancement unit-test Tickets related to the OSAL unit testing (functional and/or coverage) labels Jan 11, 2021
@skliper skliper added this to the 6.0.0 milestone Jan 11, 2021
skliper added a commit to skliper/osal that referenced this issue Jan 11, 2021
@skliper
Fix nasa#753, Remove UT_CheckForOpenSockets
c968de0
This was referenced Jan 11, 2021
Merged
Merged
astrogeco added a commit that referenced this issue Jan 25, 2021
@astrogeco
Merge pull request #754 from skliper/fix753-rm_ut_checkforopensockets
95cab23 
Fix #753, Remove UT_CheckForOpenSockets
@astrogeco astrogeco mentioned this issue Jan 26, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
removed unit-test Tickets related to the OSAL unit testing (functional and/or coverage)
Projects
None yet
Milestone
Caelum
2 participants
@skliper @astrogeco