Hide IPP learn more section for CIAB sites

Author: hichamboushaba

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/payments/methodselection/SelectPaymentMethodFragment.kt

@@ -126,6 +126,7 @@ class SelectPaymentMethodFragment : BaseFragment(R.layout.fragment_select_paymen
         with(binding.learnMoreIppPaymentMethodsTv) {
             learnMore.setOnClickListener { state.learnMoreIpp.onClick.invoke() }
             UiHelpers.setTextOrHide(binding.learnMoreIppPaymentMethodsTv.learnMore, state.learnMoreIpp.label)
+            learnMore.isVisible = state.learnMoreIpp.isVisible
         }
     }
 

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/payments/methodselection/SelectPaymentMethodViewModel.kt

@@ -11,6 +11,8 @@ import com.woocommerce.android.analytics.AnalyticsTracker.Companion.VALUE_SIMPLE
 import com.woocommerce.android.analytics.AnalyticsTracker.Companion.VALUE_SIMPLE_PAYMENTS_COLLECT_CASH
 import com.woocommerce.android.analytics.AnalyticsTracker.Companion.VALUE_SIMPLE_PAYMENTS_COLLECT_LINK
 import com.woocommerce.android.cardreader.internal.payments.PaymentUtils
+import com.woocommerce.android.ciab.CIABAffectedFeature
+import com.woocommerce.android.ciab.CIABSiteGateKeeper
 import com.woocommerce.android.extensions.CASH_ON_DELIVERY_PAYMENT_TYPE
 import com.woocommerce.android.extensions.isNotNullOrEmpty
 import com.woocommerce.android.model.Order
@@ -78,6 +80,7 @@ class SelectPaymentMethodViewModel @Inject constructor(
     private val cardReaderTrackingInfoKeeper: CardReaderTrackingInfoKeeper,
     private val paymentsUtils: PaymentUtils,
     private val logOrderCurrencyMismatchWithSiteSettings: SelectPaymentMethodCurrencyMissMatchLog,
+    private val ciabSiteGateKeeper: CIABSiteGateKeeper
 ) : ScopedViewModel(savedState) {
     private val navArgs: SelectPaymentMethodFragmentArgs by savedState.navArgs()
 
@@ -173,7 +176,8 @@ class SelectPaymentMethodViewModel @Inject constructor(
                     R.string.card_reader_connect_learn_more,
                     containsHtml = true
                 ),
-                onClick = ::onLearnMoreIppClicked
+                onClick = ::onLearnMoreIppClicked,
+                isVisible = ciabSiteGateKeeper.isFeatureSupported(CIABAffectedFeature.WooPayments)
             )
         )
     }

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/payments/methodselection/SelectPaymentMethodViewState.kt

@@ -30,6 +30,7 @@ sealed class SelectPaymentMethodViewState {
 
         data class LearnMoreIpp(
             val label: UiString,
+            val isVisible: Boolean,
             val onClick: () -> Unit,
         )
     }

WooCommerce/src/test/kotlin/com/woocommerce/android/ui/payments/SelectPaymentMethodViewModelTest.kt

@@ -3,6 +3,8 @@ package com.woocommerce.android.ui.payments
 import com.woocommerce.android.AppUrls
 import com.woocommerce.android.R
 import com.woocommerce.android.cardreader.internal.payments.PaymentUtils
+import com.woocommerce.android.ciab.CIABAffectedFeature
+import com.woocommerce.android.ciab.CIABSiteGateKeeper
 import com.woocommerce.android.extensions.CASH_ON_DELIVERY_PAYMENT_TYPE
 import com.woocommerce.android.model.Order
 import com.woocommerce.android.model.OrderMapper
@@ -45,6 +47,7 @@ import kotlinx.coroutines.test.runTest
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.Test
 import org.mockito.kotlin.any
+import org.mockito.kotlin.doReturn
 import org.mockito.kotlin.eq
 import org.mockito.kotlin.mock
 import org.mockito.kotlin.never
@@ -113,6 +116,9 @@ class SelectPaymentMethodViewModelTest : BaseUnitTest() {
     private val paymentsUtils: PaymentUtils = mock()
     private val cardReaderTrackingInfoKeeper: CardReaderTrackingInfoKeeper = mock()
     private val logOrderCurrencyMismatchWithSiteSettings = mock<SelectPaymentMethodCurrencyMissMatchLog>()
+    private val ciabSiteGateKeeper: CIABSiteGateKeeper = mock {
+        on { isFeatureSupported(CIABAffectedFeature.WooPayments) } doReturn true
+    }
 
     @Test
     fun `given hub flow, when view model init, then navigate to hub flow emitted`() = testBlocking {
@@ -1190,24 +1196,41 @@ class SelectPaymentMethodViewModelTest : BaseUnitTest() {
         )
     }
 
+    @Test
+    fun `given WooPayments unsupported by CIAB, when view model initialized, then hide IPP learn more link`() =
+        testBlocking {
+            // GIVEN
+            whenever(ciabSiteGateKeeper.isFeatureSupported(CIABAffectedFeature.WooPayments)).thenReturn(false)
+
+            // WHEN
+            val viewModel = initViewModel(Payment(1L, ORDER))
+
+            // THEN
+            val state = (viewModel.viewStateData.value as Success).learnMoreIpp
+            assertThat(state.isVisible).isFalse()
+        }
+
     private fun initViewModel(cardReaderFlowParam: CardReaderFlowParam): SelectPaymentMethodViewModel {
         return SelectPaymentMethodViewModel(
-            SelectPaymentMethodFragmentArgs(cardReaderFlowParam = cardReaderFlowParam).toSavedStateHandle(),
-            selectedSite,
-            orderStore,
-            gatewayStore,
-            coroutinesTestRule.testDispatchers,
-            networkStatus,
-            currencyFormatter,
-            wooCommerceStore,
-            orderMapper,
-            cardPaymentCollectibilityChecker,
-            learnMoreUrlProvider,
-            paymentsFlowTracker,
-            tapToPayAvailabilityStatus,
-            cardReaderTrackingInfoKeeper,
-            paymentsUtils,
-            logOrderCurrencyMismatchWithSiteSettings,
+            savedState = SelectPaymentMethodFragmentArgs(
+                cardReaderFlowParam = cardReaderFlowParam
+            ).toSavedStateHandle(),
+            selectedSite = selectedSite,
+            orderStore = orderStore,
+            gatewayStore = gatewayStore,
+            dispatchers = coroutinesTestRule.testDispatchers,
+            networkStatus = networkStatus,
+            currencyFormatter = currencyFormatter,
+            wooCommerceStore = wooCommerceStore,
+            orderMapper = orderMapper,
+            cardPaymentCollectibilityChecker = cardPaymentCollectibilityChecker,
+            learnMoreUrlProvider = learnMoreUrlProvider,
+            paymentsFlowTracker = paymentsFlowTracker,
+            tapToPayAvailabilityStatus = tapToPayAvailabilityStatus,
+            cardReaderTrackingInfoKeeper = cardReaderTrackingInfoKeeper,
+            paymentsUtils = paymentsUtils,
+            logOrderCurrencyMismatchWithSiteSettings = logOrderCurrencyMismatchWithSiteSettings,
+            ciabSiteGateKeeper = ciabSiteGateKeeper
         )
     }