Bluetooth: CCAP bsim test multiple acceptors #85222
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The cap_broadcast test has been split into 3 tests: 1) Regular test 2) Regular test with metadata update 3) Test for invalid parameters This will make it easier to use the regular cap_broadcast test for other test cases, as it does less now. It also makes each test run faster seperately and reduce their scopes. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Modify stream_rx.c so that the calls to FAIL are guarded by the newly added valid_rx_cnt. This helps prevent timing issues, especially with broadcast, where the first SDU(s) may be delayed from the application, and thus may be missed/contain errors. Now it will only treat missing or error SDUs as a FAIL if we have received a valid SDU. All fails, will still be logged. We now also log both valid and total count. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Expand the CAP broadcast reception test to also run with multiple acceptors to better verify correctness in the procedures. For this to run more stable, the maximum number of streams has been reduce to 2, rather than CONFIG_BT_BAP_BROADCAST_SRC_STREAM_COUNT. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
Thalley
force-pushed
the
cap_bsim_test_dual_acc
branch
from
45e915c to
3ba270e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds testing of multiple acceptors in the CAP broadcast reception test.
Required before #82483 can be implemented