Merge pull request #17239 from wordpress-mobile/feat/add-home-deep-link-intent

zwarm · web-flow · commit 88241018b46e · 2022-10-06T13:10:01.000-04:00
feat: Add Home deep link intent
diff --git a/WordPress/src/jetpack/AndroidManifest.xml b/WordPress/src/jetpack/AndroidManifest.xml
@@ -37,6 +37,9 @@
                 <data
                     android:host="notifications"
                     android:scheme="jetpack" />
+                <data
+                    android:host="home"
+                    android:scheme="jetpack" />
             </intent-filter>
         </activity>
 
diff --git a/WordPress/src/main/AndroidManifest.xml b/WordPress/src/main/AndroidManifest.xml
@@ -400,6 +400,9 @@
                 <data
                     android:host="notifications"
                     android:scheme="wordpress" />
+                <data
+                    android:host="home"
+                    android:scheme="wordpress" />
             </intent-filter>
 
             <intent-filter android:autoVerify="true">
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/ActivityLauncher.java b/WordPress/src/main/java/org/wordpress/android/ui/ActivityLauncher.java
@@ -1751,4 +1751,9 @@ public static void startQRCodeAuthFlowInNewStack(@NonNull Context context, @NonN
 
         taskStackBuilder.startActivities();
     }
+
+    public static void showLoginPrologue(@NonNull Context context) {
+        Intent intent = new Intent(context, LoginActivity.class);
+        context.startActivity(intent);
+    }
 }
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/deeplinks/DeepLinkNavigator.kt b/WordPress/src/main/java/org/wordpress/android/ui/deeplinks/DeepLinkNavigator.kt
@@ -8,8 +8,10 @@ import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.Login
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenEditor
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenEditorForPost
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenEditorForSite
+import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenMySite
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenInBrowser
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenInReader
+import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenLoginPrologue
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenNotifications
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenPages
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenPagesForSite
@@ -74,6 +76,8 @@ class DeepLinkNavigator
             is OpenPagesForSite -> ActivityLauncher.viewPagesInNewStack(activity, navigateAction.site)
             OpenPages -> ActivityLauncher.viewPagesInNewStack(activity)
             is OpenQRCodeAuthFlow -> ActivityLauncher.startQRCodeAuthFlowInNewStack(activity, navigateAction.uri)
+            OpenMySite -> ActivityLauncher.viewMySiteInNewStack(activity)
+            OpenLoginPrologue -> ActivityLauncher.showLoginPrologue(activity)
         }
         if (navigateAction != LoginForResult) {
             activity.finish()
@@ -101,5 +105,7 @@ class DeepLinkNavigator
         data class OpenPagesForSite(val site: SiteModel) : NavigateAction()
         object OpenPages : NavigateAction()
         data class OpenQRCodeAuthFlow(val uri: String) : NavigateAction()
+        object OpenMySite : NavigateAction()
+        object OpenLoginPrologue : NavigateAction()
     }
 }
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/deeplinks/DeepLinkingIntentReceiverViewModel.kt b/WordPress/src/main/java/org/wordpress/android/ui/deeplinks/DeepLinkingIntentReceiverViewModel.kt
@@ -9,6 +9,7 @@ import org.wordpress.android.modules.UI_THREAD
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.LoginForResult
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenInBrowser
+import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenLoginPrologue
 import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.ShowSignInFlow
 import org.wordpress.android.ui.deeplinks.handlers.DeepLinkHandlers
 import org.wordpress.android.ui.deeplinks.handlers.ServerTrackingHandler
@@ -69,14 +70,21 @@ class DeepLinkingIntentReceiverViewModel
             if (action != null) {
                 deepLinkTrackingUtils.track(action, it, uriWrapper)
             }
-            if (accountStore.hasAccessToken() || it is OpenInBrowser || it is ShowSignInFlow) {
+            if (loginIsUnnecessary(it)) {
                 _navigateAction.value = Event(it)
             } else {
                 _navigateAction.value = Event(LoginForResult)
             }
         } != null
     }
 
