Ipc 186 unit tests payment component

health-sdk/sdk/src/main/java/net/gini/android/health/sdk/paymentcomponent/PaymentComponent.kt

@@ -1,6 +1,7 @@
 package net.gini.android.health.sdk.paymentcomponent
 
 import android.content.Context
+import androidx.annotation.VisibleForTesting
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.asStateFlow
@@ -22,7 +23,8 @@ class PaymentComponent(private val context: Context, private val giniHealth: Gin
         MutableStateFlow<SelectedPaymentProviderAppState>(SelectedPaymentProviderAppState.NothingSelected)
     val selectedPaymentProviderAppFlow: StateFlow<SelectedPaymentProviderAppState> = _selectedPaymentProviderAppFlow.asStateFlow()
 
-    private val paymentComponentPreferences = PaymentComponentPreferences(context)
+    @VisibleForTesting
+    internal val paymentComponentPreferences = PaymentComponentPreferences(context)
 
     var listener: Listener? = null
 

health-sdk/sdk/src/main/java/net/gini/android/health/sdk/paymentprovider/PaymentProviderApp.kt

@@ -5,15 +5,12 @@ import android.content.Context
 import android.content.Intent
 import android.content.pm.PackageManager
 import android.content.pm.ResolveInfo
-import android.graphics.Bitmap
-import android.graphics.BitmapFactory
 import android.graphics.Color
 import android.graphics.drawable.BitmapDrawable
-import android.graphics.drawable.Drawable
 import android.net.Uri
-import android.util.TypedValue
 import androidx.annotation.ColorInt
 import net.gini.android.health.api.models.PaymentProvider
+import net.gini.android.health.sdk.util.extensions.generateBitmapDrawableIcon
 import org.slf4j.LoggerFactory
 
 internal const val Scheme = "ginipay" // It has to match the scheme in query tag in manifest
@@ -122,22 +119,7 @@ data class PaymentProviderApp(
             }
             return PaymentProviderApp(
                 name = paymentProvider.name,
-                icon = BitmapFactory.decodeByteArray(paymentProvider.icon, 0, paymentProvider.icon.size)
-                    ?.let { bitmap ->
-                        val iconSizePx = TypedValue.applyDimension(
-                            TypedValue.COMPLEX_UNIT_DIP,
-                            ICON_SIZE,
-                            context.resources.displayMetrics
-                        ).toInt()
-                        val scaledBitamp = Bitmap.createScaledBitmap(
-                            bitmap,
-                            iconSizePx,
-                            iconSizePx,
-                            true
-                        )
-                        bitmap.recycle()
-                        BitmapDrawable(context.resources, scaledBitamp)
-                    },
+                icon = context.generateBitmapDrawableIcon(paymentProvider.icon, paymentProvider.icon.size),
                 colors = PaymentProviderAppColors(
                     backgroundColor = Color.parseColor("#${paymentProvider.colors.backgroundColorRGBHex}"),
                     textColor = Color.parseColor("#${paymentProvider.colors.textColoRGBHex}")

health-sdk/sdk/src/main/java/net/gini/android/health/sdk/util/extensions/Context.kt

@@ -0,0 +1,30 @@
+package net.gini.android.health.sdk.util.extensions
+
+import android.content.Context
+import android.graphics.Bitmap
+import android.graphics.BitmapFactory
+import android.graphics.drawable.BitmapDrawable
+import android.util.TypedValue
+import androidx.annotation.VisibleForTesting
+import net.gini.android.health.sdk.paymentprovider.PaymentProviderApp
+
+// In a future refactoring we can split extensions into files according to what component they extend
+@VisibleForTesting
+internal fun Context.generateBitmapDrawableIcon(icon: ByteArray, iconSize: Int): BitmapDrawable? {
+    return BitmapFactory.decodeByteArray(icon, 0, iconSize)
+        ?.let { bitmap ->
+            val iconSizePx = TypedValue.applyDimension(
+                TypedValue.COMPLEX_UNIT_DIP,
+                PaymentProviderApp.ICON_SIZE,
+                this.resources.displayMetrics
+            ).toInt()
+            val scaledBitmap = Bitmap.createScaledBitmap(
+                bitmap,
+                iconSizePx,
+                iconSizePx,
+                true
+            )
+            bitmap.recycle()
+            BitmapDrawable(this.resources, scaledBitmap)
+        }
+}

health-sdk/sdk/src/test/java/net/gini/android/health/sdk/paymentComponent/PaymentComponentPreferencesTest.kt

@@ -0,0 +1,58 @@
+package net.gini.android.health.sdk.paymentComponent
+
+import android.content.Context
+import androidx.test.core.app.ApplicationProvider
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import com.google.common.truth.Truth.assertThat
+import kotlinx.coroutines.ExperimentalCoroutinesApi
+import kotlinx.coroutines.test.runTest
+import net.gini.android.health.sdk.paymentcomponent.PaymentComponentPreferences
+import net.gini.android.health.sdk.test.ViewModelTestCoroutineRule
+import org.junit.Before
+import org.junit.Rule
+import org.junit.Test
+import org.junit.runner.RunWith
+
+@ExperimentalCoroutinesApi
+@RunWith(AndroidJUnit4::class)
+class PaymentComponentPreferencesTest {
+
+    @get:Rule
+    val testCoroutineRule = ViewModelTestCoroutineRule()
+    private lateinit var context: Context
+
+    @Before
+    fun setup() {
+        context = ApplicationProvider.getApplicationContext()
+    }
+
+    @Test
+    fun `sets selected payment provider app`() = runTest {
+        // Given
+        val paymentComponentPreferences = PaymentComponentPreferences(context)
+
+        assertThat(paymentComponentPreferences.getSelectedPaymentProviderId()).isNull()
+
+        // When
+        paymentComponentPreferences.saveSelectedPaymentProviderId("123")
+
+        // Then
+        assertThat(paymentComponentPreferences.getSelectedPaymentProviderId()).isEqualTo("123")
+    }
+
+    @Test
+    fun `deletes selected payment provider app`() = runTest {
+        // Given
+        val paymentComponentPreferences = PaymentComponentPreferences(context)
+
+        assertThat(paymentComponentPreferences.getSelectedPaymentProviderId()).isNull()
+        paymentComponentPreferences.saveSelectedPaymentProviderId("123")
+        assertThat(paymentComponentPreferences.getSelectedPaymentProviderId()).isEqualTo("123")
+
+        // When
+        paymentComponentPreferences.deleteSelectedPaymentProviderId()
+
+        // Then
+        assertThat(paymentComponentPreferences.getSelectedPaymentProviderId()).isNull()
+    }
+}