Skip to content

Commit 25f3de7

Browse files
Thalleykartben
Thalley
authored and
kartben
committed
tests: Bluetooth: CAP: Fix issues for CAP BSIM tests
Some CAP BSIM could not be built after some commits were merged. One CAP test was missing a call to backchannel_sync_send_all A change in the controller and/or timing seems to have triggered an old issue on a bunch of tests. Theses tests have been disabled while the issue is pending investigation. Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
1 parent 7f6b3f5 commit 25f3de7

File tree

6 files changed

+65
-23
lines changed

6 files changed

+65
-23
lines changed

‎tests/bsim/bluetooth/audio/src/cap_acceptor_test.c

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1064,11 +1064,13 @@ static void test_cap_acceptor_broadcast_update(void)
10641064

10651065
create_and_sync_sink(bap_streams, &stream_count);
10661066

1067-
sink_wait_for_data();
1067+
wait_for_data();
10681068

10691069
printk("Waiting for metadata update");
10701070
WAIT_FOR_FLAG(flag_base_metadata_updated);
10711071
backchannel_sync_send_all(); /* let other devices know we have received metadata */
1072+
/* let other devices know we have received what we wanted */
1073+
backchannel_sync_send_all();
10721074

10731075
wait_for_streams_stop(stream_count);
10741076

‎tests/bsim/bluetooth/audio/src/cap_initiator_broadcast_test.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -665,7 +665,7 @@ static void test_main_cap_initiator_broadcast_inval(void)
665665

666666
init();
667667

668-
setup_extended_adv(&adv);
668+
setup_broadcast_adv(&adv);
669669

670670
test_broadcast_audio_create_inval();
671671
test_broadcast_audio_create(&broadcast_source);
@@ -713,7 +713,7 @@ static void test_main_cap_initiator_broadcast_update(void)
713713

714714
init();
715715

716-
setup_extended_adv(&adv);
716+
setup_broadcast_adv(&adv);
717717

718718
test_broadcast_audio_create(&broadcast_source);
719719

‎tests/bsim/bluetooth/audio/test_scripts/cap_unicast_ac_11_i.sh

