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

Commit afebfaba authored by changbetty's avatar changbetty
Browse files

[Mainline] Use isActiveSubscriptionId instead of isActiveSubId for mainline 

Bug: 147206736
Test: make RunSettingsRoboTests
Change-Id: I9c06f47e2b1189820d2f7002d811468e5f66a095
parent 2f8be92f

packages/SettingsLib/src/com/android/settingslib/net/DataUsageUtils.java

+1 −1
Original line number Diff line number Diff line
@@ -40,7 +40,7 @@ public class DataUsageUtils { 
        final NetworkTemplate mobileAll = NetworkTemplate.buildTemplateMobileAll(

                telephonyManager.getSubscriberId(subId));



        if (!subscriptionManager.isActiveSubId(subId)) {

        if (!subscriptionManager.isActiveSubscriptionId(subId)) {

            Log.i(TAG, "Subscription is not active: " + subId);

            return mobileAll;

        }

packages/SettingsLib/tests/robotests/src/com/android/settingslib/net/DataUsageUtilsTest.java

+2 −2
Original line number Diff line number Diff line
@@ -71,12 +71,12 @@ public class DataUsageUtilsTest { 
        when(mTelephonyManager.getSubscriberId(SUB_ID)).thenReturn(SUBSCRIBER_ID);

        when(mTelephonyManager.getSubscriberId(SUB_ID_2)).thenReturn(SUBSCRIBER_ID_2);

        when(mTelephonyManager.createForSubscriptionId(anyInt())).thenReturn(mTelephonyManager);

        when(mSubscriptionManager.isActiveSubId(anyInt())).thenReturn(true);

        when(mSubscriptionManager.isActiveSubscriptionId(anyInt())).thenReturn(true);

    }



    @Test

    public void getMobileTemplate_infoNull_returnMobileAll() {

        when(mSubscriptionManager.isActiveSubId(SUB_ID)).thenReturn(false);

        when(mSubscriptionManager.isActiveSubscriptionId(SUB_ID)).thenReturn(false);



        final NetworkTemplate networkTemplate = DataUsageUtils.getMobileTemplate(mContext, SUB_ID);

        assertThat(networkTemplate.matchesSubscriberId(SUBSCRIBER_ID)).isTrue();