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

Commit ea80f22f authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Fix result mismatch in broadcast radio service" into udc-dev am:... 

Merge "Fix result mismatch in broadcast radio service" into udc-dev am: 2edc38cc am: 647ea986 am: d9f510b8

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/22877086



Change-Id: I82aa354eb340f5f36beef32aff7282e5b2e24272
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 2f5f5b7b d9f510b8

core/tests/BroadcastRadioTests/src/com/android/server/broadcastradio/aidl/AidlTestUtils.java

+1 −1
Original line number Diff line number Diff line
@@ -80,7 +80,7 @@ final class AidlTestUtils { 
                /* vendorIds= */ null);

    }



    static android.hardware.broadcastradio.ProgramSelector makeHalFmSelector(int freq) {

    static android.hardware.broadcastradio.ProgramSelector makeHalFmSelector(long freq) {

        ProgramIdentifier halId = makeHalIdentifier(IdentifierType.AMFM_FREQUENCY_KHZ, freq);

        return makeHalSelector(halId, /* secondaryIds= */ new ProgramIdentifier[0]);

    }

core/tests/BroadcastRadioTests/src/com/android/server/broadcastradio/aidl/ConversionUtilsResultTest.java

+1 −0
Original line number Diff line number Diff line
@@ -52,6 +52,7 @@ public final class ConversionUtilsResultTest { 
                {Result.INVALID_STATE, RadioTuner.TUNER_RESULT_INVALID_STATE},

                {Result.NOT_SUPPORTED, RadioTuner.TUNER_RESULT_NOT_SUPPORTED},

                {Result.TIMEOUT, RadioTuner.TUNER_RESULT_TIMEOUT},

                {Result.CANCELED, RadioTuner.TUNER_RESULT_CANCELED},

                {Result.UNKNOWN_ERROR, RadioTuner.TUNER_RESULT_UNKNOWN_ERROR}

        });

    }

core/tests/BroadcastRadioTests/src/com/android/server/broadcastradio/aidl/ConversionUtilsTest.java

+12 −0
Original line number Diff line number Diff line
@@ -27,11 +27,13 @@ import android.hardware.broadcastradio.ProgramIdentifier; 
import android.hardware.broadcastradio.ProgramInfo;

import android.hardware.broadcastradio.ProgramListChunk;

import android.hardware.broadcastradio.Properties;

import android.hardware.broadcastradio.Result;

import android.hardware.broadcastradio.VendorKeyValue;

import android.hardware.radio.Announcement;

import android.hardware.radio.ProgramList;

import android.hardware.radio.ProgramSelector;

import android.hardware.radio.RadioManager;

import android.os.ServiceSpecificException;



import com.android.dx.mockito.inline.extended.StaticMockitoSessionBuilder;

import com.android.server.broadcastradio.ExtendedRadioMockitoTestCase;
@@ -152,6 +154,16 @@ public final class ConversionUtilsTest extends ExtendedRadioMockitoTestCase { 
                .that(ConversionUtils.isAtLeastU(U_APP_UID)).isTrue();

    }



    @Test

    public void throwOnError_withCancelException() {

        ServiceSpecificException halException = new ServiceSpecificException(Result.CANCELED);



        RuntimeException thrown = ConversionUtils.throwOnError(halException, "tune");



        expect.withMessage("Exception thrown for canceling error").that(thrown)

                .hasMessageThat().contains("tune: CANCELED");

    }



    @Test

    public void propertiesFromHalProperties_idsMatch() {

        expect.withMessage("Properties id")

core/tests/BroadcastRadioTests/src/com/android/server/broadcastradio/aidl/TunerSessionTest.java

+17 −1
Original line number Diff line number Diff line
@@ -1168,12 +1168,28 @@ public final class TunerSessionTest extends ExtendedRadioMockitoTestCase { 
        doReturn(USER_ID_2).when(() -> RadioServiceUserController.getCurrentUser());



        mHalTunerCallback.onCurrentProgramInfoChanged(AidlTestUtils.makeHalProgramInfo(

                AidlTestUtils.makeHalFmSelector(/* freq= */ 97300), SIGNAL_QUALITY));

                AidlTestUtils.makeHalFmSelector(AM_FM_FREQUENCY_LIST[1]), SIGNAL_QUALITY));



        verify(mAidlTunerCallbackMocks[0], CALLBACK_TIMEOUT.times(0))

                .onCurrentProgramInfoChanged(any());

    }



    @Test

    public void onTuneFailed_forTunerCallback() throws Exception {

        int numSessions = 3;

        openAidlClients(numSessions);

        android.hardware.broadcastradio.ProgramSelector halSel = AidlTestUtils.makeHalFmSelector(

                AM_FM_FREQUENCY_LIST[1]);

        ProgramSelector sel = AidlTestUtils.makeFmSelector(AM_FM_FREQUENCY_LIST[1]);



        mHalTunerCallback.onTuneFailed(Result.CANCELED, halSel);



        for (int index = 0; index < numSessions; index++) {

            verify(mAidlTunerCallbackMocks[index], CALLBACK_TIMEOUT)

                    .onTuneFailed(RadioTuner.TUNER_RESULT_CANCELED, sel);

        }

    }



    @Test

    public void onAntennaStateChange_forTunerCallback() throws Exception {

        int numSessions = 3;

services/core/java/com/android/server/broadcastradio/aidl/ConversionUtils.java

+4 −0
Original line number Diff line number Diff line
@@ -102,6 +102,8 @@ final class ConversionUtils { 
                return new UnsupportedOperationException(action + ": NOT_SUPPORTED");

            case Result.TIMEOUT:

                return new ParcelableException(new RuntimeException(action + ": TIMEOUT"));

            case Result.CANCELED:

                return new IllegalStateException(action + ": CANCELED");

            default:

                return new ParcelableException(new RuntimeException(

                        action + ": unknown error (" + result + ")"));
@@ -123,6 +125,8 @@ final class ConversionUtils { 
                return RadioTuner.TUNER_RESULT_NOT_SUPPORTED;

            case Result.TIMEOUT:

                return RadioTuner.TUNER_RESULT_TIMEOUT;

            case Result.CANCELED:

                return RadioTuner.TUNER_RESULT_CANCELED;

            case Result.UNKNOWN_ERROR:

            default:

                return RadioTuner.TUNER_RESULT_UNKNOWN_ERROR;