[ALFMOB-81] Add to Bag

data/src/main/java/au/com/alfie/ecomm/data/bag/BagRepositoryImpl.kt

@@ -0,0 +1,38 @@
+package au.com.alfie.ecomm.data.bag
+
+import au.com.alfie.ecomm.data.toRepositoryResult
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import au.com.alfie.ecomm.repository.result.RepositoryResult
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.MutableStateFlow
+import kotlinx.coroutines.flow.map
+import javax.inject.Inject
+
+class BagRepositoryImpl @Inject constructor() : BagRepository {
+
+    // TODO consider removing this property when the products in the bag are saved on database or api
+    private val _bag = MutableStateFlow<List<BagProduct>>(listOf())
+
+    // TODO change this implementation to a proper implementation using data base or api to save the product
+    override fun addToBag(bagProduct: BagProduct): RepositoryResult<Boolean> {
+        _bag.value.firstOrNull {
+            bagProduct.productId == it.productId && bagProduct.variantSku == it.variantSku
+        } ?: run {
+            _bag.value = _bag.value.toMutableList().apply { add(bagProduct) }
+        }
+        return RepositoryResult.Success(true)
+    }
+
+    // TODO change this implementation to a proper implementation using data base or api to get the products in the bag
+    override fun getBag(): Flow<RepositoryResult<List<BagProduct>>> {
+        return _bag.map { bag ->
+            Result.success(bag).toRepositoryResult()
+        }
+    }
+
+    override fun removeFromBag(bagProduct: BagProduct): RepositoryResult<Boolean> {
+        _bag.value = _bag.value.toMutableList().apply { remove(bagProduct) }
+        return RepositoryResult.Success(true)
+    }
+}

data/src/main/java/au/com/alfie/ecomm/data/bag/di/BagModule.kt

@@ -0,0 +1,18 @@
+package au.com.alfie.ecomm.data.bag.di
+
+import au.com.alfie.ecomm.data.bag.BagRepositoryImpl
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import dagger.Binds
+import dagger.Module
+import dagger.hilt.InstallIn
+import dagger.hilt.components.SingletonComponent
+import javax.inject.Singleton
+
+@Module
+@InstallIn(SingletonComponent::class)
+internal abstract class BagModule {
+
+    @Binds
+    @Singleton
+    abstract fun bindBagRepository(bagRepositoryImpl: BagRepositoryImpl): BagRepository
+}

designsystem/src/main/java/au/com/alfie/ecomm/designsystem/component/productcard/ProductCardType.kt

@@ -28,6 +28,7 @@ sealed interface ProductCardType {
         override val brand: String,
         override val name: String,
         override val price: PriceType,
+        val onRemoveClick: ClickEvent? = null,
         val color: String,
         val size: String,
         override val cardTestTag: String = PRODUCT_CARD,

designsystem/src/main/java/au/com/alfie/ecomm/designsystem/component/productcard/size/ProductCardXSmall.kt

@@ -7,10 +7,13 @@ import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.fillMaxWidth
 import androidx.compose.foundation.layout.size
 import androidx.compose.foundation.layout.width
+import androidx.compose.material3.Icon
+import androidx.compose.material3.IconButton
 import androidx.compose.material3.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.platform.testTag
+import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.text.style.TextOverflow
 import androidx.compose.ui.tooling.preview.Preview
@@ -137,6 +140,18 @@ internal fun ProductCardXSmall(
                     .testTag(productCard.priceTestTag)
             )
         }
+        if (isLoading.not() && productCard.onRemoveClick != null) {
+            IconButton(
+                modifier = Modifier.size(Theme.iconSize.large),
+                onClick = productCard.onRemoveClick
+            ) {
+                Icon(
+                    painter = painterResource(id = R.drawable.ic_action_close_dark),
+                    contentDescription = null,
+                    modifier = Modifier.size(Theme.iconSize.small)
+                )
+            }
+        }
     }
 }
 

domain/repository/src/main/java/au/com/alfie/ecomm/repository/bag/BagProduct.kt

