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

Commit bc71e2d9 authored by Jisun Jung's avatar Jisun Jung
Browse files

Updates registered_times as soon as the UE is registered 

Bug: 312808192
Test: atest ImsStatsTest
Test: manual test to set registeredTimes when UE is registered.
Change-Id: I59a4e1cc11fce51c609dd8f4beae3b4cc3a99e75
parent d5aa1125

src/java/com/android/internal/telephony/metrics/ImsStats.java

+7 −6
Original line number Diff line number Diff line
@@ -338,6 +338,13 @@ public class ImsStats { 


    /** Updates the stats when IMS registration succeeds. */

    public synchronized void onImsRegistered(ImsRegistrationAttributes attributes) {

        // Updates registered_times as soon as the UE is registered

        if (mLastRegistrationState != REGISTRATION_STATE_REGISTERED) {

            // RegistrationStats captures in every state. Changing REGISTERED state has to capture

            // only once.

            mLastRegistrationStats.registeredTimes = 1;

        }



        conclude();



        mLastTransportType = attributes.getTransportType();
@@ -346,12 +353,6 @@ public class ImsStats { 
            updateImsRegistrationStats();

        }



        if (mLastRegistrationState != REGISTRATION_STATE_REGISTERED) {

            // RegistrationStats captures in every state. Changing REGISTERED state has to capture

            // only once.

            mLastRegistrationStats.registeredTimes = 1;

        }



        mLastRegistrationStats.rat =

                convertTransportTypeToNetworkType(attributes.getTransportType());

        mLastRegistrationStats.isIwlanCrossSim = attributes.getRegistrationTechnology()

tests/telephonytests/src/com/android/internal/telephony/metrics/ImsStatsTest.java

+1 −1
Original line number Diff line number Diff line
@@ -753,7 +753,7 @@ public class ImsStatsTest extends TelephonyTest { 
        assertEquals(TelephonyManager.NETWORK_TYPE_LTE, stats.rat);

        assertEquals(0L, stats.registeredMillis);

        assertEquals(2000L, stats.registeringMillis);

        assertEquals(0, stats.registeredTimes);

        assertEquals(1, stats.registeredTimes);

    }



    @Test