Merge branch 'currency-refactor' into main

Author: shorthouse

app/src/main/java/dev/shorthouse/coinwatch/data/mapper/CoinSearchResultsMapper.kt

@@ -1,14 +1,12 @@
 package dev.shorthouse.coinwatch.data.mapper
 
-import dev.shorthouse.coinwatch.common.Mapper
 import dev.shorthouse.coinwatch.data.source.remote.model.CoinSearchResultsApiModel
 import dev.shorthouse.coinwatch.model.SearchCoin
 import javax.inject.Inject
 
-class CoinSearchResultsMapper @Inject constructor() :
-    Mapper<CoinSearchResultsApiModel, List<SearchCoin>> {
-    override fun mapApiModelToModel(from: CoinSearchResultsApiModel): List<SearchCoin> {
-        val validSearchResultsCoins = from.coinsSearchResultsData?.coinSearchResults
+class CoinSearchResultsMapper @Inject constructor() {
+    fun mapApiModelToModel(apiModel: CoinSearchResultsApiModel): List<SearchCoin> {
+        val validSearchResultsCoins = apiModel.coinsSearchResultsData?.coinSearchResults
             .orEmpty()
             .filterNotNull()
             .filter { it.id != null }

app/src/test/java/dev/shorthouse/coinwatch/data/source/remote/FakeCoinNetworkDataSource.kt

@@ -1,5 +1,7 @@
 package dev.shorthouse.coinwatch.data.source.remote
 
+import dev.shorthouse.coinwatch.data.datastore.CoinSort
+import dev.shorthouse.coinwatch.data.datastore.Currency
 import dev.shorthouse.coinwatch.data.source.remote.model.CoinChartApiModel
 import dev.shorthouse.coinwatch.data.source.remote.model.CoinDetailsApiModel
 import dev.shorthouse.coinwatch.data.source.remote.model.CoinSearchResultsApiModel
@@ -9,21 +11,37 @@ import retrofit2.Response
 class FakeCoinNetworkDataSource(
     private val coinApi: CoinApi
 ) : CoinNetworkDataSource {
-    override suspend fun getCoins(coinIds: List<String>): Response<CoinsApiModel> {
-        return coinApi.getCoins(coinIds = coinIds)
+    override suspend fun getCoins(
+        coinIds: List<String>,
+        coinSort: CoinSort,
+        currency: Currency
+    ): Response<CoinsApiModel> {
+        return coinApi.getCoins(
+            coinIds = coinIds,
+            orderBy = coinSort.toOrderByString(),
+            currencyUUID = currency.toCurrencyUUID()
+        )
     }
 
-    override suspend fun getCoinDetails(coinId: String): Response<CoinDetailsApiModel> {
-        return coinApi.getCoinDetails(coinId = coinId)
+    override suspend fun getCoinDetails(
+        coinId: String,
+        currency: Currency
+    ): Response<CoinDetailsApiModel> {
+        return coinApi.getCoinDetails(
+            coinId = coinId,
+            currencyUUID = currency.toCurrencyUUID()
+        )
     }
 
     override suspend fun getCoinChart(
         coinId: String,
-        chartPeriod: String
+        chartPeriod: String,
+        currency: Currency
     ): Response<CoinChartApiModel> {
         return coinApi.getCoinChart(
             coinId = coinId,
-            chartPeriod = chartPeriod
+            chartPeriod = chartPeriod,
+            currencyUUID = currency.toCurrencyUUID()
         )
     }
 
@@ -33,3 +51,20 @@ class FakeCoinNetworkDataSource(
         return coinApi.getCoinSearchResults(searchQuery = searchQuery)
     }
 }
+
+private fun CoinSort.toOrderByString(): String {
+    return when (this) {
+        CoinSort.MarketCap -> "marketCap"
+        CoinSort.Price -> "price"
+        CoinSort.PriceChange24h -> "change"
+        CoinSort.Volume24h -> "24hVolume"
+    }
+}
+
+private fun Currency.toCurrencyUUID(): String {
+    return when (this) {
+        Currency.USD -> "yhjMzLPhuIDl"
+        Currency.GBP -> "Hokyui45Z38f"
+        Currency.EUR -> "5k-_VTxqtCEI"
+    }
+}