michaelh/zephyr
1
0
Fork 0
Code Releases Activity

Bluetooth: Test: Update ccc_store bsim test

Browse source 
Update `ccc_store` bsim test to remove the 'expected failure'. The issue
being fixed in the following commit.

Signed-off-by: Théo Battrel <theo.battrel@nordicsemi.no>
This commit is contained in:
Théo Battrel 2023-06-02 15:24:12 +02:00 • committed by Anas Nashif
commit cb48626399
2 changed files with 2 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

7
tests/bsim/bluetooth/host/gatt/ccc_store/prj_2.conf
View file

@ -21,10 +21,5 @@ CONFIG_SETTINGS_NVS=y
CONFIG_ASSERT=y
# By doing so, BT_SETTINGS_DELAYED_STORE will be enabled because
# BT_SETTINGS_CF_STORE_ON_WRITE is enabled by default and selecting it.
# This cause the CCC to not be stored at all.
# If BT_SETTINGS_CCC_LAZY_LOADING is enabled, the test will pass because the
# CCC will not be cleared after the first disconnection and will then still be
# in RAM.
# Test that CCC is stored even if BT_SETTINGS_DELAYED_STORE is enabled
CONFIG_BT_SETTINGS_CCC_STORE_ON_WRITE=n

7
tests/bsim/bluetooth/host/gatt/ccc_store/src/peripheral.c
View file

@ -197,12 +197,7 @@ static void connect_restore_sec_check_subscribtion(struct bt_le_ext_adv *adv)
/* check that subscribtion has been restored */
if (!is_peer_subscribed(default_conn)) {
if (!IS_ENABLED(CONFIG_BT_SETTINGS_CCC_STORE_ON_WRITE)) {
LOG_ERR("Expected failure: client is not subscribed. See issue #58471 for "
"more details.");
} else {
FAIL("Client is not subscribed\n");
}
FAIL("Client is not subscribed\n");
} else {
LOG_DBG("Client is subscribed");
}