From c53a401998f77766001d4b7744722408f04d4150 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Danis?= Date: Fri, 19 Sep 2025 10:23:34 +0200 Subject: [PATCH] unit/test-hfp: Add Terminate Call tests for HF This add the following tests: - /HFP/HF/TCA/BV-01-C Verify that the HF can terminate an ongoing call in the AG. - /HFP/HF/TCA/BV-02-C Verify that the AG, upon the corresponding action, terminates an ongoing call. The AG then indicates this event to the HF. - /HFP/HF/TCA/BV-04-C Verify that the HF can release a call after dialing and prior to call completion. /HFP/HF/TCA/BV-03-C (Verify that after a call is terminated from the remote party, the HF receives the proper indication from the AG) is similar to /HFP/HF/TCA/BV-02-C for the HF side. --- unit/test-hfp.c | 88 +++++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 85 insertions(+), 3 deletions(-) diff --git a/unit/test-hfp.c b/unit/test-hfp.c index a0faf98d7..190604ba2 100644 --- a/unit/test-hfp.c +++ b/unit/test-hfp.c @@ -831,9 +831,22 @@ static void hf_call_added(uint id, enum hfp_call_status status, if (g_str_equal(test_name, "/HFP/HF/CLI/BV-01-C") || g_str_equal(test_name, "/HFP/HF/ICA/BV-04-C") || g_str_equal(test_name, "/HFP/HF/ICA/BV-06-C") || - g_str_equal(test_name, "/HFP/HF/ICR/BV-01-C")) { + g_str_equal(test_name, "/HFP/HF/ICR/BV-01-C") || + g_str_equal(test_name, "/HFP/HF/TCA/BV-01-C") || + g_str_equal(test_name, "/HFP/HF/TCA/BV-02-C")) { g_assert_cmpint(id, ==, 1); g_assert_cmpint(status, ==, CALL_STATUS_INCOMING); + } else if (g_str_equal(test_name, "/HFP/HF/TCA/BV-04-C")) { + bool ret; + + g_assert_cmpint(id, ==, 1); + g_assert_cmpint(status, ==, CALL_STATUS_DIALING); + + if (tester_use_debug()) + tester_debug("call %d: ending call", id); + ret = hfp_hf_call_hangup(context->hfp_hf, id, hf_cmd_complete, + context); + g_assert(ret); } } @@ -854,7 +867,9 @@ static void hf_call_line_id_updated(uint id, const char *number, str = hfp_hf_call_get_number(context->hfp_hf, id); g_assert_cmpstr(number, ==, str); - if (g_str_equal(test_name, "/HFP/HF/ICA/BV-04-C")) { + if (g_str_equal(test_name, "/HFP/HF/ICA/BV-04-C") || + g_str_equal(test_name, "/HFP/HF/TCA/BV-01-C") || + g_str_equal(test_name, "/HFP/HF/TCA/BV-02-C")) { bool ret; if (tester_use_debug()) @@ -890,7 +905,8 @@ static void hf_call_status_updated(uint id, enum hfp_call_status status, tester_debug("call %d updated: status %u", id, status); if (g_str_equal(test_name, "/HFP/HF/ICA/BV-04-C") || - g_str_equal(test_name, "/HFP/HF/ICA/BV-06-C")) { + g_str_equal(test_name, "/HFP/HF/ICA/BV-06-C") || + g_str_equal(test_name, "/HFP/HF/TCA/BV-02-C")) { const char *number; g_assert_cmpint(id, ==, 1); @@ -901,6 +917,20 @@ static void hf_call_status_updated(uint id, enum hfp_call_status status, if (tester_use_debug()) tester_debug("Error: unexpected update"); tester_test_failed(); + } else if (g_str_equal(test_name, "/HFP/HF/TCA/BV-01-C")) { + const char *number; + bool ret; + + g_assert_cmpint(id, ==, 1); + g_assert_cmpint(status, ==, CALL_STATUS_ACTIVE); + number = hfp_hf_call_get_number(context->hfp_hf, id); + g_assert_cmpstr(number, ==, "1234567"); + + if (tester_use_debug()) + tester_debug("call %d: ending call", id); + ret = hfp_hf_call_hangup(context->hfp_hf, id, hf_cmd_complete, + context); + g_assert(ret); } } @@ -1233,6 +1263,58 @@ int main(int argc, char *argv[]) MINIMAL_SLC_SESSION('1', '0', '0', '0'), data_end()); + /* Terminate ongoing call - HF */ + define_hf_test("/HFP/HF/TCA/BV-01-C", test_hf_session, + NULL, test_hf_session_done, + MINIMAL_SLC_SESSION('1', '0', '0', '0'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '3', ',', '1', '\r', '\n'), + frg_pdu('\r', '\n', 'R', 'I', 'N', 'G', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'L', 'I', 'P', ':', + '\"', '1', '2', '3', '4', '5', '6', '7', '\"', + ',', '1', '2', '9', ',', ',', '\r', '\n'), + raw_pdu('\r', '\n', 'O', 'K', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '2', ',', '1', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '3', ',', '0', '\r', '\n'), + raw_pdu('\r', '\n', 'O', 'K', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '2', ',', '0', '\r', '\n'), + data_end()); + + /* Terminate ongoing call on AG - HF + * idem for /HFP/HF/TCA/BV-03-C Remote party terminates the call + */ + define_hf_test("/HFP/HF/TCA/BV-02-C", test_hf_session, + NULL, test_hf_session_done, + MINIMAL_SLC_SESSION('1', '0', '0', '0'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '3', ',', '1', '\r', '\n'), + frg_pdu('\r', '\n', 'R', 'I', 'N', 'G', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'L', 'I', 'P', ':', + '\"', '1', '2', '3', '4', '5', '6', '7', '\"', + ',', '1', '2', '9', ',', ',', '\r', '\n'), + raw_pdu('\r', '\n', 'O', 'K', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '2', ',', '1', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '3', ',', '0', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '2', ',', '0', '\r', '\n'), + data_end()); + + /* Abandon outgoing call - HF */ + define_hf_test("/HFP/HF/TCA/BV-04-C", test_hf_session, + NULL, test_hf_session_done, + MINIMAL_SLC_SESSION('1', '0', '0', '0'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '3', ',', '2', '\r', '\n'), + raw_pdu('\r', '\n', 'O', 'K', '\r', '\n'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '3', ',', '0', '\r', '\n'), + data_end()); + /* Transfer Registration Status - HF */ define_hf_test("/HFP/HF/TRS/BV-01-C", test_hf_session, NULL, test_hf_session_done, -- 2.47.3