Skip to content

Commit bee6a06

Browse files
committed
Fix evaluations with badged icons
1 parent 8bf43de commit bee6a06

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

app/src/main/java/com/klee/sapio/data/InstalledApplicationsRepository.kt

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,18 @@ class InstalledApplicationsRepository @Inject constructor() {
4848
return InstalledApplication(
4949
packageManager.getApplicationLabel(info).toString(),
5050
info.packageName,
51-
packageManager.getApplicationIcon(info)
51+
fetchIcon(packageManager, info)
5252
)
5353
}
5454

55+
private fun fetchIcon(packageManager: PackageManager, info: ApplicationInfo): Drawable {
56+
return if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP_MR1) {
57+
info.loadUnbadgedIcon(packageManager)
58+
} else {
59+
info.loadIcon(packageManager)
60+
}
61+
}
62+
5563
@VisibleForTesting
5664
fun isSystemApp(info: ApplicationInfo): Boolean {
5765
return info.flags and ApplicationInfo.FLAG_SYSTEM != 0

app/src/test/java/com/klee/sapio/InstalledApplicationsRepositoryTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ class InstalledApplicationsRepositoryTest {
8181
Mockito.`when`(mockedPackageManager.getApplicationLabel(eq(fakeSystemApplicationInfo)))
8282
.thenReturn("FakeApplicationTwo")
8383

84-
Mockito.`when`(mockedPackageManager.getApplicationIcon(eq(fakeRegularApplicationInfo)))
84+
Mockito.`when`(fakeRegularApplicationInfo.loadUnbadgedIcon(mockedPackageManager))
8585
.thenReturn(mockedDrawable)
8686
}
8787

0 commit comments

Comments
 (0)