[Update] remove navigation from library-marvelapi

[Update] remove search for characters and comics

Author: jeprubio

app/src/androidTest/java/com/rumosoft/marvelcompose/data/FakeCharactersNetwork.kt

@@ -1,24 +1,23 @@
 package com.rumosoft.marvelcompose.data
 
-import com.rumosoft.characters.data.repository.CHARACTERS_SEARCH_LIMIT
+import com.rumosoft.characters.data.repository.CHARACTERS_LIMIT
 import com.rumosoft.marvelapi.data.network.CharactersNetwork
 import com.rumosoft.marvelapi.data.network.HeroesResult
 import com.rumosoft.marvelapi.data.network.apimodels.HeroDto
 import com.rumosoft.marvelapi.data.network.apimodels.PaginationInfo
 import javax.inject.Inject
 
 class FakeCharactersNetwork @Inject constructor(): CharactersNetwork {
-    override suspend fun searchHeroes(
+    override suspend fun getHeroes(
         offset: Int,
         limit: Int,
-        nameStartsWith: String
     ): Result<HeroesResult> {
         val heroesResult = HeroesResult(
             paginationInfo = PaginationInfo(
                 current = 0,
                 total = 0,
             ),
-            characters = (offset .. offset + CHARACTERS_SEARCH_LIMIT).map {
+            characters = (offset .. offset + CHARACTERS_LIMIT).map {
                 HeroDto(
                     id = it.toLong(),
                     name = "character $it",
@@ -33,7 +32,7 @@ class FakeCharactersNetwork @Inject constructor(): CharactersNetwork {
     }
 
     override suspend fun getHeroDetails(heroId: Long): Result<HeroDto?> {
-        val hero = searchHeroes(0, 0, "").getOrNull()?.characters?.find { it.id == heroId }
+        val hero = getHeroes(0, 0).getOrNull()?.characters?.find { it.id == heroId }
         return hero?.let {
             Result.success(it)
         } ?: Result.failure(Exception("Hero not found"))

app/src/androidTest/java/com/rumosoft/marvelcompose/data/FakeComicsNetwork.kt

@@ -1,24 +1,22 @@
 package com.rumosoft.marvelcompose.data
 
-import com.rumosoft.characters.data.repository.CHARACTERS_SEARCH_LIMIT
 import com.rumosoft.marvelapi.data.network.ComicsNetwork
 import com.rumosoft.marvelapi.data.network.ComicsResult
 import com.rumosoft.marvelapi.data.network.apimodels.ComicDto
 import com.rumosoft.marvelapi.data.network.apimodels.PaginationInfo
 import javax.inject.Inject
 
 class FakeComicsNetwork @Inject constructor(): ComicsNetwork {
-    override suspend fun searchComics(
+    override suspend fun getComics(
         offset: Int,
         limit: Int,
-        titleStartsWith: String
     ): Result<ComicsResult> {
         val comicsResult = ComicsResult(
             paginationInfo = PaginationInfo(
                 current = 0,
                 total = 0,
             ),
-            comics = (offset..offset + CHARACTERS_SEARCH_LIMIT).map {
+            comics = (offset..offset + limit).map {
                 ComicDto(
                     id = it,
                     title = "comic $it",

feature-characters/src/main/java/com/rumosoft/characters/data/repository/SearchRepositoryImpl.kt

@@ -2,29 +2,28 @@ package com.rumosoft.characters.data.repository
 
 import com.rumosoft.characters.data.mappers.toHero
 import com.rumosoft.characters.domain.model.Character
-import com.rumosoft.characters.domain.usecase.interfaces.SearchRepository
+import com.rumosoft.characters.domain.usecase.interfaces.CharactersRepository
 import com.rumosoft.marvelapi.data.network.CallInProgressException
 import com.rumosoft.marvelapi.data.network.CharactersNetwork
 import timber.log.Timber
 import javax.inject.Inject
 
-const val CHARACTERS_SEARCH_LIMIT = 20
+const val CHARACTERS_LIMIT = 20
 
-class SearchRepositoryImpl @Inject constructor(
+class CharactersRepositoryImpl @Inject constructor(
     private val network: CharactersNetwork,
-) : SearchRepository {
+) : CharactersRepository {
     private var isRequestInProgress = false
 
-    override suspend fun performSearch(
-        nameStartsWith: String,
+    override suspend fun getCharacters(
         page: Int,
     ): Result<List<Character>> {
         if (isRequestInProgress) {
             Timber.d("Request is in progress")
             return Result.failure(CallInProgressException("Request is in progress"))
         }
-        Timber.d("Performing Network Search")
-        val networkResult = performNetworkSearch(nameStartsWith, page)
+        Timber.d("Fetching characters")
+        val networkResult = performNetworkFetch(page)
         if (networkResult.isSuccess) {
             Timber.d("Returned results")
         }
@@ -44,13 +43,12 @@ class SearchRepositoryImpl @Inject constructor(
         return network.getComicThumbnail(comicId)
     }
 
-    private suspend fun performNetworkSearch(
-        nameStartsWith: String,
+    private suspend fun performNetworkFetch(
         page: Int,
     ): Result<List<Character>> {
         isRequestInProgress = true
-        val offset = (page - 1) * CHARACTERS_SEARCH_LIMIT
-        val networkResult = network.searchHeroes(offset, CHARACTERS_SEARCH_LIMIT, nameStartsWith)
+        val offset = (page - 1) * CHARACTERS_LIMIT
+        val networkResult = network.getHeroes(offset, CHARACTERS_LIMIT)
         isRequestInProgress = false
         return networkResult.map { result ->
             result.characters?.map { it.toHero() } ?: emptyList()

feature-characters/src/main/java/com/rumosoft/characters/domain/model/Character.kt

@@ -1,14 +1,9 @@
 package com.rumosoft.characters.domain.model
 
-import android.net.Uri
-import android.os.Bundle
 import android.os.Parcelable
 import androidx.annotation.Keep
-import androidx.navigation.NavType
-import com.rumosoft.marvelapi.infrastructure.extensions.parcelable
 import kotlinx.parcelize.Parcelize
 import kotlinx.serialization.Serializable
-import kotlinx.serialization.json.Json
 
 @Keep
 @Parcelize
@@ -30,16 +25,3 @@ data class Link(
     val url: String,
 ) : Parcelable
 
-val CharacterNavType = object : NavType<Character>(isNullableAllowed = false) {
-    override fun put(bundle: Bundle, key: String, value: Character) {
-        bundle.putParcelable(key, value)
-    }
-
-    override fun get(bundle: Bundle, key: String): Character? {
-        return bundle.parcelable(key) as Character?
-    }
-
-    override fun parseValue(value: String): Character {
-        return Json.decodeFromString(Uri.decode(value))
-    }
-}

feature-characters/src/main/java/com/rumosoft/characters/domain/usecase/GetCharacterDetailsUseCase.kt

@@ -1,11 +1,11 @@
 package com.rumosoft.characters.domain.usecase
 
 import com.rumosoft.characters.domain.model.Character
-import com.rumosoft.characters.domain.usecase.interfaces.SearchRepository
+import com.rumosoft.characters.domain.usecase.interfaces.CharactersRepository
 import javax.inject.Inject
 
 class GetCharacterDetailsUseCase @Inject constructor(
-    private val repository: SearchRepository,
+    private val repository: CharactersRepository,
 ) {
     suspend operator fun invoke(
         characterId: Long,

feature-characters/src/main/java/com/rumosoft/characters/domain/usecase/GetComicThumbnailUseCase.kt

@@ -1,10 +1,10 @@
 package com.rumosoft.characters.domain.usecase
 
-import com.rumosoft.characters.domain.usecase.interfaces.SearchRepository
+import com.rumosoft.characters.domain.usecase.interfaces.CharactersRepository
 import javax.inject.Inject
 
 class GetComicThumbnailUseCase @Inject constructor(
-    private val repository: SearchRepository,
+    private val repository: CharactersRepository,
 ) {
     suspend operator fun invoke(comicId: Int): Result<String> =
         repository.getThumbnail(comicId)

feature-characters/src/main/java/com/rumosoft/characters/domain/usecase/SearchUseCase.kt

@@ -1,12 +1,12 @@
 package com.rumosoft.characters.domain.usecase
 
 import com.rumosoft.characters.domain.model.Character
-import com.rumosoft.characters.domain.usecase.interfaces.SearchRepository
+import com.rumosoft.characters.domain.usecase.interfaces.CharactersRepository
 import javax.inject.Inject
 
-class SearchUseCase @Inject constructor(
-    private val repository: SearchRepository,
+class GetCharactersUseCase @Inject constructor(
+    private val repository: CharactersRepository,
 ) {
-    suspend operator fun invoke(nameStart: String, page: Int): Result<List<Character>> =
-        repository.performSearch(nameStart, page)
+    suspend operator fun invoke(page: Int): Result<List<Character>> =
+        repository.getCharacters(page)
 }

feature-characters/src/main/java/com/rumosoft/characters/domain/usecase/interfaces/SearchRepository.kt

@@ -2,8 +2,8 @@ package com.rumosoft.characters.domain.usecase.interfaces
 
 import com.rumosoft.characters.domain.model.Character
 
-interface SearchRepository {
-    suspend fun performSearch(nameStartsWith: String, page: Int): Result<List<Character>>
+interface CharactersRepository {
+    suspend fun getCharacters(page: Int): Result<List<Character>>
     suspend fun getCharacterDetails(heroId: Long): Result<Character?>
     suspend fun getThumbnail(comicId: Int): Result<String>
 }

feature-characters/src/main/java/com/rumosoft/characters/infrastructure/di/CharactersRepositoryModule.kt

@@ -1,7 +1,7 @@
 package com.rumosoft.characters.infrastructure.di
 
-import com.rumosoft.characters.data.repository.SearchRepositoryImpl
-import com.rumosoft.characters.domain.usecase.interfaces.SearchRepository
+import com.rumosoft.characters.data.repository.CharactersRepositoryImpl
+import com.rumosoft.characters.domain.usecase.interfaces.CharactersRepository
 import dagger.Binds
 import dagger.Module
 import dagger.hilt.InstallIn
@@ -12,6 +12,6 @@ import dagger.hilt.components.SingletonComponent
 abstract class CharactersRepositoryModule {
     @Binds
     abstract fun bindCharacterRepository(
-        searchRepository: SearchRepositoryImpl,
-    ): SearchRepository
+        charactersRepository: CharactersRepositoryImpl,
+    ): CharactersRepository
 }

feature-characters/src/main/java/com/rumosoft/characters/presentation/navigation/CharacterNavType.kt

@@ -0,0 +1,22 @@
+package com.rumosoft.characters.presentation.navigation
+
+import android.net.Uri
+import android.os.Bundle
+import androidx.navigation.NavType
+import com.rumosoft.characters.domain.model.Character
+import com.rumosoft.marvelapi.infrastructure.extensions.parcelable
+import kotlinx.serialization.json.Json
+
+val CharacterNavType = object : NavType<Character>(isNullableAllowed = false) {
+    override fun put(bundle: Bundle, key: String, value: Character) {
+        bundle.putParcelable(key, value)
+    }
+
+    override fun get(bundle: Bundle, key: String): Character? {
+        return bundle.parcelable(key) as Character?
+    }
+
+    override fun parseValue(value: String): Character {
+        return Json.decodeFromString(Uri.decode(value))
+    }
+}