Merge "[Provider Model] Don't show Wi-Fi icon if the level is -1" into sc-qpr1-dev
This commit is contained in:
committed by
Android (Google) Code Review
commit
ebd9d182d0
@@ -337,6 +337,9 @@ public class InternetDialogController implements WifiEntry.DisconnectCallback,
|
|||||||
}
|
}
|
||||||
|
|
||||||
Drawable getInternetWifiDrawable(@NonNull WifiEntry wifiEntry) {
|
Drawable getInternetWifiDrawable(@NonNull WifiEntry wifiEntry) {
|
||||||
|
if (wifiEntry.getLevel() == WifiEntry.WIFI_LEVEL_UNREACHABLE) {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
final Drawable drawable =
|
final Drawable drawable =
|
||||||
mWifiIconInjector.getIcon(wifiEntry.shouldShowXLevelIcon(), wifiEntry.getLevel());
|
mWifiIconInjector.getIcon(wifiEntry.shouldShowXLevelIcon(), wifiEntry.getLevel());
|
||||||
if (drawable == null) {
|
if (drawable == null) {
|
||||||
|
|||||||
@@ -259,7 +259,7 @@ public class InternetDialogControllerTest extends SysuiTestCase {
|
|||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void getWifiDrawable_withConnectedEntry_returnIntentIconWithCorrectColor() {
|
public void getInternetWifiDrawable_withConnectedEntry_returnIntentIconWithCorrectColor() {
|
||||||
final Drawable drawable = mock(Drawable.class);
|
final Drawable drawable = mock(Drawable.class);
|
||||||
when(mWifiIconInjector.getIcon(anyBoolean(), anyInt())).thenReturn(drawable);
|
when(mWifiIconInjector.getIcon(anyBoolean(), anyInt())).thenReturn(drawable);
|
||||||
|
|
||||||
@@ -269,6 +269,15 @@ public class InternetDialogControllerTest extends SysuiTestCase {
|
|||||||
verify(drawable).setTint(mContext.getColor(R.color.connected_network_primary_color));
|
verify(drawable).setTint(mContext.getColor(R.color.connected_network_primary_color));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void getInternetWifiDrawable_withWifiLevelUnreachable_returnNull() {
|
||||||
|
when(mConnectedEntry.getLevel()).thenReturn(WifiEntry.WIFI_LEVEL_UNREACHABLE);
|
||||||
|
|
||||||
|
Drawable drawable = mInternetDialogController.getInternetWifiDrawable(mConnectedEntry);
|
||||||
|
|
||||||
|
assertThat(drawable).isNull();
|
||||||
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void launchWifiNetworkDetailsSetting_withNoWifiEntryKey_doNothing() {
|
public void launchWifiNetworkDetailsSetting_withNoWifiEntryKey_doNothing() {
|
||||||
mInternetDialogController.launchWifiNetworkDetailsSetting(null /* key */);
|
mInternetDialogController.launchWifiNetworkDetailsSetting(null /* key */);
|
||||||
|
|||||||
Reference in New Issue
Block a user