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

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

Merge "Corretly handle SIM not ready state" into udc-dev am: 2b5f925e am: 1637641c 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/opt/telephony/+/23228136



Change-Id: Idf31f6febc9e458e3f6aa0ec867d4f9c03d13ce0
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 4308975f 1637641c

src/java/com/android/internal/telephony/subscription/SubscriptionManagerService.java

+1 −4
Original line number Diff line number Diff line
@@ -46,7 +46,6 @@ import android.os.Process; 
import android.os.RemoteException;

import android.os.TelephonyServiceManager;

import android.os.UserHandle;

import android.provider.DeviceConfig;

import android.provider.Settings;

import android.provider.Telephony.SimInfo;

import android.service.carrier.CarrierIdentifier;
@@ -1311,9 +1310,7 @@ public class SubscriptionManagerService extends ISub.Stub { 
                log("updateSubscription: SIM_STATE_NOT_READY is not a final state. Will update "

                        + "subscription later.");

                return;

            }



            if (!areUiccAppsEnabledOnCard(phoneId)) {

            } else {

                logl("updateSubscription: UICC app disabled on slot " + phoneId);

                markSubscriptionsInactive(phoneId);

            }

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

+13 −0
Original line number Diff line number Diff line
@@ -2235,6 +2235,19 @@ public class SubscriptionManagerServiceTest extends TelephonyTest { 
        assertThat(mSubscriptionManagerServiceUT.getActiveSubIdList(false)).isEmpty();

    }



    @Test

    public void testSimNotReadyBySimDeactivate() {

        insertSubscription(FAKE_SUBSCRIPTION_INFO1);



        mContextFixture.addCallingOrSelfPermission(Manifest.permission.READ_PRIVILEGED_PHONE_STATE);

        mSubscriptionManagerServiceUT.updateSimState(

                0, TelephonyManager.SIM_STATE_NOT_READY, null, null);

        doReturn(true).when(mUiccProfile).isEmptyProfile();

        processAllMessages();



        assertThat(mSubscriptionManagerServiceUT.getActiveSubIdList(false)).isEmpty();

    }



    @Test

    public void testInactiveSimRemoval() {

        insertSubscription(FAKE_SUBSCRIPTION_INFO2);