SB functional test - exercise CFE_SB_ReceiveBuffer CFE_SB_PEND_FOREVER and CFE_SB_POLL options #1829
Comments
|
I actually added a couple calls to ReceiveBuffer w/CFE_SB_POLL as part of #1812. I can add a call w/CFE_SB_PEND_FOREVER - didn't do this at first because if the queue is empty, that would hang the test, but that would also be a failure so probably ok? Obviously won't be able to test the case of PEND_FOREVER + empty queue.... |
|
Great! Please add a PEND_FOREVER call with a message on the bus so it returns immediately, no need for a test that actually pends forever. It'd also be good for the final test where it does pend for a short timeout to actually confirm at least a minimum amount of time passed. |
jphickey
added a commit
to jphickey/cFE
that referenced
this issue
Aug 24, 2021
Use the CFE_SB_PEND_FOREVER flag on a few of the calls to CFE_SB_ReceiveBuffer to confirm that this works as expected. Previously only CFE_SB_POLL and a nonzero timeout were used. Note this can only be used when it is known/expected that the message queue is not empty. If there is any possiblility that the queue is empty, then this cannot be used or else the test may block. This also means that if the test fails, it may fail by blocking and never finishing the test rather than an actual FAIL testcase.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is your feature request related to a problem? Please describe.
SB functional test only calls CFE_SB_ReceiveBuffer with a timeout of 100, requirements include poll and pend forever.
Describe the solution you'd like
Exercise CFE_SB_POLL and CFE_SB_PEND_FOREVER options.
Describe alternatives you've considered
None
Additional context
None
Requester Info
Jacob Hageman - NASA/GSFC
The text was updated successfully, but these errors were encountered: