From 628a335326766921d28220e1a1352cf072e1c61d Mon Sep 17 00:00:00 2001 From: Shuo Qian Date: Thu, 7 Jan 2021 10:54:01 -0800 Subject: [PATCH] Skip emergency dialing VTS if device doesn't support radio Test: vts Bug: 176862267 Change-Id: I9e72ba5cf6cb118cd68f753355f4119d33fe0019 --- radio/1.0/vts/functional/vts_test_util.h | 2 ++ radio/1.4/vts/functional/radio_hidl_hal_api.cpp | 9 +++++++++ 2 files changed, 11 insertions(+) diff --git a/radio/1.0/vts/functional/vts_test_util.h b/radio/1.0/vts/functional/vts_test_util.h index df8dd7718b..5df162e09a 100644 --- a/radio/1.0/vts/functional/vts_test_util.h +++ b/radio/1.0/vts/functional/vts_test_util.h @@ -34,6 +34,8 @@ enum CheckFlag { static constexpr const char* FEATURE_VOICE_CALL = "android.software.connectionservice"; +static constexpr const char* FEATURE_TELEPHONY = "android.hardware.telephony"; + /* * Generate random serial number for radio test */ diff --git a/radio/1.4/vts/functional/radio_hidl_hal_api.cpp b/radio/1.4/vts/functional/radio_hidl_hal_api.cpp index 3ba9b9db9d..ea17024f18 100644 --- a/radio/1.4/vts/functional/radio_hidl_hal_api.cpp +++ b/radio/1.4/vts/functional/radio_hidl_hal_api.cpp @@ -34,6 +34,9 @@ TEST_P(RadioHidlTest_v1_4, emergencyDial) { if (!deviceSupportsFeature(FEATURE_VOICE_CALL)) { ALOGI("Skipping emergencyDial because voice call is not supported in device"); return; + } else if (!deviceSupportsFeature(FEATURE_TELEPHONY)) { + ALOGI("Skipping emergencyDial because telephony radio is not supported in device"); + return; } else { ALOGI("Running emergencyDial because voice call is supported in device"); } @@ -72,6 +75,9 @@ TEST_P(RadioHidlTest_v1_4, emergencyDial_withServices) { if (!deviceSupportsFeature(FEATURE_VOICE_CALL)) { ALOGI("Skipping emergencyDial because voice call is not supported in device"); return; + } else if (!deviceSupportsFeature(FEATURE_TELEPHONY)) { + ALOGI("Skipping emergencyDial because telephony radio is not supported in device"); + return; } else { ALOGI("Running emergencyDial because voice call is supported in device"); } @@ -111,6 +117,9 @@ TEST_P(RadioHidlTest_v1_4, emergencyDial_withEmergencyRouting) { if (!deviceSupportsFeature(FEATURE_VOICE_CALL)) { ALOGI("Skipping emergencyDial because voice call is not supported in device"); return; + } else if (!deviceSupportsFeature(FEATURE_TELEPHONY)) { + ALOGI("Skipping emergencyDial because telephony radio is not supported in device"); + return; } else { ALOGI("Running emergencyDial because voice call is supported in device"); }