Merge "Do not show a summary in Tail button preferences if no app found" into udc-qpr-dev
This commit is contained in:
@@ -124,7 +124,7 @@ public class StylusDevicesController extends AbstractPreferenceController implem
|
|||||||
try {
|
try {
|
||||||
ApplicationInfo ai = pm.getApplicationInfo(packageName,
|
ApplicationInfo ai = pm.getApplicationInfo(packageName,
|
||||||
PackageManager.ApplicationInfoFlags.of(0));
|
PackageManager.ApplicationInfoFlags.of(0));
|
||||||
appName = ai == null ? packageName : pm.getApplicationLabel(ai).toString();
|
appName = ai == null ? "" : pm.getApplicationLabel(ai).toString();
|
||||||
} catch (PackageManager.NameNotFoundException e) {
|
} catch (PackageManager.NameNotFoundException e) {
|
||||||
Log.e(TAG, "Notes role package not found.");
|
Log.e(TAG, "Notes role package not found.");
|
||||||
}
|
}
|
||||||
|
@@ -272,6 +272,20 @@ public class StylusDevicesControllerTest {
|
|||||||
NOTES_APP_LABEL.toString()));
|
NOTES_APP_LABEL.toString()));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void defaultNotesPreference_noApplicationInfo_showsBlankSummary()
|
||||||
|
throws PackageManager.NameNotFoundException {
|
||||||
|
when(mPm.getApplicationInfo(eq(NOTES_PACKAGE_NAME),
|
||||||
|
any(PackageManager.ApplicationInfoFlags.class))).thenReturn(null);
|
||||||
|
|
||||||
|
showScreen(mController);
|
||||||
|
|
||||||
|
Preference defaultNotesPref = mPreferenceContainer.getPreference(0);
|
||||||
|
assertThat(defaultNotesPref.getTitle().toString()).isEqualTo(
|
||||||
|
mContext.getString(R.string.stylus_default_notes_app));
|
||||||
|
assertThat(defaultNotesPref.getSummary().toString()).isEqualTo("");
|
||||||
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void defaultNotesPreference_roleHolderChanges_updatesPreference() {
|
public void defaultNotesPreference_roleHolderChanges_updatesPreference() {
|
||||||
showScreen(mController);
|
showScreen(mController);
|
||||||
|
Reference in New Issue
Block a user