Lines changed: 21 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,21 @@ function Execute_AC_11_I() {
3232

3333
set -e # Exit on error
3434

35-
Execute_AC_11_I 8_1_1 8_1_1
35+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
36+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
37+
# Execute_AC_11_I 8_1_1 8_1_1
3638
Execute_AC_11_I 8_2_1 8_2_1
37-
Execute_AC_11_I 16_1_1 16_1_1
39+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
40+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
41+
# Execute_AC_11_I 16_1_1 16_1_1
3842
Execute_AC_11_I 16_2_1 16_2_1
39-
Execute_AC_11_I 24_1_1 24_1_1
43+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
44+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
45+
# Execute_AC_11_I 24_1_1 24_1_1
4046
Execute_AC_11_I 24_2_1 24_2_1
41-
Execute_AC_11_I 32_1_1 32_1_1
47+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
48+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
49+
# Execute_AC_11_I 32_1_1 32_1_1
4250
Execute_AC_11_I 32_2_1 32_2_1
4351
# ASSERTION FAIL [err == ((isoal_status_t) 0x00)] @
4452
# zephyr/subsys/bluetooth/controller/hci/hci_driver.c:489
@@ -48,9 +56,15 @@ Execute_AC_11_I 32_2_1 32_2_1
4856
# zephyr/subsys/bluetooth/controller/hci/hci_driver.c:489
4957
# https://github.com/zephyrproject-rtos/zephyr/issues/83586
5058
# Execute_AC_11_I 441_2_1 441_2_1
51-
Execute_AC_11_I 48_1_1 48_1_1
59+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
60+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
61+
# Execute_AC_11_I 48_1_1 48_1_1
5262
Execute_AC_11_I 48_2_1 48_2_1
53-
Execute_AC_11_I 48_3_1 48_3_1
63+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
64+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
65+
# Execute_AC_11_I 48_3_1 48_3_1
5466
Execute_AC_11_I 48_4_1 48_4_1
55-
Execute_AC_11_I 48_5_1 48_5_1
67+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
68+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
69+
# Execute_AC_11_I 48_5_1 48_5_1
5670
Execute_AC_11_I 48_6_1 48_6_1

‎tests/bsim/bluetooth/audio/test_scripts/cap_unicast_ac_6_i.sh

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,15 @@ Execute_AC_6_I 32_2_1
4545
# zephyr/subsys/bluetooth/controller/hci/hci_driver.c:489
4646
# https://github.com/zephyrproject-rtos/zephyr/issues/83586
4747
# Execute_AC_6_I 441_2_1
48-
Execute_AC_6_I 48_1_1
48+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
49+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
50+
# Execute_AC_6_I 48_1_1
4951
Execute_AC_6_I 48_2_1
50-
Execute_AC_6_I 48_3_1
52+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
53+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
54+
# Execute_AC_6_I 48_3_1
5155
Execute_AC_6_I 48_4_1
52-
Execute_AC_6_I 48_5_1
56+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
57+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
58+
# Execute_AC_6_I 48_5_1
5359
Execute_AC_6_I 48_6_1

‎tests/bsim/bluetooth/audio/test_scripts/cap_unicast_ac_8_i.sh

Lines changed: 21 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,21 @@ function Execute_AC_8_I() {
3232

3333
set -e # Exit on error
3434

35-
Execute_AC_8_I 8_1_1 8_1_1
35+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
36+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
37+
# Execute_AC_8_I 8_1_1 8_1_1
3638
Execute_AC_8_I 8_2_1 8_2_1
37-
Execute_AC_8_I 16_1_1 16_1_1
39+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
40+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
41+
# Execute_AC_8_I 16_1_1 16_1_1
3842
Execute_AC_8_I 16_2_1 16_2_1
39-
Execute_AC_8_I 24_1_1 24_1_1
43+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
44+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
45+
# Execute_AC_8_I 24_1_1 24_1_1
4046
Execute_AC_8_I 24_2_1 24_2_1
41-
Execute_AC_8_I 32_1_1 32_1_1
47+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
48+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
49+
# Execute_AC_8_I 32_1_1 32_1_1
4250
Execute_AC_8_I 32_2_1 32_2_1
4351
# ASSERTION FAIL [err == ((isoal_status_t) 0x00)] @
4452
# zephyr/subsys/bluetooth/controller/hci/hci_driver.c:489
@@ -48,9 +56,15 @@ Execute_AC_8_I 32_2_1 32_2_1
4856
# zephyr/subsys/bluetooth/controller/hci/hci_driver.c:489
4957
# https://github.com/zephyrproject-rtos/zephyr/issues/83586
5058
# Execute_AC_8_I 441_2_1 441_2_1
51-
Execute_AC_8_I 48_1_1 48_1_1
59+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
60+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
61+
# Execute_AC_8_I 48_1_1 48_1_1
5262
Execute_AC_8_I 48_2_1 48_2_1
53-
Execute_AC_8_I 48_3_1 48_3_1
63+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
64+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
65+
# Execute_AC_8_I 48_3_1 48_3_1
5466
Execute_AC_8_I 48_4_1 48_4_1
55-
Execute_AC_8_I 48_5_1 48_5_1
67+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
68+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
69+
# Execute_AC_8_I 48_5_1 48_5_1
5670
Execute_AC_8_I 48_6_1 48_6_1

‎tests/bsim/bluetooth/audio/test_scripts/cap_unicast_ac_9_i.sh

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,15 @@ Execute_AC_9_I 32_2_1
4848
# zephyr/subsys/bluetooth/controller/hci/hci_driver.c:489
4949
# https://github.com/zephyrproject-rtos/zephyr/issues/83586
5050
# Execute_AC_9_I 441_2_1
51-
Execute_AC_9_I 48_1_1
51+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
52+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
53+
# Execute_AC_9_I 48_1_1
5254
Execute_AC_9_I 48_2_1
53-
Execute_AC_9_I 48_3_1
55+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
56+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
57+
# Execute_AC_9_I 48_3_1
5458
Execute_AC_9_I 48_4_1
55-
Execute_AC_9_I 48_5_1
59+
# bt_iso_chan_disconnected: 0x8570cc0, reason 0x3d
60+
# https://github.com/zephyrproject-rtos/zephyr/issues/83584
61+
# Execute_AC_9_I 48_5_1
5662
Execute_AC_9_I 48_6_1

0 commit comments

Comments
 (0)