tests: Bluetooth: BR: Add test suite sdp_server #81769
Merged
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
lylezhu2012
force-pushed
the
pytest_evaluation
branch
from
484f20f to
654e7d6
Compare
lylezhu2012
force-pushed
the
pytest_evaluation
branch
4 times, most recently
from
0513d1a to
3775d00
Compare
lylezhu2012
changed the title
lylezhu2012
force-pushed
the
pytest_evaluation
branch
2 times, most recently
from
155380c to
35d383d
Compare
lylezhu2012
changed the title
lylezhu2012
force-pushed
the
pytest_evaluation
branch
2 times, most recently
from
b270ea5 to
0f27a88
Compare
jhedberg
requested changes
Dec 4, 2024
lylezhu2012
force-pushed
the
pytest_evaluation
branch
from
0f27a88 to
875a68c
Compare
zephyrbot
added
area: Bluetooth
area: Bluetooth Host
zephyrbot
requested review from
alwa-nordic,
hermabe,
rugeGerritsen,
sjanc,
Thalley and
theob-pro
lylezhu2012
force-pushed
the
pytest_evaluation
branch
from
875a68c to
917e1c2
Compare
lylezhu2012
force-pushed
the
pytest_evaluation
branch
2 times, most recently
from
d84a0fe to
03bcc7b
Compare
lylezhu2012
force-pushed
the
pytest_evaluation
branch
from
03bcc7b to
ccad59d
Compare
|
hi @lylezhu2012 , I tried this pr on the local with HW board setup (RT1170 + rtk usb dongle),
but could work with legacy bumble version, 0.0.203.dev8+g01dc0d5, as you shared. |
lylezhu2012
force-pushed
the
pytest_evaluation
branch
from
ccad59d to
fceb5a8
Compare
It is caused by the upgrade of bumble. It has been fixed now. The version 0.0.208 of bumble can work properly. |
|
@jhedberg Do you have any comments of this PR? I saw there is a "Requested changes" from you. And I have updated the folder path |
lylezhu2012
force-pushed
the
pytest_evaluation
branch
4 times, most recently
from
0fe452b to
96cf8a1
Compare
lylezhu2012
force-pushed
the
pytest_evaluation
branch
from
96cf8a1 to
96face6
Compare
jhedberg
requested changes
Mar 18, 2025
| @@ -0,0 +1,15 @@ | |||
| tests: | |||
| sdp.s: | |||
There was a problem hiding this comment.
This should be bluetooth.sdp.server for clarity and consistency. You could even consider creating a bluetooth.classic.* namespace for all Bluetooth Classic tests.
lylezhu2012
force-pushed
the
pytest_evaluation
branch
2 times, most recently
from
ca86f74 to
0037e2e
Compare
IUT works as a SDP Server. The peer device, SDP client, is a PC with running `bumble` on it. Define 10 SDP records. 8 normal SDP records. one SDP record with large service name that cannot be discovered by peer since the data length exceeds the MTU of SDP. And one SDP record with large service name that can be discovered by peer but `continuation state` needs to be involved. By default, no SDP record will be registered. Add shell command `register_sdp` to register specific SDP record. Add shell command `register_sdp_all` to register all normal SDP records. Add shell command `register_sdp_large` to register SDP record with large service name that cannot be discovered by peer. Add shell command `register_sdp_large_valid` to register SDP record with large service name that can be discovered by peer. In the test suite, there two test cases, Case 1, discover the IUT. Case 2, connect to the IUT. And send serial SDP discovery requests. Signed-off-by: Lyle Zhu <lyle.zhu@nxp.com>
lylezhu2012
force-pushed
the
pytest_evaluation
branch
from
0037e2e to
4cfc4b4
Compare
|
Remove the using of |
jhedberg
approved these changes
Mar 24, 2025
makeshi
approved these changes
Mar 24, 2025
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.
IUT works as a SDP Server. The peer device, SDP client, is a PC with running
bumbleon it.Define 10 SDP records. 8 normal SDP records. one SDP record with large service name that cannot be discovered by peer since the data length exceeds the MTU of SDP. And one SDP record with large service name that can be discovered by peer but
continuation stateneeds to be involved.By default, no SDP record will be registered.
Add shell command
register_sdpto register specific SDP record. Add shell commandregister_sdp_allto register all normal SDP records.Add shell command
register_sdp_largeto register SDP record with large service name that cannot be discovered by peer. Add shell commandregister_sdp_large_validto register SDP record with large service name that can be discovered by peer.In the test suite, there two test cases,
Case 1, discover the IUT.
Case 2, connect to the IUT. And send serial SDP discovery request.