[Woo POS] Update conditions for showing POS entry point

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/moremenu/MoreMenuViewModel.kt

@@ -29,6 +29,7 @@ import com.woocommerce.android.ui.payments.taptopay.isAvailable
 import com.woocommerce.android.ui.plans.domain.SitePlan
 import com.woocommerce.android.ui.plans.repository.SitePlanRepository
 import com.woocommerce.android.ui.woopos.IsWooPosEnabled
+import com.woocommerce.android.util.FeatureFlag
 import com.woocommerce.android.viewmodel.MultiLiveEvent
 import com.woocommerce.android.viewmodel.ResourceProvider
 import com.woocommerce.android.viewmodel.ScopedViewModel
@@ -140,7 +141,7 @@ class MoreMenuViewModel @Inject constructor(
             title = R.string.more_menu_button_woo_pos,
             description = R.string.more_menu_button_woo_pos_description,
             icon = R.drawable.ic_more_menu_payments,
-            isEnabled = isWooPosEnabled(),
+            isEnabled = isWooPosEnabled() && FeatureFlag.WOO_POS.isEnabled(),
             onClick = {
                 triggerEvent(MoreMenuEvent.NavigateToWooPosEvent)
             }

WooCommerce/src/main/kotlin/com/woocommerce/android/ui/woopos/IsWooPosEnabled.kt

@@ -1,8 +1,33 @@
 package com.woocommerce.android.ui.woopos
 
-import com.woocommerce.android.util.FeatureFlag
+import com.woocommerce.android.tools.SelectedSite
+import com.woocommerce.android.ui.payments.GetActivePaymentsPlugin
+import com.woocommerce.android.util.IsWindowClassExpandedAndBigger
+import org.wordpress.android.fluxc.model.payments.inperson.WCPaymentAccountResult
+import org.wordpress.android.fluxc.store.WCInPersonPaymentsStore
+import org.wordpress.android.fluxc.store.WCInPersonPaymentsStore.InPersonPaymentsPluginType.WOOCOMMERCE_PAYMENTS
 import javax.inject.Inject
 
-class IsWooPosEnabled @Inject constructor() {
-    operator fun invoke() = FeatureFlag.WOO_POS.isEnabled()
+class IsWooPosEnabled @Inject constructor(
+    private val selectedSite: SelectedSite,
+    private val ippStore: WCInPersonPaymentsStore,
+    private val getActivePaymentsPlugin: GetActivePaymentsPlugin,
+    private val isWindowSizeExpandedAndBigger: IsWindowClassExpandedAndBigger,
+) {
+    @Suppress("ReturnCount")
+    suspend operator fun invoke(): Boolean {
+        val ippPlugin = getActivePaymentsPlugin() ?: return false
+        val selectedSite = selectedSite.getOrNull() ?: return false
+        val paymentAccount = ippStore.loadAccount(ippPlugin, selectedSite).model ?: return false
+        val countryCode = paymentAccount.country
+
+        return countryCode.lowercase() == "us" &&
+            ippPlugin == WOOCOMMERCE_PAYMENTS &&
+            paymentAccount.storeCurrencies.default.lowercase() == "usd" &&
+            isPluginSetupCompleted(paymentAccount) &&
+            isWindowSizeExpandedAndBigger()
+    }
+
+    private fun isPluginSetupCompleted(paymentAccount: WCPaymentAccountResult): Boolean =
+        paymentAccount.status != WCPaymentAccountResult.WCPaymentAccountStatus.NO_ACCOUNT
 }

WooCommerce/src/main/kotlin/com/woocommerce/android/util/UiHelpers.kt

@@ -89,3 +89,7 @@ object UiHelpers {
 class IsWindowClassLargeThanCompact @Inject constructor(val context: Context) {
     operator fun invoke() = context.windowSizeClass != WindowSizeClass.Compact
 }
+
+class IsWindowClassExpandedAndBigger @Inject constructor(val context: Context) {
+    operator fun invoke() = context.windowSizeClass == WindowSizeClass.ExpandedAndBigger
+}

WooCommerce/src/test/kotlin/com/woocommerce/android/ui/moremenu/MoreMenuViewModelTests.kt

@@ -74,7 +74,7 @@ class MoreMenuViewModelTests : BaseUnitTest() {
         onBlocking { invoke() } doReturn true
     }
     private val isWooPosEnabled: IsWooPosEnabled = mock {
-        on { invoke() } doReturn true
+        onBlocking { invoke() } doReturn true
     }
 
     private val blazeCampaignsStore: BlazeCampaignsStore = mock()

WooCommerce/src/test/kotlin/com/woocommerce/android/ui/woopos/IsWooPosEnabledTest.kt

@@ -0,0 +1,111 @@
+package com.woocommerce.android.ui.woopos
+
+import com.woocommerce.android.tools.SelectedSite
+import com.woocommerce.android.ui.payments.GetActivePaymentsPlugin
+import com.woocommerce.android.util.IsWindowClassExpandedAndBigger
+import com.woocommerce.android.viewmodel.BaseUnitTest
+import kotlinx.coroutines.ExperimentalCoroutinesApi
+import org.junit.Before
+import org.mockito.kotlin.any
+import org.mockito.kotlin.mock
+import org.mockito.kotlin.whenever
+import org.wordpress.android.fluxc.model.SiteModel
+import org.wordpress.android.fluxc.model.payments.inperson.WCPaymentAccountResult
+import org.wordpress.android.fluxc.model.payments.inperson.WCPaymentAccountResult.WCPaymentAccountStatus.COMPLETE
+import org.wordpress.android.fluxc.model.payments.inperson.WCPaymentAccountResult.WCPaymentAccountStatus.StoreCurrencies
+import org.wordpress.android.fluxc.network.rest.wpcom.wc.WooResult
+import org.wordpress.android.fluxc.store.WCInPersonPaymentsStore
+import org.wordpress.android.fluxc.store.WCInPersonPaymentsStore.InPersonPaymentsPluginType.STRIPE
+import org.wordpress.android.fluxc.store.WCInPersonPaymentsStore.InPersonPaymentsPluginType.WOOCOMMERCE_PAYMENTS
+import kotlin.test.Test
+import kotlin.test.assertFalse
+import kotlin.test.assertTrue
+
+@OptIn(ExperimentalCoroutinesApi::class)
+class IsWooPosEnabledTest : BaseUnitTest() {
+    private val selectedSite: SelectedSite = mock()
+    private val ippStore: WCInPersonPaymentsStore = mock()
+    private val getActivePaymentsPlugin: GetActivePaymentsPlugin = mock()
+    private val isWindowSizeExpandedAndBigger: IsWindowClassExpandedAndBigger = mock()
+
+    private lateinit var sut: IsWooPosEnabled
+
+    @Before
+    fun setup() = testBlocking {
+        whenever(selectedSite.getOrNull()).thenReturn(SiteModel())
+        whenever(getActivePaymentsPlugin()).thenReturn(WOOCOMMERCE_PAYMENTS)
+        whenever(isWindowSizeExpandedAndBigger()).thenReturn(true)
+        whenever(ippStore.loadAccount(any(), any())).thenReturn(buildPaymentAccountResult())
+
+        sut = IsWooPosEnabled(
+            selectedSite = selectedSite,
+            ippStore = ippStore,
+            getActivePaymentsPlugin = getActivePaymentsPlugin,
+            isWindowSizeExpandedAndBigger = isWindowSizeExpandedAndBigger
+        )
+    }
+
+    @Test
+    fun `given store not in the US, then return false`() = testBlocking {
+        val result = buildPaymentAccountResult(countryCode = "CA")
+        whenever(ippStore.loadAccount(any(), any())).thenReturn(result)
+        assertFalse(sut())
+    }
+
+    @Test
+    fun `given not big enough screen, then return false`() = testBlocking {
+        whenever(isWindowSizeExpandedAndBigger()).thenReturn(false)
+        assertFalse(sut())
+    }
+
+    @Test
+    fun `given currency is not USD, then return false`() = testBlocking {
+        val result = buildPaymentAccountResult(defaultCurrency = "CAD")
+        whenever(ippStore.loadAccount(any(), any())).thenReturn(result)
+        assertFalse(sut())
+    }
+
+    @Test
+    fun `given ipp plugin is not enabled, then return false`() = testBlocking {
+        whenever(getActivePaymentsPlugin()).thenReturn(null)
+        assertFalse(sut())
+    }
+
+    @Test
+    fun `given ipp plugin is not woo payments, then return false`() = testBlocking {
+        whenever(getActivePaymentsPlugin()).thenReturn(STRIPE)
+        assertFalse(sut())
+    }
+
+    @Test
+    fun `given woo payments setup not completed, then return false`() = testBlocking {
+        val result = buildPaymentAccountResult(status = WCPaymentAccountResult.WCPaymentAccountStatus.NO_ACCOUNT)
+        whenever(ippStore.loadAccount(any(), any())).thenReturn(result)
+        assertFalse(sut())
+    }
+
+    @Test
+    fun `given big enough screen, woo payments enabled, USD currency and store in the US, then return true`() = testBlocking {
+        val result = buildPaymentAccountResult(defaultCurrency = "USD", countryCode = "US", status = COMPLETE)
+        whenever(ippStore.loadAccount(any(), any())).thenReturn(result)
+        assertTrue(sut())
+    }
+
+    private fun buildPaymentAccountResult(
+        status: WCPaymentAccountResult.WCPaymentAccountStatus = COMPLETE,
+        countryCode: String = "US",
+        defaultCurrency: String = "USD"
+    ) = WooResult(
+        WCPaymentAccountResult(
+            status,
+            hasPendingRequirements = false,
+            hasOverdueRequirements = false,
+            currentDeadline = null,
+            statementDescriptor = "",
+            storeCurrencies = StoreCurrencies(defaultCurrency, listOf(defaultCurrency)),
+            country = countryCode,
+            isLive = true,
+            testMode = false,
+        )
+    )
+}