Remove clear cache mechanism on refresh and rename saveImages function

Author: mbakgun

shared/src/androidInstrumentedTest/kotlin/ui/ScreenTestUtil.kt

@@ -89,5 +89,5 @@ class OfflineMjImagesLocalDataSource : MjImagesDataSource.Local {
 
     override suspend fun clearImages() = Unit
 
-    override suspend fun saveImages(page: Int, response: MjImagesResponse) = Unit
+    override suspend fun cacheResponse(page: Int, response: MjImagesResponse) = Unit
 }

shared/src/commonMain/kotlin/data/repository/MjImagesRepository.kt

@@ -21,7 +21,7 @@ class MjImagesRepository : KoinComponent {
         }
 
         val mjImagesResponse = remoteSource.getImages(page)
-        localSource.saveImages(page,mjImagesResponse)
+        localSource.cacheResponse(page,mjImagesResponse)
         emit(mjImagesResponse)
     }
 
@@ -38,8 +38,4 @@ class MjImagesRepository : KoinComponent {
      suspend fun setDarkMode(enabled: Boolean) {
         localSource.setDarkMode(enabled)
     }
-
-    suspend fun clearImages() {
-        localSource.clearImages()
-    }
 }

shared/src/commonMain/kotlin/data/source/MjImagesDataSource.kt

@@ -13,7 +13,7 @@ interface MjImagesDataSource {
         suspend fun isCacheValid(): Boolean
         suspend fun getImages(page: Int): MjImagesResponse?
         suspend fun clearImages()
-        suspend fun saveImages(page: Int, response: MjImagesResponse)
+        suspend fun cacheResponse(page: Int, response: MjImagesResponse)
     }
 
     interface Remote {

shared/src/commonMain/kotlin/data/source/local/MjImagesLocalDataSource.kt

@@ -42,7 +42,8 @@ class MjImagesLocalDataSource(
 
     override suspend fun getImages(page: Int): MjImagesResponse? =
         withContext(dispatcherProvider.io) {
-            val jsonString = settings.getStringOrNull(CACHE_PREFIX_KEY + page) ?: return@withContext null
+            val jsonString =
+                settings.getStringOrNull(CACHE_PREFIX_KEY + page) ?: return@withContext null
             Json.decodeFromString<MjImagesResponse?>(jsonString)
         }
 
@@ -56,7 +57,7 @@ class MjImagesLocalDataSource(
         }
     }
 
-    override suspend fun saveImages(page: Int, response: MjImagesResponse) {
+    override suspend fun cacheResponse(page: Int, response: MjImagesResponse) {
         withContext(dispatcherProvider.io) {
             if (response.mjImageResponses.isNullOrEmpty()) return@withContext
             if (page == 1) clearImages()

shared/src/commonMain/kotlin/domain/usecase/MjImagesUseCase.kt

@@ -21,8 +21,4 @@ class MjImagesUseCase : KoinComponent {
     suspend fun setDarkMode(enabled: Boolean) {
         repository.setDarkMode(enabled)
     }
-
-    suspend fun clearImages() {
-        repository.clearImages()
-    }
 }

shared/src/commonMain/kotlin/ui/MjImagesViewModel.kt

@@ -55,11 +55,8 @@ class MjImagesViewModel(
     }
 
     fun refreshImages() {
-        viewModelScope.launch {
-            useCase.clearImages()
-            _images.value = MjImages()
-            fetchImages()
-        }
+        _images.value = MjImages()
+        fetchImages()
     }
 
     private fun fetchImages(

shared/src/commonTest/kotlin/fakes/MjImagesLocalFakeDataSource.kt

@@ -5,8 +5,7 @@ import data.source.remote.model.MjImagesResponse
 
 class MjImagesLocalFakeDataSource : MjImagesDataSource.Local {
 
-    override suspend fun isEligibleToShowSnackMessage(): Boolean =
-        true
+    override suspend fun isEligibleToShowSnackMessage(): Boolean = true
 
     override suspend fun setSnackMessageShown() = Unit
 
@@ -20,7 +19,8 @@ class MjImagesLocalFakeDataSource : MjImagesDataSource.Local {
 
     override suspend fun clearImages() = Unit
 
-    override suspend fun saveImages(
-        page: Int, response: MjImagesResponse
+    override suspend fun cacheResponse(
+        page: Int,
+        response: MjImagesResponse
     ) = Unit
 }