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

Commit 97d409d4 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "DNC set retry state FAILED if evaluation fails" am: 1a126858 

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

Change-Id: Iec96107c4e2a043c9104d8d348db39cefb6d490c
parents 97451f5a 1a126858

src/java/com/android/internal/telephony/data/DataNetworkController.java

+3 −0
Original line number Original line Diff line number Diff line
@@ -2079,7 +2079,10 @@ public class DataNetworkController extends Handler { 
                        evaluation.getDataAllowedReason());

                        evaluation.getDataAllowedReason());

            } else {

            } else {

                loge("onDataNetworkSetupRetry: Not able to find a suitable data profile to retry.");

                loge("onDataNetworkSetupRetry: Not able to find a suitable data profile to retry.");

                dataSetupRetryEntry.setState(DataRetryEntry.RETRY_STATE_FAILED);

            }

            }

        } else {

            dataSetupRetryEntry.setState(DataRetryEntry.RETRY_STATE_FAILED);

        }

        }

    }

    }

tests/telephonytests/src/com/android/internal/telephony/data/DataNetworkControllerTest.java

+51 −0
Original line number Original line Diff line number Diff line
@@ -1282,6 +1282,57 @@ public class DataNetworkControllerTest extends TelephonyTest { 
                any(), any(), anyBoolean(), any(Message.class));

                any(), any(), anyBoolean(), any(Message.class));

    }

    }





    @Test

    public void testSetupDataNetworkRetryFailed() {

        mDataNetworkControllerUT.getDataRetryManager()

                .registerCallback(mMockedDataRetryManagerCallback);

        setFailedSetupDataResponse(mMockedWwanDataServiceManager, DataFailCause.CONGESTION,

                DataCallResponse.RETRY_DURATION_UNDEFINED);

        mDataNetworkControllerUT.addNetworkRequest(

                createNetworkRequest(NetworkCapabilities.NET_CAPABILITY_INTERNET));

        processAllMessages();



        verify(mMockedWwanDataServiceManager, times(1)).setupDataCall(anyInt(),

                any(DataProfile.class), anyBoolean(), anyBoolean(), anyInt(), any(), anyInt(),

                any(), any(), anyBoolean(), any(Message.class));



        // Process first retry

        moveTimeForward(2500);

        processAllMessages();

        verify(mMockedWwanDataServiceManager, times(2)).setupDataCall(anyInt(),

                any(DataProfile.class), anyBoolean(), anyBoolean(), anyInt(), any(), anyInt(),

                any(), any(), anyBoolean(), any(Message.class));

        ArgumentCaptor<DataRetryManager.DataSetupRetryEntry> retryEntry =

                ArgumentCaptor.forClass(DataRetryManager.DataSetupRetryEntry.class);

        verify(mMockedDataRetryManagerCallback, times(1))

                .onDataNetworkSetupRetry(retryEntry.capture());

        assertThat(retryEntry.getValue().getState()).isEqualTo(

                DataRetryManager.DataRetryEntry.RETRY_STATE_FAILED);



        // Cause data network setup failed due to RADIO_DISABLED_BY_CARRIER

        doReturn(false).when(mSST).getPowerStateFromCarrier();



        // Process second retry and ensure data network setup failed

        moveTimeForward(3000);

        processAllMessages();

        verify(mMockedWwanDataServiceManager, times(2)).setupDataCall(anyInt(),

                any(DataProfile.class), anyBoolean(), anyBoolean(), anyInt(), any(), anyInt(),

                any(), any(), anyBoolean(), any(Message.class));

        verify(mMockedDataRetryManagerCallback, times(2))

                .onDataNetworkSetupRetry(retryEntry.capture());

        assertThat(retryEntry.getValue().getState()).isEqualTo(

                DataRetryManager.DataRetryEntry.RETRY_STATE_FAILED);



        // Data network setup allowed again

        doReturn(true).when(mSST).getPowerStateFromCarrier();



        // Should not retry again after retry failure

        processAllFutureMessages();

        verify(mMockedWwanDataServiceManager, times(2)).setupDataCall(anyInt(),

                any(DataProfile.class), anyBoolean(), anyBoolean(), anyInt(), any(), anyInt(),

                any(), any(), anyBoolean(), any(Message.class));

    }



    @Test

    @Test

    public void testSetupDataNetworkPermanentFailure() {

    public void testSetupDataNetworkPermanentFailure() {

        setFailedSetupDataResponse(mMockedWwanDataServiceManager, DataFailCause.PROTOCOL_ERRORS,

        setFailedSetupDataResponse(mMockedWwanDataServiceManager, DataFailCause.PROTOCOL_ERRORS,