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

PTS: Test Framework: Bluetooth: SM/MAS/SIP/BV-02-C-INCONCLUSIV #22927

Closed
agata-ponitka opened this issue Feb 19, 2020 · 3 comments · Fixed by #23037
Closed

PTS: Test Framework: Bluetooth: SM/MAS/SIP/BV-02-C-INCONCLUSIV #22927

agata-ponitka opened this issue Feb 19, 2020 · 3 comments · Fixed by #23037
Assignees
@carlescufi
Labels
area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Milestone
v2.2.0

Comments

@agata-ponitka
Copy link
Collaborator

Describe the bug
Zephyr set IO capability to KeyboardOnly instead of KeyboardDisplay

To Reproduce
run auto-pts

Expected behavior
IO capability needs to be set to KeyboardDisplay

SM_MAS_SIP_BV-02-C-INCONCLUSIV.zip
@agata-ponitka agata-ponitka added the bug The issue is a bug, or the PR is fixing a bug label Feb 19, 2020
@aescolar
Copy link
Member

CC @jhedberg @joerchan

@aescolar aescolar added area: Bluetooth area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests priority: high High impact/importance bug labels Feb 19, 2020
@jhedberg
Copy link
Member

@aescolar this sounds more like a test setup error than a stack bug, do you agree?

@jhedberg
Copy link
Member

Is the high priority really justified? I just took a look at get_io_capa() in host/smp.c and it is able to return KeyboardOnly, so this looks more like an auto-pts or tester app issue, and should therefore not be high priority (since the stack poses no obstacle for qualification in this particular case)

@aescolar aescolar added priority: medium Medium impact/importance bug and removed priority: high High impact/importance bug labels Feb 19, 2020
@carlescufi carlescufi assigned agata-ponitka and unassigned joerchan Feb 20, 2020
@carlescufi carlescufi changed the title (PTS)Bluetooth: SM/MAS/SIP/BV-02-C-INCONCLUSIV PTS: Test Framework: Bluetooth: SM/MAS/SIP/BV-02-C-INCONCLUSIV Feb 20, 2020
@carlescufi carlescufi added this to the v2.2.0 milestone Feb 20, 2020
@carlescufi carlescufi mentioned this issue Feb 23, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@carlescufi carlescufi

Labels
area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth bug The issue is a bug, or the PR is fixing a bug priority: medium Medium impact/importance bug
Projects
None yet
Milestone
v2.2.0
Development

Successfully merging a pull request may close this issue.

Bluetooth: tester: Fix 3 GAP qualification tests carlescufi/zephyr
5 participants
@jhedberg @joerchan @carlescufi @aescolar @agata-ponitka