浏览代码

work on codacy warnings

Signed-off-by: sowjanyakch <sowjanya.kch@gmail.com>
sowjanyakch 9 月之前
父节点
当前提交
4508cbb0aa

+ 4 - 3
app/src/main/java/com/nextcloud/talk/contacts/MyApplication.kt → app/src/main/java/com/nextcloud/talk/contacts/ContactsApplication.kt

@@ -13,18 +13,19 @@ import coil.ImageLoaderFactory
 import coil.disk.DiskCache
 import coil.disk.DiskCache
 import coil.memory.MemoryCache
 import coil.memory.MemoryCache
 import coil.util.DebugLogger
 import coil.util.DebugLogger
+import com.nextcloud.talk.utils.ContactUtils
 
 
-class MyApplication : Application(), ImageLoaderFactory {
+class ContactsApplication : Application(), ImageLoaderFactory {
     override fun newImageLoader(): ImageLoader {
     override fun newImageLoader(): ImageLoader {
         val imageLoader = ImageLoader.Builder(this)
         val imageLoader = ImageLoader.Builder(this)
             .memoryCache {
             .memoryCache {
                 MemoryCache.Builder(this)
                 MemoryCache.Builder(this)
-                    .maxSizePercent(0.1)
+                    .maxSizePercent(ContactUtils.CACHE_MEMORY_SIZE_PERCENTAGE)
                     .build()
                     .build()
             }
             }
             .diskCache {
             .diskCache {
                 DiskCache.Builder()
                 DiskCache.Builder()
-                    .maxSizePercent(0.02)
+                    .maxSizePercent(ContactUtils.CACHE_DISK_SIZE_PERCENTAGE)
                     .directory(cacheDir)
                     .directory(cacheDir)
                     .build()
                     .build()
             }
             }

+ 2 - 0
app/src/main/java/com/nextcloud/talk/utils/ContactUtils.kt

@@ -12,6 +12,8 @@ import android.provider.ContactsContract
 object ContactUtils {
 object ContactUtils {
 
 
     const val MAX_CONTACT_LIMIT = 50
     const val MAX_CONTACT_LIMIT = 50
+    const val CACHE_MEMORY_SIZE_PERCENTAGE = 0.1
+    const val CACHE_DISK_SIZE_PERCENTAGE = 0.02
 
 
     fun getDisplayNameFromDeviceContact(context: Context, id: String?): String? {
     fun getDisplayNameFromDeviceContact(context: Context, id: String?): String? {
         var displayName: String? = null
         var displayName: String? = null

+ 1 - 1
app/src/test/java/com/nextcloud/talk/contacts/repository/FakeRepositoryError.kt

@@ -11,7 +11,7 @@ import com.nextcloud.talk.contacts.ContactsRepository
 import com.nextcloud.talk.models.json.autocomplete.AutocompleteOverall
 import com.nextcloud.talk.models.json.autocomplete.AutocompleteOverall
 import com.nextcloud.talk.models.json.conversations.RoomOverall
 import com.nextcloud.talk.models.json.conversations.RoomOverall
 
 
-class FakeRepositoryError() : ContactsRepository {
+class FakeRepositoryError : ContactsRepository {
     override suspend fun getContacts(searchQuery: String?, shareTypes: List<String>): AutocompleteOverall {
     override suspend fun getContacts(searchQuery: String?, shareTypes: List<String>): AutocompleteOverall {
         throw Exception("unable to fetch contacts")
         throw Exception("unable to fetch contacts")
     }
     }

+ 1 - 1
app/src/test/java/com/nextcloud/talk/contacts/repository/FakeRepositorySuccess.kt

@@ -12,7 +12,7 @@ import com.nextcloud.talk.contacts.apiService.FakeItem
 import com.nextcloud.talk.models.json.autocomplete.AutocompleteOverall
 import com.nextcloud.talk.models.json.autocomplete.AutocompleteOverall
 import com.nextcloud.talk.models.json.conversations.RoomOverall
 import com.nextcloud.talk.models.json.conversations.RoomOverall
 
 
-class FakeRepositorySuccess() : ContactsRepository {
+class FakeRepositorySuccess : ContactsRepository {
     override suspend fun getContacts(searchQuery: String?, shareTypes: List<String>): AutocompleteOverall {
     override suspend fun getContacts(searchQuery: String?, shareTypes: List<String>): AutocompleteOverall {
         return FakeItem.contactsOverall
         return FakeItem.contactsOverall
     }
     }