70 añadir unit tests detailviewmodel

[Mj-br]

🎩 How was this resolved?

Cambios Realizados:

-Se soluciona el playCry (Me di cuenta con los test que se estaba llamando ANTES de crear la pantalla, por lo tanto al no ser success daba una excepción).

• Se cambia el nombre de la variable _pokemonDetailInfo a _uiState que contienen los estados de la ui para mayor legibilidad.
• He añadido la anotación annotation class PokemonId y la he inyectado para reemplazar el saveStateHandle simulando lo que hace Antonio al testear su DetailViewModel y poder simplificar los test.
• He creado el UseCase de PlayCry y su test.
• Se cambia el tipo de la variable uiState a StateFlow (antes estaba a MutableStateFlow) para que no sea mutable y solo pueda cambiarse dentro del viewModel.
• Se elimina la variable once ya que, ahora, no es necesaria, al hacer test y hacer varias pruebas me he dado cuenta de ello.
• A la función getPokemonDetails se le agregan las lambdas .onStart (para ejecutar el Loading inicial) y el .onEach para cambiar el estado según el resultado.
• He estado resolviendo algunas cosas de Detekt aprovechando que yo estaba testeando la feature de Detail.
• He puesto los parámetros a 12 en Detekt (constructorThreshold) para las funciones de composables que tienen muchos pero quizá se pueda hacer de otra manera.

Detalles Adicionales:

• He puesto la llamada del getPokemonDetails al crearse la pantalla y no el viewmodel. Lo hice al darme cuenta que el loading no se estaba llamando nunca al hacer los test (aunque luego al agregar el onStart y hacer pruebas si que se muestra el loading en el init pero no logre que se viera en el test).
• Tengo algunas dudas con los test, luego si tal lo vemos en una reunión y os enseño, creo que está bien así pero quería intentar hacer algunas cosas más pero no me salían bien, entonces lo vemos cuando se pueda.

Instrucciones para Revisión:

• Darle un vistazo a los test y a la app ya que he solucionado el tema del playCry que no estaba funcionando antes.