+    private fun loginIsUnnecessary(action: NavigateAction): Boolean {
+        return accountStore.hasAccessToken() ||
+                action is OpenInBrowser ||
+                action is ShowSignInFlow ||
+                action is OpenLoginPrologue
+    }
+
     private fun buildNavigateAction(uri: UriWrapper, rootUri: UriWrapper = uri): NavigateAction? {
         return when {
             deepLinkUriUtils.isTrackingUrl(uri) -> getRedirectUriAndBuildNavigateAction(uri, rootUri)
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/deeplinks/handlers/DeepLinkHandlers.kt b/WordPress/src/main/java/org/wordpress/android/ui/deeplinks/handlers/DeepLinkHandlers.kt
@@ -15,7 +15,8 @@ class DeepLinkHandlers
     readerLinkHandler: ReaderLinkHandler,
     pagesLinkHandler: PagesLinkHandler,
     notificationsLinkHandler: NotificationsLinkHandler,
-    qrCodeAuthLinkHandler: QRCodeAuthLinkHandler
+    qrCodeAuthLinkHandler: QRCodeAuthLinkHandler,
+    homeLinkHandler: HomeLinkHandler
 ) {
     private val handlers = listOf(
             editorLinkHandler,
@@ -24,7 +25,8 @@ class DeepLinkHandlers
             readerLinkHandler,
             pagesLinkHandler,
             notificationsLinkHandler,
-            qrCodeAuthLinkHandler
+            qrCodeAuthLinkHandler,
+            homeLinkHandler
     )
 
     private val _toast by lazy {
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/deeplinks/handlers/HomeLinkHandler.kt b/WordPress/src/main/java/org/wordpress/android/ui/deeplinks/handlers/HomeLinkHandler.kt
@@ -0,0 +1,40 @@
+package org.wordpress.android.ui.deeplinks.handlers
+
+import org.wordpress.android.fluxc.store.AccountStore
+import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction
+import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenMySite
+import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenLoginPrologue
+import org.wordpress.android.ui.deeplinks.DeepLinkingIntentReceiverViewModel
+import org.wordpress.android.util.UriWrapper
+import javax.inject.Inject
+
+class HomeLinkHandler
+@Inject constructor(private val accountStore: AccountStore) : DeepLinkHandler {
+    /**
+     * Returns true if the URI looks like `wordpress://home`
+     */
+    override fun shouldHandleUrl(uri: UriWrapper): Boolean {
+        return uri.host == HOME_PATH
+    }
+
+    override fun buildNavigateAction(uri: UriWrapper): NavigateAction {
+        return if (accountStore.hasAccessToken()) {
+            OpenMySite
+        } else {
+            OpenLoginPrologue
+        }
+    }
+
+    override fun stripUrl(uri: UriWrapper): String {
+        return buildString {
+            if (uri.host == HOME_PATH) {
+                append(DeepLinkingIntentReceiverViewModel.APPLINK_SCHEME)
+            }
+            append(HOME_PATH)
+        }
+    }
+
+    companion object {
+        private const val HOME_PATH = "home"
+    }
+}
diff --git a/WordPress/src/test/java/org/wordpress/android/ui/deeplinks/handlers/DeepLinkHandlersTest.kt b/WordPress/src/test/java/org/wordpress/android/ui/deeplinks/handlers/DeepLinkHandlersTest.kt
@@ -20,6 +20,7 @@ class DeepLinkHandlersTest : BaseUnitTest() {
     @Mock lateinit var pagesLinkHandler: PagesLinkHandler
     @Mock lateinit var notificationsLinkHandler: NotificationsLinkHandler
     @Mock lateinit var qrCodeAuthLinkHandler: QRCodeAuthLinkHandler
+    @Mock lateinit var homeLinkHandler: HomeLinkHandler
     @Mock lateinit var uri: UriWrapper
     private lateinit var deepLinkHandlers: DeepLinkHandlers
     private lateinit var handlers: List<DeepLinkHandler>
@@ -33,7 +34,8 @@ class DeepLinkHandlersTest : BaseUnitTest() {
                 readerLinkHandler,
                 pagesLinkHandler,
                 notificationsLinkHandler,
-                qrCodeAuthLinkHandler
+                qrCodeAuthLinkHandler,
+                homeLinkHandler
         )
         initDeepLinkHandlers()
     }
@@ -46,7 +48,8 @@ class DeepLinkHandlersTest : BaseUnitTest() {
                 readerLinkHandler,
                 pagesLinkHandler,
                 notificationsLinkHandler,
-                qrCodeAuthLinkHandler
+                qrCodeAuthLinkHandler,
+                homeLinkHandler
         )
     }
 
diff --git a/WordPress/src/test/java/org/wordpress/android/ui/deeplinks/handlers/HomeLinkHandlerTest.kt b/WordPress/src/test/java/org/wordpress/android/ui/deeplinks/handlers/HomeLinkHandlerTest.kt
@@ -0,0 +1,52 @@
+package org.wordpress.android.ui.deeplinks.handlers
+
+import com.nhaarman.mockitokotlin2.mock
+import com.nhaarman.mockitokotlin2.whenever
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.Before
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.mockito.Mock
+import org.mockito.junit.MockitoJUnitRunner
+import org.wordpress.android.fluxc.store.AccountStore
+import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenMySite
+import org.wordpress.android.ui.deeplinks.DeepLinkNavigator.NavigateAction.OpenLoginPrologue
+import org.wordpress.android.ui.deeplinks.buildUri
+
+@RunWith(MockitoJUnitRunner::class)
+class HomeLinkHandlerTest {
+    @Mock lateinit var accountStore: AccountStore
+    private lateinit var homeLinkHandler: HomeLinkHandler
+
+    @Before
+    fun setUp() {
+        homeLinkHandler = HomeLinkHandler(accountStore)
+    }
+
+    @Test
+    fun `returns login prologue action when user is logged out`() {
+        whenever(accountStore.hasAccessToken()).thenReturn(false)
+
+        val navigateAction = homeLinkHandler.buildNavigateAction(mock())
+
+        assertThat(navigateAction).isEqualTo(OpenLoginPrologue)
+    }
+
+    @Test
+    fun `returns my site action when user is logged in`() {
+        whenever(accountStore.hasAccessToken()).thenReturn(true)
+
+        val navigateAction = homeLinkHandler.buildNavigateAction(mock())
+
+        assertThat(navigateAction).isEqualTo(OpenMySite)
+    }
+
+    @Test
+    fun `builds URL for tracking`() {
+        val homeUri = buildUri("home")
+
+        val strippedUrl = homeLinkHandler.stripUrl(homeUri)
+
+        assertThat(strippedUrl).isEqualTo("wordpress://home")
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -1751,4 +1751,9 @@ public static void startQRCodeAuthFlowInNewStack(@NonNull Context context, @NonN`
`1751`	`1751`
`1752`	`1752`	`taskStackBuilder.startActivities();`
`1753`	`1753`	`}`
	`1754`	`+`
	`1755`	`+ public static void showLoginPrologue(@NonNull Context context) {`
	`1756`	`+ Intent intent = new Intent(context, LoginActivity.class);`
	`1757`	`+ context.startActivity(intent);`
	`1758`	`+ }`
`1754`	`1759`	`}`