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

Commit 67a66f4b authored by Iván Budnik's avatar Iván Budnik
Browse files

Add test for correct MBR ComponentName resolution 

This test checks that a MediaButtonReceiverHolder resolves a
PendingIntent to an appropriate ComponentName that matches the
PendingIntent's creator package.

Bug: 238177121
Test: atest FrameworksServicesTests:MediaButtonReceiverHolderTest
Change-Id: I930679c43c95633bee3a499a6d87ebeebb33b473
parent 1dee6daf

services/core/java/com/android/server/media/MediaButtonReceiverHolder.java

+4 −0
Original line number Original line Diff line number Diff line
@@ -275,6 +275,10 @@ final class MediaButtonReceiverHolder { 
                String.valueOf(mComponentType));

                String.valueOf(mComponentType));

    }

    }





    public ComponentName getComponentName() {

        return mComponentName;

    }



    @ComponentType

    @ComponentType

    private static int getComponentType(PendingIntent pendingIntent) {

    private static int getComponentType(PendingIntent pendingIntent) {

        if (pendingIntent.isBroadcast()) {

        if (pendingIntent.isBroadcast()) {

services/tests/servicestests/AndroidManifest.xml

+3 −0
Original line number Original line Diff line number Diff line
@@ -107,6 +107,9 @@ 




    <queries>

    <queries>

        <package android:name="com.android.servicestests.apps.suspendtestapp" />

        <package android:name="com.android.servicestests.apps.suspendtestapp" />

        <intent>

            <action android:name="android.media.browse.MediaBrowserService" />

        </intent>

    </queries>

    </queries>





    <!-- Uses API introduced in O (26) -->

    <!-- Uses API introduced in O (26) -->

services/tests/servicestests/AndroidTest.xml

+1 −0
Original line number Original line Diff line number Diff line
@@ -33,6 +33,7 @@ 
        <option name="test-file-name" value="SimpleServiceTestApp1.apk" />

        <option name="test-file-name" value="SimpleServiceTestApp1.apk" />

        <option name="test-file-name" value="SimpleServiceTestApp2.apk" />

        <option name="test-file-name" value="SimpleServiceTestApp2.apk" />

        <option name="test-file-name" value="SimpleServiceTestApp3.apk" />

        <option name="test-file-name" value="SimpleServiceTestApp3.apk" />

        <option name="test-file-name" value="FakeMediaApp.apk" />

    </target_preparer>

    </target_preparer>





    <!-- Create place to store apks -->

    <!-- Create place to store apks -->

services/tests/servicestests/src/com/android/server/media/MediaButtonReceiverHolderTest.java

0 → 100644
+45 −0
Original line number Original line Diff line number Diff line 
/*

 * Copyright (C) 2022 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



package com.android.server.media;



import android.app.PendingIntent;

import android.content.Context;

import android.content.Intent;



import androidx.test.ext.junit.runners.AndroidJUnit4;

import androidx.test.platform.app.InstrumentationRegistry;



import com.google.common.truth.Truth;



import org.junit.Test;

import org.junit.runner.RunWith;



@RunWith(AndroidJUnit4.class)

public class MediaButtonReceiverHolderTest {



    @Test

    public void createMediaButtonReceiverHolder_resolvesNullComponentName() {

        Context context = InstrumentationRegistry.getInstrumentation().getContext();

        Intent intent = new Intent(Intent.ACTION_MEDIA_BUTTON);

        PendingIntent pi = PendingIntent.getBroadcast(context, /* requestCode= */ 0, intent,

                PendingIntent.FLAG_IMMUTABLE);

        MediaButtonReceiverHolder a = MediaButtonReceiverHolder.create(/* userId= */ 0, pi,

                context.getPackageName());

        Truth.assertWithMessage("Component name must match PendingIntent creator package.").that(

                a.getComponentName()).isNull();

    }

}

services/tests/servicestests/src/com/android/server/media/OWNERS

0 → 100644
+2 −0
Original line number Original line Diff line number Diff line 
# Bug component: 137631

include platform/frameworks/av:/media/janitors/media_solutions_OWNERS
 
 No newline at end of file