From 976d87ba8e0255af8e050aa076b411706dea0714 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Fr=C3=A9d=C3=A9ric=20Danis?= Date: Fri, 19 Sep 2025 10:23:33 +0200 Subject: [PATCH] unit/test-hfp: Add Reject Incoming Call tests for HF This add the following tests: - /HFP/HF/ICR/BV-01-C Verify that the HF can reject an incoming call after being alerted. - /HFP/HF/ICR/BV-02-C Verify that the AG, upon the corresponding action, rejects an incoming call and properly indicates this event to the HF. --- unit/test-hfp.c | 44 +++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 43 insertions(+), 1 deletion(-) diff --git a/unit/test-hfp.c b/unit/test-hfp.c index adbcfd2fc..a0faf98d7 100644 --- a/unit/test-hfp.c +++ b/unit/test-hfp.c @@ -830,7 +830,8 @@ 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/ICA/BV-06-C") || + g_str_equal(test_name, "/HFP/HF/ICR/BV-01-C")) { g_assert_cmpint(id, ==, 1); g_assert_cmpint(status, ==, CALL_STATUS_INCOMING); } @@ -861,6 +862,14 @@ static void hf_call_line_id_updated(uint id, const char *number, ret = hfp_hf_call_answer(context->hfp_hf, id, hf_cmd_complete, context); g_assert(ret); + } else if (g_str_equal(test_name, "/HFP/HF/ICR/BV-01-C")) { + bool ret; + + if (tester_use_debug()) + tester_debug("call %d: rejecting call", id); + ret = hfp_hf_call_hangup(context->hfp_hf, id, hf_cmd_complete, + context); + g_assert(ret); } } @@ -888,6 +897,10 @@ static void hf_call_status_updated(uint id, enum hfp_call_status status, g_assert_cmpint(status, ==, CALL_STATUS_ACTIVE); number = hfp_hf_call_get_number(context->hfp_hf, id); g_assert_cmpstr(number, ==, "1234567"); + } else if (g_str_equal(test_name, "/HFP/HF/ICR/BV-01-C")) { + if (tester_use_debug()) + tester_debug("Error: unexpected update"); + tester_test_failed(); } } @@ -1159,6 +1172,35 @@ int main(int argc, char *argv[]) '2', ',', '0', '\r', '\n'), data_end()); + /* Initiate rejection of incoming call - HF */ + define_hf_test("/HFP/HF/ICR/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', ':', ' ', + '3', ',', '0', '\r', '\n'), + data_end()); + + /* Accept AG rejection of incoming call - HF */ + define_hf_test("/HFP/HF/ICR/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'), + frg_pdu('\r', '\n', '+', 'C', 'I', 'E', 'V', ':', ' ', + '3', ',', '0', '\r', '\n'), + data_end()); + /* Transfer Signal Strength Indication - HF */ define_hf_test("/HFP/HF/PSI/BV-01-C", test_hf_session, NULL, test_hf_session_done, -- 2.47.3