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

Commit 87ae2b12 authored by yuemingw's avatar yuemingw Committed by Jayachandran Chinnakkannu
Browse files

Revert to previous constant value for ApnSetting.TYPE_DEFAULT. 

Bug: 113097474
Test: atest ApnSettingTest
atest DcTrackerTest

Change-Id: I9fe2463fb43540df056b81309dedc4bea1817a3a
parent 9ec9d708

tests/telephonytests/src/com/android/internal/telephony/dataconnection/ApnSettingTest.java

+2 −2
Original line number Diff line number Diff line
@@ -293,7 +293,7 @@ public class ApnSettingTest extends TelephonyTest { 
                ApnSetting.TYPE_DEFAULT, ApnSetting.PROTOCOL_IPV6, ApnSetting.PROTOCOL_IP, true,

                4096, 0, false, 0, 0, 0, 0, ApnSetting.MVNO_TYPE_SPN, "");

        String expected = "[ApnSettingV6] Name, 99, 12345, apn, null, "

                + "null, null, null, 10, 0, default, "

                + "null, null, null, 10, 0, hipri | default, "

                + "IPV6, IP, true, 0, false, 0, 0, 0, 0, spn, , false, 4096, 0, -1";

        assertEquals(expected, apn.toString());
@@ -304,7 +304,7 @@ public class ApnSettingTest extends TelephonyTest { 
                ApnSetting.TYPE_DEFAULT, ApnSetting.PROTOCOL_IPV6, ApnSetting.PROTOCOL_IP, true,

                networkTypeBitmask, 0, false, 0, 0, 0, 0, ApnSetting.MVNO_TYPE_SPN, "", 3, -1);

        expected = "[ApnSettingV6] Name, 99, 12345, apn, null, "

                + "null, null, null, 10, 0, default, "

                + "null, null, null, 10, 0, hipri | default, "

                + "IPV6, IP, true, 0, false, 0, 0, 0, 0, spn, , false, 8192, 3, -1";

        assertEquals(expected, apn.toString());

    }

tests/telephonytests/src/com/android/internal/telephony/dataconnection/DcTrackerTest.java

+7 −7
Original line number Diff line number Diff line
@@ -596,7 +596,7 @@ public class DcTrackerTest extends TelephonyTest { 
                        mServiceState.getRilDataRadioTechnology())), dpCaptor.capture(),

                eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),

                any(Message.class));

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 5, 1, LTE_BEARER_BITMASK);

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 21, 1, LTE_BEARER_BITMASK);



        verifyDataConnected(FAKE_APN1);

    }
@@ -645,7 +645,7 @@ public class DcTrackerTest extends TelephonyTest { 
                        mServiceState.getRilDataRadioTechnology())), dpCaptor.capture(),

                eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),

                any(Message.class));

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 5, 1, LTE_BEARER_BITMASK);

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 21, 1, LTE_BEARER_BITMASK);



        // Verify the retry manger schedule another data call setup.

        verify(mAlarmManager, times(1)).setExact(eq(AlarmManager.ELAPSED_REALTIME_WAKEUP),
@@ -670,7 +670,7 @@ public class DcTrackerTest extends TelephonyTest { 
                        mServiceState.getRilDataRadioTechnology())), dpCaptor.capture(),

                eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),

                any(Message.class));

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN2, 0, 5, 1, LTE_BEARER_BITMASK);

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN2, 0, 21, 1, LTE_BEARER_BITMASK);



        // Verify connected with APN2 setting.

        verifyDataConnected(FAKE_APN2);
@@ -751,7 +751,7 @@ public class DcTrackerTest extends TelephonyTest { 
                        mServiceState.getRilDataRadioTechnology())), dpCaptor.capture(),

                eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),

                any(Message.class));

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 5, 1, LTE_BEARER_BITMASK);

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 21, 1, LTE_BEARER_BITMASK);



        //user is in roaming

        doReturn(true).when(mServiceState).getDataRoaming();
@@ -1205,7 +1205,7 @@ public class DcTrackerTest extends TelephonyTest { 
                        mServiceState.getRilDataRadioTechnology())), dpCaptor.capture(),

                eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),

                any(Message.class));

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN4, 0, 5, 2, EHRPD_BEARER_BITMASK);

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN4, 0, 21, 2, EHRPD_BEARER_BITMASK);

        assertEquals(DctConstants.State.CONNECTED, mDct.getOverallState());



        //data rat change from ehrpd to lte
@@ -1238,7 +1238,7 @@ public class DcTrackerTest extends TelephonyTest { 
                        mServiceState.getRilDataRadioTechnology())), dpCaptor.capture(),

                eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),

                any(Message.class));

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 5, 1, LTE_BEARER_BITMASK);

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN1, 0, 21, 1, LTE_BEARER_BITMASK);

        assertEquals(DctConstants.State.CONNECTED, mDct.getOverallState());

    }
@@ -1325,7 +1325,7 @@ public class DcTrackerTest extends TelephonyTest { 
                        mServiceState.getRilDataRadioTechnology())), dpCaptor.capture(),

                eq(false), eq(false), eq(DataService.REQUEST_REASON_NORMAL), any(),

                any(Message.class));

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN4, 0, 5, 2, EHRPD_BEARER_BITMASK);

        verifyDataProfile(dpCaptor.getValue(), FAKE_APN4, 0, 21, 2, EHRPD_BEARER_BITMASK);

        assertEquals(DctConstants.State.CONNECTED, mDct.getOverallState());



        // Data rat change from ehrpd to unknown due to OOS