[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/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/CharactersNavModule.kt

@@ -3,9 +3,6 @@ package com.rumosoft.characters.presentation.navigation
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.LaunchedEffect
 import androidx.compose.runtime.getValue
-import androidx.compose.runtime.mutableStateOf
-import androidx.compose.runtime.remember
-import androidx.compose.runtime.setValue
 import androidx.hilt.navigation.compose.hiltViewModel
 import androidx.lifecycle.compose.collectAsStateWithLifecycle
 import androidx.navigation.NavGraphBuilder
@@ -38,21 +35,8 @@ fun NavGraphBuilder.charactersGraph(
                 navController.navigate(CharacterDetails(selectedCharacter.id))
             }
         }
-        var search by remember { mutableStateOf("") }
-        val onToggleSearchClick = {
-            viewModel.onToggleSearchClick()
-        }
-        val onValueChanged = { value: String ->
-            search = value
-            viewModel.onQueryChanged(value)
-        }
         setTopBarContent {
-            CharactersTopBar(
-                search,
-                heroListScreenState.showingSearchBar,
-                onToggleSearchClick,
-                onValueChanged,
-            )
+            CharactersTopBar()
         }
         HeroListScreenContent(
             heroListState = heroListScreenState.heroListState,

feature-characters/src/main/java/com/rumosoft/characters/presentation/screen/HeroListScreen.kt

@@ -1,37 +1,23 @@
 package com.rumosoft.characters.presentation.screen
 
 import android.content.res.Configuration.UI_MODE_NIGHT_YES
-import androidx.compose.animation.AnimatedVisibility
-import androidx.compose.animation.fadeIn
-import androidx.compose.animation.fadeOut
-import androidx.compose.animation.slideOutVertically
 import androidx.compose.foundation.layout.Box
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Alignment
 import androidx.compose.ui.Modifier
-import androidx.compose.ui.graphics.vector.ImageVector
-import androidx.compose.ui.res.stringResource
-import androidx.compose.ui.res.vectorResource
 import androidx.compose.ui.tooling.preview.Preview
 import com.rumosoft.characters.R
-import com.rumosoft.components.R as Rcomponents
 import com.rumosoft.characters.infrastructure.sampleData.SampleData
 import com.rumosoft.characters.presentation.screen.state.BuildUI
 import com.rumosoft.characters.presentation.viewmodel.state.HeroListState
-import com.rumosoft.components.presentation.component.SearchBar
 import com.rumosoft.components.presentation.component.SectionTopBar
 import com.rumosoft.components.presentation.theme.MarvelComposeTheme
 
 @Composable
-fun CharactersTopBar(
-    searchText: String,
-    showSearchBar: Boolean,
-    onToggleSearchClick: () -> Unit,
-    onValueChanged: (String) -> Unit
-) {
-    SearchableTitle(searchText, showSearchBar, onToggleSearchClick, onValueChanged)
+fun CharactersTopBar() {
+    SectionTopBar(sectionName = R.string.characters)
 }
 
 @Composable
@@ -46,36 +32,6 @@ fun HeroListScreenContent(
     }
 }
 
-@Composable
-private fun SearchableTitle(
-    searchText: String,
-    showSearchBar: Boolean,
-    onToggleSearchClick: () -> Unit = {},
-    onValueChanged: (String) -> Unit = {},
-) {
-    Column {
-        SectionTopBar(
-            sectionName = R.string.characters,
-            icon = ImageVector.vectorResource(Rcomponents.drawable.ic_search),
-            onIconClick = onToggleSearchClick,
-        )
-        AnimatedVisibility(
-            showSearchBar,
-            enter = fadeIn(),
-            exit = slideOutVertically() + fadeOut(),
-        ) {
-            if (showSearchBar) {
-                SearchBar(
-                    search = searchText,
-                    hint = stringResource(id = com.rumosoft.components.R.string.search_hint),
-                    requestFocus = true,
-                    onValueChanged = onValueChanged,
-                    onLeadingClicked = onToggleSearchClick,
-                )
-            }
-        }
-    }
-}
 
 @Composable
 private fun ResultBox(