Merge pull request #7352 from woocommerce/feat/7318-login-wpcom-action

Login Reminder: add an action to log in with WP.com for site address error notiifcation

Author: jaclync

WooCommerce/Classes/Notifications/LocalNotification.swift

@@ -27,12 +27,15 @@ struct LocalNotification {
     /// The action type in a local notification.
     enum Action: String {
         case contactSupport
+        case loginWithWPCom
 
         /// The title of the action in a local notification.
         var title: String {
             switch self {
             case .contactSupport:
                 return NSLocalizedString("Contact support", comment: "Local notification action to contact support.")
+            case .loginWithWPCom:
+                return NSLocalizedString("Login with WordPress.com", comment: "Local notification action to log in with WordPress.com.")
             }
         }
     }
@@ -45,7 +48,7 @@ extension LocalNotification {
             self.init(title: Localization.errorLoggingInTitle,
                       body: Localization.errorLoggingInBody,
                       scenario: .loginSiteAddressError,
-                      actions: .init(category: .loginError, actions: [.contactSupport]))
+                      actions: .init(category: .loginError, actions: [.contactSupport, .loginWithWPCom]))
         }
     }
 }

WooCommerce/Classes/ViewRelated/AppCoordinator.swift

@@ -1,6 +1,7 @@
 import Combine
 import Experiments
 import UIKit
+import WordPressAuthenticator
 import Yosemite
 import class AutomatticTracks.CrashLogging
 
@@ -253,6 +254,17 @@ private extension AppCoordinator {
                 "action": "contact_support",
                 "type": response.notification.request.identifier
             ])
+        case LocalNotification.Action.loginWithWPCom.rawValue:
+            guard let loginNavigationController = window.rootViewController as? LoginNavigationController,
+                  let viewController = loginNavigationController.topViewController else {
+                return
+            }
+            let command = NavigateToEnterAccount()
+            command.execute(from: viewController)
+            analytics.track(.loginLocalNotificationTapped, withProperties: [
+                "action": "login_with_wpcom",
+                "type": response.notification.request.identifier
+            ])
         case UNNotificationDefaultActionIdentifier:
             // Triggered when the user taps on the notification itself instead of one of the actions.
             switch response.notification.request.identifier {

WooCommerce/WooCommerceTests/AppCoordinatorTests.swift

@@ -230,6 +230,30 @@ final class AppCoordinatorTests: XCTestCase {
         XCTAssertEqual(typePropertyValue, "site_address_error")
     }
 
+    func test_loginLocalNotificationTapped_is_tracked_after_notification_loginWithWPCom_action() throws {
+        // Given
+        let analytics = MockAnalyticsProvider()
+        let pushNotesManager = MockPushNotificationsManager()
+        let appCoordinator = makeCoordinator(window: window,
+                                             stores: stores,
+                                             authenticationManager: authenticationManager,
+                                             analytics: WooAnalytics(analyticsProvider: analytics),
+                                             pushNotesManager: pushNotesManager)
+        appCoordinator.start()
+
+        // When
+        let response = try XCTUnwrap(MockNotificationResponse(actionIdentifier: LocalNotification.Action.loginWithWPCom.rawValue,
+                                                              requestIdentifier: LocalNotification.Scenario.loginSiteAddressError.rawValue))
+        pushNotesManager.sendLocalNotificationResponse(response)
+
+        // Then
+        XCTAssertEqual(analytics.receivedEvents, [WooAnalyticsStat.loginLocalNotificationTapped.rawValue])
+        let actionPropertyValue = try XCTUnwrap(analytics.receivedProperties.first?["action"] as? String)
+        XCTAssertEqual(actionPropertyValue, "login_with_wpcom")
+        let typePropertyValue = try XCTUnwrap(analytics.receivedProperties.first?["type"] as? String)
+        XCTAssertEqual(typePropertyValue, "site_address_error")
+    }
+
     func test_loginLocalNotificationTapped_is_tracked_after_notification_tap_action() throws {
         // Given
         let analytics = MockAnalyticsProvider()