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

Commit 74b0a78a authored by Jack Yu's avatar Jack Yu
Browse files

Cleaned up the flag enforce_telephony_feature_mapping 

Cleaned up the 24Q3 flag enforce_telephony_feature_mapping

Test: Basic telephony functionality tests
Test: atest EuiccProfileInfoTest
Bug: 297989574
Flag: EXEMPT flag cleanup
Change-Id: Ib7fa1d2aab105d77b34a0b5716827060eb3413b6
parent 74968249

flags/misc.aconfig

+0 −8
Original line number Diff line number Diff line
@@ -52,14 +52,6 @@ flag { 
    bug:"271921464"

}



# OWNER=joonhunshin TARGET=24Q3

flag {

    name: "enforce_telephony_feature_mapping"

    namespace: "telephony"

    description: "This flag controls telephony feature flags mapping."

    bug:"297989574"

}



# OWNER=joonhunshin TARGET=24Q3

flag {

    name: "enforce_telephony_feature_mapping_for_public_apis"

tests/telephonytests/src/com/android/internal/telephony/subscription/SubscriptionManagerServiceTest.java

+9 −48
Original line number Diff line number Diff line
@@ -118,7 +118,6 @@ import com.android.internal.telephony.subscription.SubscriptionManagerService.Su 
import com.android.internal.telephony.uicc.IccCardStatus;

import com.android.internal.telephony.uicc.UiccSlot;



import libcore.junit.util.compat.CoreCompatChangeRule.DisableCompatChanges;

import libcore.junit.util.compat.CoreCompatChangeRule.EnableCompatChanges;



import org.junit.After;
@@ -449,52 +448,9 @@ public class SubscriptionManagerServiceTest extends TelephonyTest { 
                () -> mSubscriptionManagerServiceUT.setGroupOwner(1, groupOwner));

    }



    @Test

    @DisableCompatChanges({TelephonyManager.ENABLE_FEATURE_MAPPING})

    public void testSetPhoneNumber() {

        doReturn(false).when(mFeatureFlags).enforceTelephonyFeatureMapping();

        doReturn(true).when(mPackageManager).hasSystemFeature(

                eq(PackageManager.FEATURE_TELEPHONY_SUBSCRIPTION));



        mContextFixture.addCallingOrSelfPermission(Manifest.permission.MODIFY_PHONE_STATE);

        mSubscriptionManagerServiceUT.addSubInfo(FAKE_ICCID1, FAKE_CARRIER_NAME1,

                0, SubscriptionManager.SUBSCRIPTION_TYPE_LOCAL_SIM);

        processAllMessages();



        verify(mMockedSubscriptionManagerServiceCallback).onSubscriptionChanged(eq(1));

        Mockito.clearInvocations(mMockedSubscriptionManagerServiceCallback);



        // Caller does not have carrier privilege

        assertThrows(SecurityException.class,

                () -> mSubscriptionManagerServiceUT.setPhoneNumber(1,

                        SubscriptionManager.PHONE_NUMBER_SOURCE_CARRIER, FAKE_PHONE_NUMBER2,

                        CALLING_PACKAGE, CALLING_FEATURE));



        // Grant carrier privilege

        setCarrierPrivilegesForSubId(true, 1);



        // Source IMS is not acceptable

        assertThrows(IllegalArgumentException.class,

                () -> mSubscriptionManagerServiceUT.setPhoneNumber(1,

                        SubscriptionManager.PHONE_NUMBER_SOURCE_IMS, FAKE_PHONE_NUMBER2,

                        CALLING_PACKAGE, CALLING_FEATURE));



        mSubscriptionManagerServiceUT.setPhoneNumber(1,

                SubscriptionManager.PHONE_NUMBER_SOURCE_CARRIER, FAKE_PHONE_NUMBER2,

                CALLING_PACKAGE, CALLING_FEATURE);

        processAllMessages();



        SubscriptionInfoInternal subInfo = mSubscriptionManagerServiceUT

                .getSubscriptionInfoInternal(1);

        assertThat(subInfo).isNotNull();

        assertThat(subInfo.getNumberFromCarrier()).isEqualTo(FAKE_PHONE_NUMBER2);

        verify(mMockedSubscriptionManagerServiceCallback).onSubscriptionChanged(eq(1));

    }



    @Test

    @EnableCompatChanges({TelephonyManager.ENABLE_FEATURE_MAPPING})

    public void testSetPhoneNumber_EnabledEnforceTelephonyFeatureMappingForPublicApis()

            throws Exception {

    public void testSetPhoneNumber() throws Exception {

        mContextFixture.addCallingOrSelfPermission(Manifest.permission.MODIFY_PHONE_STATE);

        mSubscriptionManagerServiceUT.addSubInfo(FAKE_ICCID1, FAKE_CARRIER_NAME1,

                0, SubscriptionManager.SUBSCRIPTION_TYPE_LOCAL_SIM);
@@ -2520,10 +2476,12 @@ public class SubscriptionManagerServiceTest extends TelephonyTest { 
    }



    @Test

    @DisableCompatChanges({TelephonyManager.ENABLE_FEATURE_MAPPING})

    public void testGetPhoneNumber() {

    @EnableCompatChanges({TelephonyManager.ENABLE_FEATURE_MAPPING})

    public void testGetPhoneNumber() throws Exception {

        mContextFixture.addCallingOrSelfPermission(Manifest.permission.READ_PRIVILEGED_PHONE_STATE);

        testSetPhoneNumber();

        doReturn(true).when(mPackageManager).hasSystemFeature(

                eq(PackageManager.FEATURE_TELEPHONY_SUBSCRIPTION));

        assertThat(mSubscriptionManagerServiceUT.getPhoneNumber(1,

                SubscriptionManager.PHONE_NUMBER_SOURCE_CARRIER, CALLING_PACKAGE, CALLING_FEATURE))

                .isEqualTo(FAKE_PHONE_NUMBER2);
@@ -2534,9 +2492,12 @@ public class SubscriptionManagerServiceTest extends TelephonyTest { 
    }



    @Test

    public void testGetPhoneNumberFromUicc() {

    @EnableCompatChanges({TelephonyManager.ENABLE_FEATURE_MAPPING})

    public void testGetPhoneNumberFromUicc() throws Exception {

        mContextFixture.addCallingOrSelfPermission(Manifest.permission.READ_PRIVILEGED_PHONE_STATE);

        testSetPhoneNumber();

        doReturn(true).when(mPackageManager).hasSystemFeature(

                eq(PackageManager.FEATURE_TELEPHONY_SUBSCRIPTION));

        // Number from line1Number should be FAKE_PHONE_NUMBER1 instead of FAKE_PHONE_NUMBER2

        assertThat(mSubscriptionManagerServiceUT.getPhoneNumber(1,

                SubscriptionManager.PHONE_NUMBER_SOURCE_UICC, CALLING_PACKAGE, CALLING_FEATURE))