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

Commit 67b2c16a authored by Jason Monk's avatar Jason Monk
Browse files

Get data type from service state as well

Bug: 26276354
Bug: 26413831
Change-Id: Ic080300db7d5e5200caf611cc2ee1ef3af93736e
parent a73abeaf
Loading
Loading
Loading
Loading
+1 −0
Original line number Original line Diff line number Diff line
@@ -440,6 +440,7 @@ public class MobileSignalController extends SignalController<
                        + " dataState=" + state.getDataRegState());
                        + " dataState=" + state.getDataRegState());
            }
            }
            mServiceState = state;
            mServiceState = state;
            mDataNetType = state.getDataNetworkType();
            updateTelephony();
            updateTelephony();
        }
        }


+4 −3
Original line number Original line Diff line number Diff line
@@ -59,8 +59,8 @@ public class NetworkControllerBaseTest extends SysuiTestCase {
    protected NetworkControllerImpl mNetworkController;
    protected NetworkControllerImpl mNetworkController;
    protected MobileSignalController mMobileSignalController;
    protected MobileSignalController mMobileSignalController;
    protected PhoneStateListener mPhoneStateListener;
    protected PhoneStateListener mPhoneStateListener;
    private SignalStrength mSignalStrength;
    protected SignalStrength mSignalStrength;
    private ServiceState mServiceState;
    protected ServiceState mServiceState;
    protected ConnectivityManager mMockCm;
    protected ConnectivityManager mMockCm;
    protected WifiManager mMockWm;
    protected WifiManager mMockWm;
    protected SubscriptionManager mMockSm;
    protected SubscriptionManager mMockSm;
@@ -234,7 +234,7 @@ public class NetworkControllerBaseTest extends SysuiTestCase {
        mPhoneStateListener.onSignalStrengthsChanged(mSignalStrength);
        mPhoneStateListener.onSignalStrengthsChanged(mSignalStrength);
    }
    }


    private void updateServiceState() {
    protected void updateServiceState() {
        Log.d(TAG, "Sending Service State: " + mServiceState);
        Log.d(TAG, "Sending Service State: " + mServiceState);
        mPhoneStateListener.onServiceStateChanged(mServiceState);
        mPhoneStateListener.onServiceStateChanged(mServiceState);
    }
    }
@@ -245,6 +245,7 @@ public class NetworkControllerBaseTest extends SysuiTestCase {
    }
    }


    public void updateDataConnectionState(int dataState, int dataNetType) {
    public void updateDataConnectionState(int dataState, int dataNetType) {
        when(mServiceState.getDataNetworkType()).thenReturn(dataNetType);
        mPhoneStateListener.onDataConnectionStateChanged(dataState, dataNetType);
        mPhoneStateListener.onDataConnectionStateChanged(dataState, dataNetType);
    }
    }


+15 −0
Original line number Original line Diff line number Diff line
@@ -114,6 +114,21 @@ public class NetworkControllerDataTest extends NetworkControllerBaseTest {
                TelephonyIcons.QS_DATA_4G);
                TelephonyIcons.QS_DATA_4G);
    }
    }


    public void testDataChangeWithoutConnectionState() {
        setupDefaultSignal();
        updateDataConnectionState(TelephonyManager.DATA_CONNECTED,
                TelephonyManager.NETWORK_TYPE_LTE);

        verifyDataIndicators(TelephonyIcons.DATA_LTE[1][0 /* No direction */],
                TelephonyIcons.QS_DATA_LTE);

        Mockito.when(mServiceState.getDataNetworkType())
                .thenReturn(TelephonyManager.NETWORK_TYPE_HSPA);
        updateServiceState();
        verifyDataIndicators(TelephonyIcons.DATA_H[1][0 /* No direction */],
                TelephonyIcons.QS_DATA_H);
    }

    public void testDataActivity() {
    public void testDataActivity() {
        setupDefaultSignal();
        setupDefaultSignal();