Skip to content

Conversation

@alexsven
Copy link
Contributor

  • When we are a peripheral we want to be able to start a stream from the stream_connected_cb, given that we are in an enabling state and the stream is in the sink direction

@alexsven alexsven changed the title tests: bluetooth: tester: Start stream from connected_cb tests: bluetooth: tester: Start stream from stream_connected_cb Nov 27, 2025
@zephyrbot zephyrbot added area: Tests Issues related to a particular existing or missing test area: Bluetooth area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth Audio labels Nov 27, 2025
@Thalley
Copy link
Contributor

Thalley commented Nov 27, 2025

Bug was introduced by #99597 that only handled the case for the peripheral/server in the enabled callback, and was missing the connected callback

Copy link
Contributor

@Thalley Thalley left a comment

Choose a reason for hiding this comment

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

Minor comments

@Thalley
Copy link
Contributor

Thalley commented Nov 27, 2025

#AutoPTS run zephyr BAP/USR/SPE/BI-01-C BAP/USR/SPE/BI-02-C BAP/USR/SPE/BI-03-C BAP/USR/SPE/BI-05-C BAP/USR/STR/BV-363-C BAP/USR/STR/BV-364-C BAP/USR/STR/BV-368-C BAP/USR/STR/BV-376-C BAP/USR/STR/BV-377-C

@codecoup-tester
Copy link

Scheduled PR #100162 (comment), board: nrf54, estimated start time: 13:05:00, test case count: 9, estimated duration: 0:18:04

Test cases to be runBAP/USR/STR/BV-368-C
BAP/USR/STR/BV-376-C
BAP/USR/STR/BV-377-C
BAP/USR/STR/BV-363-C
BAP/USR/STR/BV-364-C
BAP/USR/SPE/BI-01-C
BAP/USR/SPE/BI-02-C
BAP/USR/SPE/BI-03-C
BAP/USR/SPE/BI-05-C

- When we are a peripheral we want to be able to start a stream
  from the stream_connected_cb, given that we are in an enabling state
  and the stream is in the sink direction

Signed-off-by: Alexander Svensen <alexander.svensen@nordicsemi.no>
@alexsven alexsven force-pushed the bugfix_start_stream_from_connected_cb branch from f4988ed to 6569214 Compare November 27, 2025 12:10
@alexsven alexsven requested a review from Thalley November 27, 2025 12:10
@codecoup-tester
Copy link

AutoPTS Bot results:
No failed test found.

Successful tests (9)BAP BAP/USR/SPE/BI-01-C PASS
BAP BAP/USR/SPE/BI-02-C PASS
BAP BAP/USR/SPE/BI-03-C PASS
BAP BAP/USR/SPE/BI-05-C PASS
BAP BAP/USR/STR/BV-363-C PASS
BAP BAP/USR/STR/BV-364-C PASS
BAP BAP/USR/STR/BV-368-C PASS
BAP BAP/USR/STR/BV-376-C PASS
BAP BAP/USR/STR/BV-377-C PASS

@sonarqubecloud
Copy link

Copy link
Contributor

@asbjornsabo asbjornsabo left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Bluetooth Audio area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth area: Tests Issues related to a particular existing or missing test

Projects

Status: In Review

Development

Successfully merging this pull request may close these issues.

5 participants