Merge pull request #7641 from woocommerce/feat/7547-store-picker-help

Custom help content for "Store picker" screen

Author: selanthiraiyan

RELEASE-NOTES.txt

@@ -2,7 +2,7 @@
 
 10.4
 -----
-
+- [*] Help center: Added help center web page with FAQs for "Pick a WooCommerce Store" screen. [https://github.com/woocommerce/woocommerce-ios/pull/7641]
 
 10.3
 -----

WooCommerce/Classes/Authentication/Epilogue/StorePickerViewController.swift

@@ -289,7 +289,7 @@ private extension StorePickerViewController {
     }
 
     func presentHelp() {
-        ServiceLocator.authenticationManager.presentSupport(from: self, sourceTag: .generalLogin)
+        ServiceLocator.authenticationManager.presentSupport(from: self, screen: .storePicker)
     }
 }
 

WooCommerce/Classes/Model/CustomHelpCenterContent.swift

@@ -59,6 +59,10 @@ extension CustomHelpCenterContent {
         /// Jetpack required error screen presented using `JetpackErrorViewModel`
         ///
         case jetpackRequired
+
+        /// Store picker screen - `StorePickerViewController`
+        ///
+        case storePicker
     }
 
     init(screen: Screen, flow: AuthenticatorAnalyticsTracker.Flow) {
@@ -67,6 +71,9 @@ extension CustomHelpCenterContent {
         case .jetpackRequired:
             step = "jetpack_not_connected" // Matching Android `Step` value
             url = WooConstants.URLs.helpCenterForJetpackRequiredError.asURL()
+        case .storePicker:
+            step = "site_list" // Matching Android `Step` value
+            url = WooConstants.URLs.helpCenterForStorePicker.asURL()
         }
 
         trackingProperties = [

WooCommerce/Classes/System/WooConstants.swift

@@ -111,6 +111,10 @@ extension WooConstants {
         ///
         case helpCenterForJetpackRequiredError = "https://woocommerce.com/document/android-ios-apps-login-help-faq/#jetpack-required"
 
+        /// Help Center for "Store picker" screen
+        ///
+        case helpCenterForStorePicker = "https://woocommerce.com/document/android-ios-apps-login-help-faq/#pick-store-after-entering-password"
+
         /// URL used for Learn More button in Orders empty state.
         ///
         case blog = "https://woocommerce.com/blog/"

WooCommerce/WooCommerceTests/Model/CustomHelpCenterContentTests.swift

@@ -36,12 +36,12 @@ final class CustomHelpCenterContentTests: XCTestCase {
         // Given
         let step: AuthenticatorAnalyticsTracker.Step = .start
         let flow: AuthenticatorAnalyticsTracker.Flow = .loginWithSiteAddress
-        let helpContentURL = WooConstants.URLs.helpCenterForEnterStoreAddress.asURL()
 
         // When
         let sut = try XCTUnwrap(CustomHelpCenterContent(step: step, flow: flow))
 
         // Then
+        let helpContentURL = WooConstants.URLs.helpCenterForEnterStoreAddress.asURL()
         XCTAssertEqual(sut.url, helpContentURL)
 
         // Test the `trackingProperties` dictionary values
@@ -56,12 +56,12 @@ final class CustomHelpCenterContentTests: XCTestCase {
         // Given
         let step: AuthenticatorAnalyticsTracker.Step = .enterEmailAddress
         let flow: AuthenticatorAnalyticsTracker.Flow = .loginWithSiteAddress
-        let helpContentURL = WooConstants.URLs.helpCenterForWPCOMEmailFromSiteAddressFlow.asURL()
 
         // When
         let sut = try XCTUnwrap(CustomHelpCenterContent(step: step, flow: flow))
 
         // Then
+        let helpContentURL = WooConstants.URLs.helpCenterForWPCOMEmailFromSiteAddressFlow.asURL()
         XCTAssertEqual(sut.url, helpContentURL)
 
         // Test the `trackingProperties` dictionary values
@@ -76,12 +76,12 @@ final class CustomHelpCenterContentTests: XCTestCase {
         // Given
         let step: AuthenticatorAnalyticsTracker.Step = .enterEmailAddress
         let flow: AuthenticatorAnalyticsTracker.Flow = .wpCom
-        let helpContentURL = WooConstants.URLs.helpCenterForWPCOMEmailScreen.asURL()
 
         // When
         let sut = try XCTUnwrap(CustomHelpCenterContent(step: step, flow: flow))
 
         // Then
+        let helpContentURL = WooConstants.URLs.helpCenterForWPCOMEmailScreen.asURL()
         XCTAssertEqual(sut.url, helpContentURL)
 
         // Test the `trackingProperties` dictionary values
@@ -96,12 +96,12 @@ final class CustomHelpCenterContentTests: XCTestCase {
         // Given
         let step: AuthenticatorAnalyticsTracker.Step = .usernamePassword
         let flow: AuthenticatorAnalyticsTracker.Flow = .loginWithSiteAddress
-        let helpContentURL = WooConstants.URLs.helpCenterForEnterStoreCredentials.asURL()
 
         // When
         let sut = try XCTUnwrap(CustomHelpCenterContent(step: step, flow: flow))
 
         // Then
+        let helpContentURL = WooConstants.URLs.helpCenterForEnterStoreCredentials.asURL()
         XCTAssertEqual(sut.url, helpContentURL)
 
         // Test the `trackingProperties` dictionary values
@@ -116,12 +116,32 @@ final class CustomHelpCenterContentTests: XCTestCase {
         // Given
         let step = "jetpack_not_connected"
         let flow: AuthenticatorAnalyticsTracker.Flow = .loginWithPasswordWithMagicLinkEmphasis
-        let helpContentURL = WooConstants.URLs.helpCenterForJetpackRequiredError.asURL()
 
         // When
         let sut = CustomHelpCenterContent(screen: .jetpackRequired, flow: flow)
 
         // Then
+        let helpContentURL = WooConstants.URLs.helpCenterForJetpackRequiredError.asURL()
+        XCTAssertEqual(sut.url, helpContentURL)
+
+        // Test the `trackingProperties` dictionary values
+        XCTAssertEqual(sut.trackingProperties[CustomHelpCenterContent.Key.step.rawValue], step)
+        XCTAssertEqual(sut.trackingProperties[CustomHelpCenterContent.Key.flow.rawValue], flow.rawValue)
+        XCTAssertEqual(sut.trackingProperties[CustomHelpCenterContent.Key.url.rawValue], helpContentURL.absoluteString)
+    }
+
+    // MARK: Store picker screen - `StorePickerViewController`
+    //
+    func test_init_screen_returns_valid_instance_for_store_picker_screen() throws {
+        // Given
+        let step = "site_list"
+        let flow: AuthenticatorAnalyticsTracker.Flow = .loginWithPasswordWithMagicLinkEmphasis
+
+        // When
+        let sut = CustomHelpCenterContent(screen: .storePicker, flow: flow)
+
+        // Then
+        let helpContentURL = WooConstants.URLs.helpCenterForStorePicker.asURL()
         XCTAssertEqual(sut.url, helpContentURL)
 
         // Test the `trackingProperties` dictionary values