Merge "Make pre-upgrade footer appear more consistently"
This commit is contained in:
committed by
Android (Google) Code Review
commit
afb30cdd08
@@ -58,7 +58,6 @@ public class PreUpgradePreferenceController extends BasePreferenceController {
|
|||||||
|
|
||||||
@Override
|
@Override
|
||||||
public int getAvailabilityStatus() {
|
public int getAvailabilityStatus() {
|
||||||
if (mNm.isNotificationListenerAccessGranted(mCn)) {
|
|
||||||
mNlf = mNm.getListenerFilter(mCn, mUserId);
|
mNlf = mNm.getListenerFilter(mCn, mUserId);
|
||||||
|
|
||||||
if (mTargetSdk > Build.VERSION_CODES.S) {
|
if (mTargetSdk > Build.VERSION_CODES.S) {
|
||||||
@@ -69,8 +68,5 @@ public class PreUpgradePreferenceController extends BasePreferenceController {
|
|||||||
return CONDITIONALLY_UNAVAILABLE;
|
return CONDITIONALLY_UNAVAILABLE;
|
||||||
}
|
}
|
||||||
return AVAILABLE;
|
return AVAILABLE;
|
||||||
} else {
|
|
||||||
return CONDITIONALLY_UNAVAILABLE;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
@@ -62,17 +62,8 @@ public class PreUpgradePreferenceControllerTest {
|
|||||||
mController.setUserId(0);
|
mController.setUserId(0);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
|
||||||
public void testAvailable_notGranted() {
|
|
||||||
when(mNm.isNotificationListenerAccessGranted(any())).thenReturn(false);
|
|
||||||
mController.setTargetSdk(Build.VERSION_CODES.S);
|
|
||||||
|
|
||||||
assertThat(mController.getAvailabilityStatus()).isEqualTo(CONDITIONALLY_UNAVAILABLE);
|
|
||||||
}
|
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void testAvailable_lowTargetSdk_noCustomizations() {
|
public void testAvailable_lowTargetSdk_noCustomizations() {
|
||||||
when(mNm.isNotificationListenerAccessGranted(any())).thenReturn(true);
|
|
||||||
mController.setTargetSdk(Build.VERSION_CODES.S);
|
mController.setTargetSdk(Build.VERSION_CODES.S);
|
||||||
when(mNm.getListenerFilter(mCn, 0)).thenReturn(new NotificationListenerFilter());
|
when(mNm.getListenerFilter(mCn, 0)).thenReturn(new NotificationListenerFilter());
|
||||||
|
|
||||||
@@ -81,7 +72,6 @@ public class PreUpgradePreferenceControllerTest {
|
|||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void testAvailable_lowTargetSdk_customizations() {
|
public void testAvailable_lowTargetSdk_customizations() {
|
||||||
when(mNm.isNotificationListenerAccessGranted(any())).thenReturn(true);
|
|
||||||
mController.setTargetSdk(Build.VERSION_CODES.S);
|
mController.setTargetSdk(Build.VERSION_CODES.S);
|
||||||
NotificationListenerFilter nlf = new NotificationListenerFilter();
|
NotificationListenerFilter nlf = new NotificationListenerFilter();
|
||||||
nlf.setTypes(FLAG_FILTER_TYPE_CONVERSATIONS);
|
nlf.setTypes(FLAG_FILTER_TYPE_CONVERSATIONS);
|
||||||
@@ -92,7 +82,6 @@ public class PreUpgradePreferenceControllerTest {
|
|||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void testAvailable_highTargetSdk_noCustomizations() {
|
public void testAvailable_highTargetSdk_noCustomizations() {
|
||||||
when(mNm.isNotificationListenerAccessGranted(any())).thenReturn(true);
|
|
||||||
mController.setTargetSdk(Build.VERSION_CODES.CUR_DEVELOPMENT + 1);
|
mController.setTargetSdk(Build.VERSION_CODES.CUR_DEVELOPMENT + 1);
|
||||||
when(mNm.getListenerFilter(mCn, 0)).thenReturn(new NotificationListenerFilter());
|
when(mNm.getListenerFilter(mCn, 0)).thenReturn(new NotificationListenerFilter());
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user