Do not allow to contribute for Gms related apps

Author: jonathanklee

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

@@ -15,7 +15,7 @@ class InstalledApplicationsRepository @Inject constructor() {
     fun getAppList(context: Context): List<InstalledApplication> {
         val apps = context.packageManager.getInstalledApplications(PackageManager.GET_META_DATA)
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
-            apps.removeIf { x -> isSystemApp(x) }
+            apps.removeIf { x -> isSystemApp(x) || isGmsRelated(x) }
         }
 
         val results: MutableList<InstalledApplication> = arrayListOf()
@@ -60,4 +60,9 @@ class InstalledApplicationsRepository @Inject constructor() {
     fun isSystemApp(info: ApplicationInfo): Boolean {
         return info.flags and ApplicationInfo.FLAG_SYSTEM != 0
     }
+
+    @VisibleForTesting
+    fun isGmsRelated(info: ApplicationInfo): Boolean {
+        return info.packageName.contains(".gms.")
+    }
 }

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

@@ -45,6 +45,7 @@ class InstalledApplicationsRepositoryTest {
 
     private lateinit var fakeRegularApplicationInfo: ApplicationInfo
     private lateinit var fakeSystemApplicationInfo: ApplicationInfo
+    private lateinit var fakeGmsApp: ApplicationInfo
 
     private var fakeListApplicationInfo: List<ApplicationInfo>? = null
 
@@ -66,9 +67,15 @@ class InstalledApplicationsRepositoryTest {
             flags = ApplicationInfo.FLAG_SYSTEM
         }
 
+        fakeGmsApp = ApplicationInfo().apply {
+            packageName = "fake.package.gms.three"
+            name = "FakeApplicationThree"
+        }
+
         fakeListApplicationInfo = mutableListOf(
             fakeRegularApplicationInfo,
-            fakeSystemApplicationInfo
+            fakeSystemApplicationInfo,
+            fakeGmsApp
         )
 
         Mockito.`when`(mockedContext.packageManager).thenReturn(mockedPackageManager)
@@ -103,6 +110,24 @@ class InstalledApplicationsRepositoryTest {
         )
     }
 
+    @Test
+    fun test_isGmsWithRegularApp() {
+        Assert.assertEquals(
+            "App status error",
+            false,
+            repository.isGmsRelated(fakeRegularApplicationInfo)
+        )
+    }
+
+    @Test
+    fun test_isGmsWithGmsApp() {
+        Assert.assertEquals(
+            "App status error",
+            true,
+            repository.isGmsRelated(fakeGmsApp)
+        )
+    }
+
     @Test
     fun listApplicationCheckListSize() {
         val list = repository.getAppList(mockedContext)