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

Merge "[Wi-Fi] Fix Wi-Fi broken tests"

Browse Source
This commit is contained in:
TreeHugger Robot
2020-03-20 08:53:54 +00:00
committed by Android (Google) Code Review
parent 68c2623fce 4fafe6716c
commit c2e24ebb2b
7 changed files with 35 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/robotests/src/com/android/settings/wifi/WifiScanningRequiredFragmentTest.java
View File

@@ -41,7 +41,6 @@ import androidx.fragment.app.FragmentActivity;
import com.android.settings.R;
import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.Mock;
@@ -79,7 +78,6 @@ public class WifiScanningRequiredFragmentTest {
}
@Test
@Ignore
public void onClick_positiveButtonSetsWifiScanningOn()
throws Settings.SettingNotFoundException {
mFragment.onClick(null, DialogInterface.BUTTON_POSITIVE);

1
tests/robotests/src/com/android/settings/wifi/WifiSettings2Test.java
View File

@@ -162,7 +162,6 @@ public class WifiSettings2Test {
}
@Test
@Ignore
public void setAdditionalSettingsSummaries_wifiWakeupEnabled_displayOn() {
final ContentResolver contentResolver = mContext.getContentResolver();
when(mWifiManager.isAutoWakeupEnabled()).thenReturn(true);

17
tests/robotests/src/com/android/settings/wifi/WifiSettingsTest.java
View File

@@ -49,6 +49,7 @@ import android.os.Bundle;
import android.os.PowerManager;
import android.os.UserManager;
import android.provider.Settings;
import android.util.FeatureFlagUtils;
import android.view.ContextMenu;
import android.view.View;
@@ -70,7 +71,6 @@ import com.android.settingslib.wifi.WifiTracker;
import com.android.settingslib.wifi.WifiTrackerFactory;
import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.ArgumentCaptor;
@@ -295,9 +295,12 @@ public class WifiSettingsTest {
}
@Test
@Ignore
@Config(shadows = {ShadowDataUsageUtils.class, ShadowFragment.class})
public void checkDataUsagePreference_perferenceInvisibleIfWifiNotSupported() {
if (FeatureFlagUtils.isEnabled(mContext, FeatureFlagUtils.SETTINGS_WIFITRACKER2)) {
return;
}
setUpForOnCreate();
ShadowDataUsageUtils.IS_WIFI_SUPPORTED = false;
@@ -307,9 +310,12 @@ public class WifiSettingsTest {
}
@Test
@Ignore
@Config(shadows = {ShadowDataUsageUtils.class, ShadowFragment.class})
public void checkDataUsagePreference_perferenceVisibleIfWifiSupported() {
if (FeatureFlagUtils.isEnabled(mContext, FeatureFlagUtils.SETTINGS_WIFITRACKER2)) {
return;
}
setUpForOnCreate();
ShadowDataUsageUtils.IS_WIFI_SUPPORTED = true;
@@ -352,9 +358,12 @@ public class WifiSettingsTest {
}
@Test
@Ignore
@Config(shadows = {ShadowDataUsageUtils.class, ShadowFragment.class})
public void clickOnWifiNetworkWith_shouldStartCaptivePortalApp() {
if (FeatureFlagUtils.isEnabled(mContext, FeatureFlagUtils.SETTINGS_WIFITRACKER2)) {
return;
}
when(mWifiManager.getConfiguredNetworks()).thenReturn(createMockWifiConfigurations(
NUM_NETWORKS));
when(mWifiTracker.isConnected()).thenReturn(true);

9
tests/robotests/src/com/android/settings/wifi/details/WifiDetailPreferenceControllerTest.java
View File

@@ -84,7 +84,6 @@ import com.android.settingslib.wifi.WifiTracker;
import com.android.settingslib.wifi.WifiTrackerFactory;
import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.Answers;
@@ -1428,7 +1427,6 @@ public class WifiDetailPreferenceControllerTest {
}
@Test
@Ignore
public void testConnectButton_clickConnect_displayAsSuccess() {
setUpForDisconnectedNetwork();
when(mockWifiManager.isWifiEnabled()).thenReturn(true);
@@ -1461,7 +1459,6 @@ public class WifiDetailPreferenceControllerTest {
}
@Test
@Ignore
public void testConnectButton_clickConnectButFailed_displayFailMessage() {
setUpForDisconnectedNetwork();
ArgumentCaptor<WifiManager.ActionListener> connectListenerCaptor =
@@ -1510,7 +1507,6 @@ public class WifiDetailPreferenceControllerTest {
}
@Test
@Ignore
public void testConnectButton_clickConnectButTimeout_displayFailMessage() {
setUpForDisconnectedNetwork();
when(mockWifiManager.isWifiEnabled()).thenReturn(true);
@@ -1540,7 +1536,6 @@ public class WifiDetailPreferenceControllerTest {
}
@Test
@Ignore
public void testConnectButton_clickConnectButTimeout_displayNotInRangeMessage() {
setUpForNotInRangeNetwork();
when(mockWifiManager.isWifiEnabled()).thenReturn(true);
@@ -1570,7 +1565,6 @@ public class WifiDetailPreferenceControllerTest {
}
@Test
@Ignore
public void testConnectButton_clickConnectWhenWiFiDisabled_displaySuccessMessage() {
setUpForDisconnectedNetwork();
when(mockWifiManager.isWifiEnabled()).thenReturn(false); // wifi disabled
@@ -1612,7 +1606,6 @@ public class WifiDetailPreferenceControllerTest {
}
@Test
@Ignore
public void testConnectButton_clickConnectWhenWiFiDisabled_failedToConnectWiFi() {
setUpForDisconnectedNetwork();
when(mockWifiManager.isWifiEnabled()).thenReturn(false); // wifi disabled
@@ -1651,7 +1644,6 @@ public class WifiDetailPreferenceControllerTest {
}
@Test
@Ignore
public void
testConnectButton_clickConnectWhenWiFiDisabled_failedToConnectWifiBecauseNotInRange() {
setUpForNotInRangeNetwork();
@@ -1691,7 +1683,6 @@ public class WifiDetailPreferenceControllerTest {
}
@Test
@Ignore
public void testConnectButton_clickConnectWhenWiFiDisabled_failedToEnableWifi() {
setUpForDisconnectedNetwork();
when(mockWifiManager.isWifiEnabled()).thenReturn(false); // wifi disabled

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

@@ -21,7 +21,7 @@ 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.anyInt;
import static org.mockito.Mockito.doReturn;
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.never;
import static org.mockito.Mockito.spy;
import static org.mockito.Mockito.times;
@@ -32,11 +32,11 @@ import android.content.Context;
import android.net.ConnectivityManager;
import android.net.NetworkPolicyManager;
import android.net.wifi.WifiManager;
import android.widget.Switch;
import com.android.settings.widget.SwitchBar;
import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.Mock;
@@ -99,15 +99,21 @@ public class WifiTetherSwitchBarControllerTest {
}
@Test
@Ignore
public void onSwitchToggled_onlyStartsTetherWhenNeeded() {
when(mWifiManager.isWifiApEnabled()).thenReturn(true);
mController.onClick(mSwitchBar.getSwitch());
public void onSwitchToggled_switchOff_noStartTethering() {
final Switch mockSwitch = mock(Switch.class);
when(mockSwitch.isChecked()).thenReturn(false);
mController.onClick(mockSwitch);
verify(mConnectivityManager, never()).startTethering(anyInt(), anyBoolean(), any(), any());
}
doReturn(false).when(mWifiManager).isWifiApEnabled();
mController.onClick(mSwitchBar.getSwitch());
@Test
public void onSwitchToggled_switchOn_startTethering() {
final Switch mockSwitch = mock(Switch.class);
when(mockSwitch.isChecked()).thenReturn(true);
mController.onClick(mockSwitch);
verify(mConnectivityManager, times(1))
.startTethering(anyInt(), anyBoolean(), any(), any());