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

Commit e57e5941 authored by Amit Mahajan's avatar Amit Mahajan
Browse files

New unit tests for UiccProfile.updateExternalState(). 

Test: unit tests
Bug: 72513243
Merged-in: I23432138e1c3f8db74b91cab9759594217385fdf
Change-Id: I23432138e1c3f8db74b91cab9759594217385fdf
parent 06bf7483

src/java/com/android/internal/telephony/uicc/UiccProfile.java

+6 −1
Original line number Diff line number Diff line
@@ -49,6 +49,7 @@ import android.util.LocalLog; 
import android.view.WindowManager;



import com.android.internal.R;

import com.android.internal.annotations.VisibleForTesting;

import com.android.internal.telephony.CommandsInterface;

import com.android.internal.telephony.IccCard;

import com.android.internal.telephony.IccCardConstants;
@@ -316,7 +317,11 @@ public class UiccProfile extends Handler implements IccCard { 
        }

    }



    private void updateExternalState() {

    /**

     * Update the external SIM state

     */

    @VisibleForTesting(visibility = VisibleForTesting.Visibility.PRIVATE)

    public void updateExternalState() {

        // First check if card state is IO_ERROR or RESTRICTED

        if (mUiccCard.getCardState() == IccCardStatus.CardState.CARDSTATE_ERROR) {

            setExternalState(IccCardConstants.State.CARD_IO_ERROR);

tests/telephonytests/src/com/android/internal/telephony/uicc/UiccProfileTest.java

+27 −1
Original line number Diff line number Diff line
@@ -16,6 +16,7 @@ 
package com.android.internal.telephony.uicc;



import static com.android.internal.telephony.TelephonyTestUtils.waitForMs;



import static org.junit.Assert.assertEquals;

import static org.junit.Assert.assertFalse;

import static org.junit.Assert.assertNull;
@@ -23,6 +24,7 @@ import static org.junit.Assert.assertTrue; 
import static org.mockito.Mockito.anyInt;

import static org.mockito.Mockito.anyString;

import static org.mockito.Mockito.atLeast;

import static org.mockito.Mockito.doReturn;

import static org.mockito.Mockito.isA;

import static org.mockito.Mockito.times;

import static org.mockito.Mockito.verify;
@@ -30,7 +32,7 @@ import static org.mockito.Mockito.verify; 
import android.os.Handler;

import android.os.HandlerThread;

import android.os.Message;

import android.test.suitebuilder.annotation.SmallTest;

import android.support.test.filters.SmallTest;



import com.android.internal.telephony.IccCardConstants.State;

import com.android.internal.telephony.TelephonyTest;
@@ -404,4 +406,28 @@ public class UiccProfileTest extends TelephonyTest { 
        // state is loaded since there is no applications.

        assertEquals(State.NOT_READY, mUiccProfile.getState());

    }



    @Test

    @SmallTest

    public void testUpdateExternalState() {

        // IO_ERROR

        doReturn(IccCardStatus.CardState.CARDSTATE_ERROR).when(mUiccCard).getCardState();

        mUiccProfile.updateExternalState();

        assertEquals(State.CARD_IO_ERROR, mUiccProfile.getState());



        // RESTRICTED

        doReturn(IccCardStatus.CardState.CARDSTATE_RESTRICTED).when(mUiccCard).getCardState();

        mUiccProfile.updateExternalState();

        assertEquals(State.CARD_RESTRICTED, mUiccProfile.getState());



        // CARD PRESENT; no mUiccApplication - state should be NOT_READY

        doReturn(IccCardStatus.CardState.CARDSTATE_PRESENT).when(mUiccCard).getCardState();

        mUiccProfile.updateExternalState();

        assertEquals(State.NOT_READY, mUiccProfile.getState());



        // set mUiccApplication

        testUpdateUiccProfileApplicationAllReady();

        mUiccProfile.updateExternalState();

        assertEquals(State.LOADED, mUiccProfile.getState());

    }

}