espressif/esp-nimble
1
0
Fork 0
mirror of https://github.com/espressif/esp-nimble.git synced 2026-06-05 21:04:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

nimble/host: Fix L2CAP unit tests

Browse Source 
This fixes L2CAP unit tests after latest rework.
This commit is contained in:
Andrzej Kaczmarek
2025-04-08 11:55:44 +02:00
committed by Rahul Tank
parent 4f5675f517
commit f6bb6468e2
2 changed files with 13 additions and 43 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nimble/host/test/src/ble_hs_test_util.c
+6 -32
View File
@@ -104,7 +104,7 @@ ble_hs_test_util_prev_tx_dequeue(void)
pb = BLE_HCI_DATA_PB(hci_hdr.hdh_handle_pb_bc);
TEST_ASSERT_FATAL(pb == BLE_HCI_PB_FIRST_NON_FLUSH);
rc = ble_l2cap_parse_hdr(om, 0, &l2cap_hdr);
rc = ble_l2cap_parse_hdr(om, &l2cap_hdr);
TEST_ASSERT_FATAL(rc == 0);
os_mbuf_adj(om, BLE_L2CAP_HDR_SZ);
@@ -670,36 +670,13 @@ ble_hs_test_util_l2cap_rx(uint16_t conn_handle,
struct hci_data_hdr *hci_hdr,
struct os_mbuf *om)
{
struct ble_hs_conn *conn;
ble_l2cap_rx_fn *rx_cb;
int reject_cid;
uint8_t pb;
int rc;
ble_hs_lock();
conn = ble_hs_conn_find(conn_handle);
if (conn != NULL) {
rc = ble_l2cap_rx(conn, hci_hdr, om, &rx_cb, &reject_cid);
} else {
rc = os_mbuf_free_chain(om);
TEST_ASSERT_FATAL(rc == 0);
}
ble_hs_unlock();
if (conn == NULL) {
rc = BLE_HS_ENOTCONN;
} else if (rc == 0) {
TEST_ASSERT_FATAL(rx_cb != NULL);
rc = rx_cb(conn->bhc_rx_chan);
ble_l2cap_remove_rx(conn, conn->bhc_rx_chan);
} else if (rc == BLE_HS_EAGAIN) {
/* More fragments on the way. */
pb = BLE_HCI_DATA_PB(hci_hdr->hdh_handle_pb_bc);
rc = ble_l2cap_rx(conn_handle, pb, om);
if (rc == BLE_HS_EAGAIN) {
rc = 0;
} else {
if (reject_cid != -1) {
ble_l2cap_sig_reject_invalid_cid_tx(conn_handle, 0, 0, reject_cid);
}
}
return rc;
@@ -1787,7 +1764,6 @@ ble_hs_test_util_mbuf_count(const struct ble_hs_test_util_mbuf_params *params)
{
const struct ble_att_prep_entry *prep;
const struct os_mbuf_pkthdr *omp;
const struct ble_l2cap_chan *chan;
const struct ble_hs_conn *conn;
const struct os_mbuf *om;
int count;
@@ -1813,9 +1789,7 @@ ble_hs_test_util_mbuf_count(const struct ble_hs_test_util_mbuf_params *params)
}
if (params->rx_queue) {
SLIST_FOREACH(chan, &conn->bhc_channels, next) {
count += ble_hs_test_util_mbuf_chain_len(chan->rx_buf);
}
count += ble_hs_test_util_mbuf_chain_len(conn->rx_frags);
}
if (params->prep_list) {
nimble/host/test/src/ble_l2cap_test.c
+7 -11
View File
@@ -124,7 +124,7 @@ ble_l2cap_test_util_verify_tx_update_conn(
}
static int
ble_l2cap_test_util_dummy_rx(struct ble_l2cap_chan *chan)
ble_l2cap_test_util_dummy_rx(struct ble_l2cap_chan *chan, struct os_mbuf **om)
{
return 0;
}
@@ -220,8 +220,7 @@ ble_l2cap_test_util_verify_first_frag(uint16_t conn_handle,
conn = ble_hs_conn_find(conn_handle);
TEST_ASSERT_FATAL(conn != NULL);
TEST_ASSERT(conn->bhc_rx_chan != NULL &&
conn->bhc_rx_chan->scid == BLE_L2CAP_TEST_CID);
TEST_ASSERT(conn->rx_len && conn->rx_cid == BLE_L2CAP_TEST_CID);
ble_hs_unlock();
}
@@ -240,8 +239,7 @@ ble_l2cap_test_util_verify_middle_frag(uint16_t conn_handle,
conn = ble_hs_conn_find(conn_handle);
TEST_ASSERT_FATAL(conn != NULL);
TEST_ASSERT(conn->bhc_rx_chan != NULL &&
conn->bhc_rx_chan->scid == BLE_L2CAP_TEST_CID);
TEST_ASSERT(conn->rx_len && conn->rx_cid == BLE_L2CAP_TEST_CID);
ble_hs_unlock();
}
@@ -260,7 +258,7 @@ ble_l2cap_test_util_verify_last_frag(uint16_t conn_handle,
conn = ble_hs_conn_find(conn_handle);
TEST_ASSERT_FATAL(conn != NULL);
TEST_ASSERT(conn->bhc_rx_chan == NULL);
TEST_ASSERT(conn->rx_len == 0);
ble_hs_unlock();
}
@@ -279,7 +277,7 @@ TEST_CASE_SELF(ble_l2cap_test_case_bad_header)
NULL, NULL);
rc = ble_l2cap_test_util_rx_first_frag(2, 14, 1234, 10);
TEST_ASSERT(rc == BLE_HS_ENOENT);
TEST_ASSERT(rc == BLE_HS_EBADDATA);
ble_hs_test_util_assert_mbufs_freed(NULL);
}
@@ -386,8 +384,7 @@ TEST_CASE_SELF(ble_l2cap_test_case_frag_channels)
ble_hs_lock();
conn = ble_hs_conn_find(2);
TEST_ASSERT_FATAL(conn != NULL);
TEST_ASSERT(conn->bhc_rx_chan != NULL &&
conn->bhc_rx_chan->scid == BLE_L2CAP_TEST_CID);
TEST_ASSERT(conn->rx_len && conn->rx_cid == BLE_L2CAP_TEST_CID);
ble_hs_unlock();
/* Receive a starting fragment on a different channel. The first fragment
@@ -400,8 +397,7 @@ TEST_CASE_SELF(ble_l2cap_test_case_frag_channels)
ble_hs_lock();
conn = ble_hs_conn_find(2);
TEST_ASSERT_FATAL(conn != NULL);
TEST_ASSERT(conn->bhc_rx_chan != NULL &&
conn->bhc_rx_chan->scid == BLE_L2CAP_CID_ATT);
TEST_ASSERT(conn->rx_len && conn->rx_cid == BLE_L2CAP_CID_ATT);
ble_hs_unlock();
/* Terminate the connection. The received fragments should get freed.