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.

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

tests: Bluetooth: BR: Add test suite sdp_server #81769

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lylezhu2012
Copy link
Contributor

@lylezhu2012 lylezhu2012 commented Nov 22, 2024

  • tests: Bluetooth: BR: Add test suite sdp_server

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 request.

@lylezhu2012 lylezhu2012 requested a review from jhedberg November 25, 2024 01:42
@lylezhu2012 lylezhu2012 self-assigned this Nov 25, 2024
@lylezhu2012 lylezhu2012 force-pushed the pytest_evaluation branch 4 times, most recently from 0513d1a to 3775d00 Compare November 28, 2024 07:02
@lylezhu2012 lylezhu2012 changed the title tests: BR: Add test suite sdp_server tests: Bluetooth: BR: Add test suite sdp_server and sdp_client Nov 28, 2024
@lylezhu2012 lylezhu2012 force-pushed the pytest_evaluation branch 2 times, most recently from 155380c to 35d383d Compare November 29, 2024 07:27
@lylezhu2012 lylezhu2012 changed the title tests: Bluetooth: BR: Add test suite sdp_server and sdp_client tests: Bluetooth: BR: Add test suite sdp_server Nov 29, 2024
@lylezhu2012 lylezhu2012 requested a review from dleach02 November 29, 2024 08:33
@lylezhu2012 lylezhu2012 force-pushed the pytest_evaluation branch 2 times, most recently from b270ea5 to 0f27a88 Compare December 4, 2024 06:07
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tests/classic is not correct. Any Bluetooth tests must go under tests/bluetooth. I saw your comment in the other PR about Windows apparently having some limit on full path length for object files, however whatever the solution is going to be this is not it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Btw, this is also the reason why this PR didn't get any automatic labels, reviewers or assignees.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well. Updated the path.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is done.

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 request.

Signed-off-by: Lyle Zhu <[email protected]>
@zephyrbot zephyrbot requested a review from cvinayak December 6, 2024 09:58
@cvinayak cvinayak removed their request for review December 6, 2024 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants