PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Fix unexpected Wi-Fi hotspot shutdown" into tm-dev am: 3ed73b7b2d

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/17630131

Change-Id: I2535c12e487cce2beb462995f0abda80415cf626
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
TreeHugger Robot
2022-04-09 09:22:00 +00:00
committed by Automerger Merge Worker
parent e82d15ef8a 3ed73b7b2d
commit 66903e1328
3 changed files with 109 additions and 48 deletions
Download Patch File Download Diff File Expand all files Collapse all files

107
tests/robotests/src/com/android/settings/wifi/tether/WifiTetherSwitchBarControllerTest.java
View File

@@ -16,66 +16,76 @@
package com.android.settings.wifi.tether;
import static android.net.ConnectivityManager.TETHERING_WIFI;
import static android.net.wifi.WifiManager.WIFI_AP_STATE_DISABLED;
import static android.net.wifi.WifiManager.WIFI_AP_STATE_DISABLING;
import static android.net.wifi.WifiManager.WIFI_AP_STATE_ENABLED;
import static android.net.wifi.WifiManager.WIFI_AP_STATE_ENABLING;
import static android.net.wifi.WifiManager.WIFI_AP_STATE_FAILED;
import static com.google.common.truth.Truth.assertThat;
import static org.mockito.ArgumentMatchers.any;
import static org.mockito.ArgumentMatchers.anyBoolean;
import static org.mockito.ArgumentMatchers.eq;
import static org.mockito.Mockito.spy;
import static org.mockito.ArgumentMatchers.anyInt;
import static org.mockito.Mockito.never;
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
import android.content.Context;
import android.net.ConnectivityManager;
import android.net.NetworkPolicyManager;
import android.net.wifi.WifiManager;
import android.widget.Switch;
import androidx.test.core.app.ApplicationProvider;
import com.android.settings.datausage.DataSaverBackend;
import com.android.settings.widget.SettingsMainSwitchBar;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.Mock;
import org.mockito.MockitoAnnotations;
import org.mockito.Spy;
import org.mockito.junit.MockitoJUnit;
import org.mockito.junit.MockitoRule;
import org.robolectric.RobolectricTestRunner;
import org.robolectric.RuntimeEnvironment;
@RunWith(RobolectricTestRunner.class)
public class WifiTetherSwitchBarControllerTest {
@Rule
public final MockitoRule mMockitoRule = MockitoJUnit.rule();
@Spy
Context mContext = ApplicationProvider.getApplicationContext();
@Mock
private WifiManager mWifiManager;
@Mock
private ConnectivityManager mConnectivityManager;
@Mock
private NetworkPolicyManager mNetworkPolicyManager;
private DataSaverBackend mDataSaverBackend;
@Mock
private Switch mSwitch;
private Context mContext;
private SettingsMainSwitchBar mSwitchBar;
private WifiTetherSwitchBarController mController;
@Before
public void setUp() {
MockitoAnnotations.initMocks(this);
mContext = spy(RuntimeEnvironment.application);
mSwitchBar = new SettingsMainSwitchBar(mContext);
when(mContext.getSystemService(Context.WIFI_SERVICE)).thenReturn(mWifiManager);
when(mContext.getSystemService(Context.CONNECTIVITY_SERVICE)).thenReturn(
mConnectivityManager);
when(mContext.getSystemService(Context.NETWORK_POLICY_SERVICE)).thenReturn(
mNetworkPolicyManager);
when(mDataSaverBackend.isDataSaverEnabled()).thenReturn(false);
when(mSwitch.isEnabled()).thenReturn(true);
mController = new WifiTetherSwitchBarController(mContext, mSwitchBar);
mController.mDataSaverBackend = mDataSaverBackend;
}
@Test
public void startTether_fail_resetSwitchBar() {
when(mNetworkPolicyManager.getRestrictBackground()).thenReturn(false);
when(mDataSaverBackend.isDataSaverEnabled()).thenReturn(false);
mController.startTether();
mController.mOnStartTetheringCallback.onTetheringFailed();
@@ -89,23 +99,33 @@ public class WifiTetherSwitchBarControllerTest {
assertThat(mSwitchBar.isEnabled()).isTrue();
// try to turn data saver on
when(mNetworkPolicyManager.getRestrictBackground()).thenReturn(true);
when(mDataSaverBackend.isDataSaverEnabled()).thenReturn(true);
mController.onDataSaverChanged(true);
assertThat(mSwitchBar.isEnabled()).isFalse();
// lets turn data saver off again
when(mNetworkPolicyManager.getRestrictBackground()).thenReturn(false);
when(mDataSaverBackend.isDataSaverEnabled()).thenReturn(false);
mController.onDataSaverChanged(false);
assertThat(mSwitchBar.isEnabled()).isTrue();
}
@Test
public void onSwitchChanged_switchNotEnabled_doNothingForTethering() {
when(mSwitch.isEnabled()).thenReturn(false);
mController.onSwitchChanged(mSwitch, mSwitch.isChecked());
verify(mConnectivityManager, never()).startTethering(anyInt(), anyBoolean(), any(), any());
verify(mConnectivityManager, never()).stopTethering(anyInt());
}
@Test
public void onSwitchChanged_isChecked_startTethering() {
when(mSwitch.isChecked()).thenReturn(true);
mController.onSwitchChanged(mSwitch, mSwitch.isChecked());
verify(mConnectivityManager).startTethering(eq(TETHERING_WIFI), anyBoolean(), any(), any());
verify(mConnectivityManager).startTethering(anyInt(), anyBoolean(), any(), any());
}
@Test
@@ -114,6 +134,57 @@ public class WifiTetherSwitchBarControllerTest {
mController.onSwitchChanged(mSwitch, mSwitch.isChecked());
verify(mConnectivityManager).stopTethering(TETHERING_WIFI);
verify(mConnectivityManager).stopTethering(anyInt());
}
@Test
public void handleWifiApStateChanged_stateIsEnabling_notEnabledSwitchBar() {
mSwitchBar.setEnabled(false);
mController.handleWifiApStateChanged(WIFI_AP_STATE_ENABLING);
assertThat(mSwitchBar.isEnabled()).isFalse();
}
@Test
public void handleWifiApStateChanged_stateIsDisabling_notEnabledSwitchBar() {
mSwitchBar.setEnabled(false);
mController.handleWifiApStateChanged(WIFI_AP_STATE_DISABLING);
assertThat(mSwitchBar.isEnabled()).isFalse();
}
@Test
public void handleWifiApStateChanged_stateIsEnabled_enabledAndCheckedSwitchBar() {
mSwitchBar.setEnabled(false);
mSwitchBar.setChecked(false);
mController.handleWifiApStateChanged(WIFI_AP_STATE_ENABLED);
assertThat(mSwitchBar.isEnabled()).isTrue();
assertThat(mSwitchBar.isChecked()).isTrue();
}
@Test
public void handleWifiApStateChanged_stateIsDisabled_enabledAndUncheckedSwitchBar() {
mSwitchBar.setEnabled(false);
mSwitchBar.setChecked(true);
mController.handleWifiApStateChanged(WIFI_AP_STATE_DISABLED);
assertThat(mSwitchBar.isEnabled()).isTrue();
assertThat(mSwitchBar.isChecked()).isFalse();
}
@Test
public void handleWifiApStateChanged_stateIsFailed_enabledAndUncheckedSwitchBar() {
mSwitchBar.setEnabled(false);
mSwitchBar.setChecked(true);
mController.handleWifiApStateChanged(WIFI_AP_STATE_FAILED);
assertThat(mSwitchBar.isEnabled()).isTrue();
assertThat(mSwitchBar.isChecked()).isFalse();
}
}