Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 77e2a675 authored by Aaron Tsai's avatar Aaron Tsai
Browse files

Fix emergencyDial tests in VTS 1.6 

 cherry-pick from aosp/1708089

 - Change to calling getVoiceRegistrationState_1_6() instead of getVoiceRegistrationState().
 - Fix incorrect voice registration state value


Bug: 187488263
Test: atest VtsHalRadioV1_6TargetTest

Merged-In: Iebf32061c7944104a8d4a0297af9204ca214e627
Merged-In: Ic2745c48b3d3451e5065a77054d9e7b930ea4ce5
Merged-In: I787ead62a70acd69a4676d5e53509fe2ff6b0fd8
Change-Id: I0a04238f03c7d4fe1eae00c33c18d6dd9da610a8
parent d02d9f27

radio/1.6/vts/functional/radio_hidl_hal_api.cpp

+3 −3
Original line number Diff line number Diff line
@@ -599,7 +599,7 @@ TEST_P(RadioHidlTest_v1_6, emergencyDial_1_6) { 
    // or Emergency_Only.

    if (isDsDsEnabled() || isTsTsEnabled()) {

        serial = GetRandomSerialNumber();

        radio_v1_6->getVoiceRegistrationState(serial);

        radio_v1_6->getVoiceRegistrationState_1_6(serial);

        EXPECT_EQ(std::cv_status::no_timeout, wait());

        if (isVoiceEmergencyOnly(radioRsp_v1_6->voiceRegResp.regState) ||

            isVoiceInService(radioRsp_v1_6->voiceRegResp.regState)) {
@@ -653,7 +653,7 @@ TEST_P(RadioHidlTest_v1_6, emergencyDial_1_6_withServices) { 
    // or Emergency_Only.

    if (isDsDsEnabled() || isTsTsEnabled()) {

        serial = GetRandomSerialNumber();

        radio_v1_6->getVoiceRegistrationState(serial);

        radio_v1_6->getVoiceRegistrationState_1_6(serial);

        EXPECT_EQ(std::cv_status::no_timeout, wait());

        if (isVoiceEmergencyOnly(radioRsp_v1_6->voiceRegResp.regState) ||

            isVoiceInService(radioRsp_v1_6->voiceRegResp.regState)) {
@@ -706,7 +706,7 @@ TEST_P(RadioHidlTest_v1_6, emergencyDial_1_6_withEmergencyRouting) { 
    // or Emergency_Only.

    if (isDsDsEnabled() || isTsTsEnabled()) {

        serial = GetRandomSerialNumber();

        radio_v1_6->getVoiceRegistrationState(serial);

        radio_v1_6->getVoiceRegistrationState_1_6(serial);

        EXPECT_EQ(std::cv_status::no_timeout, wait());

        if (isVoiceEmergencyOnly(radioRsp_v1_6->voiceRegResp.regState) ||

            isVoiceInService(radioRsp_v1_6->voiceRegResp.regState)) {

radio/1.6/vts/functional/radio_response.cpp

+2 −1
Original line number Diff line number Diff line
@@ -1209,8 +1209,9 @@ Return<void> RadioResponse_v1_6::getSystemSelectionChannelsResponse( 


Return<void> RadioResponse_v1_6::getVoiceRegistrationStateResponse_1_6(

        const ::android::hardware::radio::V1_6::RadioResponseInfo& info,

        const ::android::hardware::radio::V1_6::RegStateResult& /*regResponse*/) {

        const ::android::hardware::radio::V1_6::RegStateResult& regResponse) {

    rspInfo = info;

    voiceRegResp.regState = regResponse.regState;

    parent_v1_6.notify(info.serial);

    return Void();

}