search

zephyrproject-rtos / zephyr

Primary Git Repository for the Zephyr Project. Zephyr is a new generation, scalable, optimized, secure RTOS for multiple hardware architectures.
https://docs.zephyrproject.org
Apache License 2.0
10.01k stars 6.16k forks source link

LE Audio: CCP invalid unit tests #51384

Open Thalley opened 1 year ago

Thalley commented 1 year ago

Is your enhancement proposal related to a problem? Please describe. The current babblesim tests for CCP mainly focuses on the case where everything goes right, and do not test much invalid behavior or error handling.

Describe the solution you'd like All APIs should be tested with invalid values to verify that they will return an error.

To the extent possible, the APIs should also be used in cases where the the remote device should perform some error handling.

Finally the APIs should be tested with disconnects.

Describe alternatives you've considered To test against a non-conforming device, some verification and checks in the stack needs to be disabled. TBD if this is a good idea to test somehow.

Additional context Depends on https://github.com/zephyrproject-rtos/zephyr/issues/41210

zephyrbot commented 4 months ago

Hi @jhedberg, @Thalley, @nashif,

This issue, marked as an Enhancement, was opened a while ago and did not get any traction. It was just assigned to you based on the labels. If you don't consider yourself the right person to address this issue, please re-assing it to the right person.

Please take a moment to review if the issue is still relevant to the project. If it is, please provide feedback and direction on how to move forward. If it is not, has already been addressed, is a duplicate, or is no longer relevant, please close it with a short comment explaining the reason.

Thanks!

Thalley commented 4 months ago

This is still valid