@@ -0,0 +1,6 @@
+package au.com.alfie.ecomm.repository.bag
+
+data class BagProduct(
+    val productId: String,
+    val variantSku: String
+)

domain/repository/src/main/java/au/com/alfie/ecomm/repository/bag/BagRepository.kt

@@ -0,0 +1,13 @@
+package au.com.alfie.ecomm.repository.bag
+
+import au.com.alfie.ecomm.repository.result.RepositoryResult
+import kotlinx.coroutines.flow.Flow
+
+interface BagRepository {
+
+    fun addToBag(bagProduct: BagProduct): RepositoryResult<Boolean>
+
+    fun getBag(): Flow<RepositoryResult<List<BagProduct>>>
+
+    fun removeFromBag(bagProduct: BagProduct): RepositoryResult<Boolean>
+}

domain/repository/src/main/java/au/com/alfie/ecomm/repository/product/model/Variant.kt

@@ -7,7 +7,7 @@ import au.com.alfie.ecomm.repository.shared.model.Size
 data class Variant(
     val attributes: List<Attribute>,
     val color: Color?,
-    val media: Media,
+    val media: Media.Image,
     val price: Price,
     val size: Size?,
     val sku: String,

domain/src/main/java/au/com/alfie/ecomm/domain/usecase/bag/AddToBagUseCase.kt

@@ -0,0 +1,19 @@
+package au.com.alfie.ecomm.domain.usecase.bag
+
+import au.com.alfie.ecomm.domain.UseCaseInteractor
+import au.com.alfie.ecomm.domain.UseCaseResult
+import au.com.alfie.ecomm.domain.doOnResult
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import javax.inject.Inject
+
+class AddToBagUseCase @Inject constructor(
+    private val bagRepository: BagRepository
+) : UseCaseInteractor {
+
+    suspend operator fun invoke(productId: String, variantSku: String) =
+        run(bagRepository.addToBag(BagProduct(productId = productId, variantSku = variantSku))).doOnResult(
+            onSuccess = { UseCaseResult.Success(it) },
+            onError = { UseCaseResult.Error(it) }
+        )
+}

domain/src/main/java/au/com/alfie/ecomm/domain/usecase/bag/GetBagUseCase.kt

@@ -0,0 +1,19 @@
+package au.com.alfie.ecomm.domain.usecase.bag
+
+import au.com.alfie.ecomm.domain.UseCaseInteractor
+import au.com.alfie.ecomm.domain.UseCaseResult
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.map
+import javax.inject.Inject
+
+class GetBagUseCase @Inject constructor(
+    private val bagRepository: BagRepository
+) : UseCaseInteractor {
+
+    suspend operator fun invoke(): Flow<UseCaseResult<List<BagProduct>>> =
+        bagRepository.getBag().map { repositoryResult ->
+            run(repositoryResult)
+        }
+}

domain/src/main/java/au/com/alfie/ecomm/domain/usecase/bag/RemoveFromBagUseCase.kt

@@ -0,0 +1,19 @@
+package au.com.alfie.ecomm.domain.usecase.bag
+
+import au.com.alfie.ecomm.domain.UseCaseInteractor
+import au.com.alfie.ecomm.domain.UseCaseResult
+import au.com.alfie.ecomm.domain.doOnResult
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import javax.inject.Inject
+
+class RemoveFromBagUseCase @Inject constructor(
+    private val bagRepository: BagRepository
+) : UseCaseInteractor {
+
+    suspend operator fun invoke(bagProduct: BagProduct) =
+        run(bagRepository.removeFromBag(bagProduct)).doOnResult(
+            onSuccess = { UseCaseResult.Success(it) },
+            onError = { UseCaseResult.Error(it) }
+        )
+}

domain/src/test/java/au/com/alfie/ecomm/domain/bag/AddToBagUseCaseTest.kt

@@ -0,0 +1,55 @@
+package au.com.alfie.ecomm.domain.bag
+
+import au.com.alfie.ecomm.domain.usecase.bag.AddToBagUseCase
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import au.com.alfie.ecomm.repository.product.ProductRepository
+import au.com.alfie.ecomm.repository.result.ErrorResult
+import au.com.alfie.ecomm.repository.result.RepositoryResult
+import io.mockk.coEvery
+import io.mockk.coVerify
+import io.mockk.every
+import io.mockk.impl.annotations.InjectMockKs
+import io.mockk.impl.annotations.RelaxedMockK
+import io.mockk.junit5.MockKExtension
+import io.mockk.mockk
+import kotlinx.coroutines.test.runTest
+import org.junit.jupiter.api.Test
+import org.junit.jupiter.api.extension.ExtendWith
+
+@ExtendWith(MockKExtension::class)
+class AddToBagUseCaseTest {
+    @RelaxedMockK
+    private lateinit var bagRepository: BagRepository
+
+    @RelaxedMockK
+    private lateinit var productRepository: ProductRepository
+
+    @InjectMockKs
+    lateinit var subject: AddToBagUseCase
+
+    @Test
+    fun `add to bag, with success result`() = runTest {
+        val mockProduct = mockk<BagProduct> {
+            every { productId } returns "10"
+            every { variantSku } returns "34535"
+        }
+
+        coEvery { bagRepository.addToBag(mockProduct) } returns RepositoryResult.Success(true)
+        subject(productId = "10", variantSku = "34535")
+        coVerify { bagRepository.addToBag(mockProduct) }
+    }
+
+    @Test
+    fun `add to bag, with error result`() = runTest {
+        val mockProduct = mockk<BagProduct> {
+            every { productId } returns "10"
+            every { variantSku } returns "34535"
+        }
+        val errorResult = mockk<ErrorResult>()
+
+        coEvery { bagRepository.addToBag(mockProduct) } returns RepositoryResult.Error(errorResult)
+        subject("10", "34535")
+        coVerify { bagRepository.addToBag(mockProduct) }
+    }
+}

domain/src/test/java/au/com/alfie/ecomm/domain/bag/GetBagUseCaseTest.kt

@@ -0,0 +1,50 @@
+package au.com.alfie.ecomm.domain.bag
+
+import au.com.alfie.ecomm.domain.UseCaseResult
+import au.com.alfie.ecomm.domain.usecase.bag.GetBagUseCase
+import au.com.alfie.ecomm.repository.bag.BagProduct
+import au.com.alfie.ecomm.repository.bag.BagRepository
+import au.com.alfie.ecomm.repository.result.ErrorResult
+import au.com.alfie.ecomm.repository.result.RepositoryResult
+import io.mockk.coEvery
+import io.mockk.impl.annotations.InjectMockKs
+import io.mockk.impl.annotations.RelaxedMockK
+import io.mockk.junit5.MockKExtension
+import io.mockk.mockk
+import kotlinx.coroutines.flow.first
+import kotlinx.coroutines.flow.flowOf
+import kotlinx.coroutines.test.runTest
+import org.junit.jupiter.api.Test
+import org.junit.jupiter.api.extension.ExtendWith
+import kotlin.test.assertEquals
+
+@ExtendWith(MockKExtension::class)
+class GetBagUseCaseTest {
+    @RelaxedMockK
+    private lateinit var bagRepository: BagRepository
+
+    @InjectMockKs
+    lateinit var subject: GetBagUseCase
+
+    @Test
+    fun `get list of products in the bag`() = runTest {
+        val mockBag = mockk<List<BagProduct>>()
+        coEvery { bagRepository.getBag() } returns flowOf(RepositoryResult.Success(mockBag))
+
+        val expected = UseCaseResult.Success(mockBag)
+        val result = subject().first()
+
+        assertEquals(expected, result)
+    }
+
+    @Test
+    fun `get list of products in the bag, and returns an error`() = runTest {
+        val errorResult = mockk<ErrorResult>()
+        coEvery { bagRepository.getBag() } returns flowOf(RepositoryResult.Error(errorResult))
+
+        val expected = UseCaseResult.Error(errorResult)
+        val result = subject().first()
+
+        assertEquals(expected, result)
+    }
+}