Skip to content

tests: Bluetooth: Tester: VCP BSIM test #86664

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

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

Conversation

Thalley
Copy link
Collaborator

@Thalley Thalley commented Mar 5, 2025

Adds BSIM testing of the VCP features of the BT Tester.

Part of #86073

@Thalley Thalley force-pushed the bttester_bsim_vcp branch 5 times, most recently from 3532311 to b817f40 Compare March 6, 2025 13:32
@Thalley Thalley force-pushed the bttester_bsim_vcp branch 3 times, most recently from 12dd064 to 44c4b1b Compare March 17, 2025 09:11
@Thalley Thalley force-pushed the bttester_bsim_vcp branch 3 times, most recently from 9e0b85c to 1aa846b Compare March 27, 2025 14:40
@Thalley Thalley force-pushed the bttester_bsim_vcp branch from 1aa846b to a87222f Compare April 1, 2025 12:18
@Thalley Thalley requested a review from Copilot April 7, 2025 08:59
@Thalley Thalley force-pushed the bttester_bsim_vcp branch from a87222f to d8f12f3 Compare April 7, 2025 08:59
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 14 out of 26 changed files in this pull request and generated no comments.

Files not reviewed (12)
  • tests/bluetooth/tester/Kconfig.sysbuild: Language not supported
  • tests/bluetooth/tester/boards/nrf52_bsim.conf: Language not supported
  • tests/bluetooth/tester/boards/nrf52_bsim.overlay: Language not supported
  • tests/bluetooth/tester/boards/nrf5340bsim_nrf5340_cpuapp.conf: Language not supported
  • tests/bluetooth/tester/boards/nrf5340bsim_nrf5340_cpuapp.overlay: Language not supported
  • tests/bsim/bluetooth/compile.nrf5340bsim_nrf5340_cpuapp.sh: Language not supported
  • tests/bsim/bluetooth/compile.sh: Language not supported
  • tests/bsim/bluetooth/tester/CMakeLists.txt: Language not supported
  • tests/bsim/bluetooth/tester/Kconfig: Language not supported
  • tests/bsim/bluetooth/tester/boards/nrf52_bsim.overlay: Language not supported
  • tests/bsim/bluetooth/tester/compile.sh: Language not supported
  • tests/bsim/bluetooth/tester/prj.conf: Language not supported

@Thalley Thalley force-pushed the bttester_bsim_vcp branch 2 times, most recently from e48ab9f to 43accc8 Compare April 8, 2025 09:04
@Thalley Thalley marked this pull request as ready for review April 8, 2025 09:04
@Thalley Thalley self-assigned this Apr 8, 2025
@github-actions github-actions bot added platform: nRF BSIM Nordic Semiconductors, nRF BabbleSim area: Process area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests labels Apr 8, 2025
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Looks like the change to this file does not apply when assigning reviewers and labels

Copy link
Member

Choose a reason for hiding this comment

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

Correct. Only the maintainers file owner itself is notified.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That's pretty interesting, because that means there's a chicken-and-egg situation here, right?

We cannot files to MAINTAINERS.yml before they exist, and the file isn't applied when added together :)

It's not a big issue, but was a bit surprising as generally changes to files in a PR is applied to CI

@Thalley Thalley force-pushed the bttester_bsim_vcp branch from 43accc8 to c5a3ce4 Compare April 9, 2025 12:03
@Thalley Thalley force-pushed the bttester_bsim_vcp branch 3 times, most recently from dc4169f to e22626d Compare May 7, 2025 11:35
Adds BSIM testing of the VCP features of the BT Tester.

Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
@Thalley Thalley force-pushed the bttester_bsim_vcp branch from e22626d to 6e122cf Compare May 19, 2025 08:54
Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bluetooth Audio area: Bluetooth Host Bluetooth Host (excluding BR/EDR) area: Bluetooth Qualification Bluetooth Qualification -related issues and pull requests area: Bluetooth area: Process platform: nRF BSIM Nordic Semiconductors, nRF BabbleSim
Projects
Status: In Review
Development

Successfully merging this pull request may close these issues.

3 participants