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

Fix #1825, #1826, #1829, #1830, SB test improvements #1876

Merged
merged 4 commits into from
Aug 27, 2021
Merged

Fix #1825, #1826, #1829, #1830, SB test improvements #1876

merged 4 commits into from
Aug 27, 2021

Commits on Aug 24, 2021

  1. Fix nasa#1829, confirm CFE_SB_PEND_FOREVER flag 

    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.
    @jphickey
    jphickey committed Aug 24, 2021
    Configuration menu
    Copy the full SHA
    5c388e3 View commit details
    Browse the repository at this point in the history

  2. Fix nasa#1826, Add testcases for message broadcasting 

    Add test cases to confirm that when multiple pipes are subscribed
to the same message ID, that:

1. Broadcasting (transmit side) is successful even if some of the pipes
   are not deliverable.
2. MsgLimit (from subscription) and PipeDepth (from pipe create) are honored.
3. If some pipes are undeliverable for any reason, delivery to open pipes is
   not affected.
    @jphickey
    jphickey committed Aug 24, 2021
    Configuration menu
    Copy the full SHA
    7170dff View commit details
    Browse the repository at this point in the history

  3. Fix nasa#1825, test unsubscribe from single pipe 

    Extends the broadcast test case to cover unsubscription of a single pipe ID.
Confirms that other pipes/subscriptions are not affected, and that the limits
initially enforced on those original subscriptions still apply.
    @jphickey
    jphickey committed Aug 24, 2021
    Configuration menu
    Copy the full SHA
    b108e7e View commit details
    Browse the repository at this point in the history

  4. Fix nasa#1830, add sequence number validation to zero copy test 

    Add test cases to the zero copy sequence to confirm proper override
and/or increment of the sequence count.
    @jphickey
    jphickey committed Aug 24, 2021
    Configuration menu
    Copy the full SHA
    7e7f5a5 View commit details
    Browse the repository at this point in the history