From f16e98dd9866876fd37ef4cd2703383feeae6a55 Mon Sep 17 00:00:00 2001
From: Tony Li <tony.li@automattic.com>
Date: Thu, 14 Nov 2024 21:30:14 +1300
Subject: [PATCH] Append "+testing" to feedback form email for Automatticians

---
 WordPress/Classes/Utility/ZendeskUtils.swift  | 34 ++++++++++++++++--
 .../Support/SupportTableViewController.swift  |  3 +-
 WordPress/WordPress.xcodeproj/project.pbxproj |  4 +++
 .../ZendeskUtilsTests+A8CEmail.swift          | 35 +++++++++++++++++++
 4 files changed, 72 insertions(+), 4 deletions(-)
 create mode 100644 WordPress/WordPressTest/ZendeskUtilsTests+A8CEmail.swift

diff --git a/WordPress/Classes/Utility/ZendeskUtils.swift b/WordPress/Classes/Utility/ZendeskUtils.swift
index 696a45881647..22bc6cff823a 100644
--- a/WordPress/Classes/Utility/ZendeskUtils.swift
+++ b/WordPress/Classes/Utility/ZendeskUtils.swift
@@ -42,7 +42,6 @@ protocol ZendeskUtilsProtocol {
 /// as well as displaying views for the Help Center, new tickets, and ticket list.
 ///
 @objc class ZendeskUtils: NSObject, ZendeskUtilsProtocol {
-
     // MARK: - Public Properties
 
     static var sharedInstance: ZendeskUtils = ZendeskUtils(contextManager: ContextManager.shared)
@@ -63,7 +62,15 @@ protocol ZendeskUtilsProtocol {
     private var sourceTag: WordPressSupportSourceTag?
 
     private var userName: String?
-    private var userEmail: String?
+    private var userEmail: String? {
+        set {
+            _userEmail = newValue.map(ZendeskUtils.a8cTestEmail(_:))
+        }
+        get {
+            _userEmail
+        }
+    }
+    private var _userEmail: String?
     private var userNameConfirmed = false
 
     private var deviceID: String?
@@ -1201,3 +1208,26 @@ extension ZendeskUtils {
         }
     }
 }
+
+extension ZendeskUtils {
+    static var automatticEmails = ["@automattic.com", "@a8c.com"]
+
+    /// Insert "+testing" to Automattic email address.
+    /// - SeeAlso: https://github.com/wordpress-mobile/WordPress-iOS/issues/23794
+    static func a8cTestEmail(_ email: String) -> String {
+        guard let suffix = ZendeskUtils.automatticEmails.first(where: { email.lowercased().hasSuffix($0) }) else {
+            return email
+        }
+
+        let atSymbolIndex = email.index(email.endIndex, offsetBy: -suffix.count)
+
+        // Do nothing if the email is already an "alias email".
+        if email[email.startIndex..<atSymbolIndex].contains("+") {
+            return email
+        }
+
+        var newEmail = email
+        newEmail.insert(contentsOf: "+testing", at: atSymbolIndex)
+        return newEmail
+    }
+}
diff --git a/WordPress/Classes/ViewRelated/Support/SupportTableViewController.swift b/WordPress/Classes/ViewRelated/Support/SupportTableViewController.swift
index a9b349e06d1e..1109d9400c87 100644
--- a/WordPress/Classes/ViewRelated/Support/SupportTableViewController.swift
+++ b/WordPress/Classes/ViewRelated/Support/SupportTableViewController.swift
@@ -333,7 +333,7 @@ private extension SupportTableViewController {
     ///
     func checkForAutomatticEmail() {
         guard let email = ZendeskUtils.userSupportEmail(),
-            (Constants.automatticEmails.first { email.contains($0) }) != nil else {
+              (ZendeskUtils.automatticEmails.first { email.contains($0) }) != nil else {
                 return
         }
 
@@ -526,7 +526,6 @@ private extension SupportTableViewController {
         static let jetpackMigrationFAQURL = URL(string: "https://jetpack.com/support/switch-to-the-jetpack-app/")
 
         static let forumsURL = URL(string: "https://wordpress.org/support/forum/mobile/")
-        static let automatticEmails = ["@automattic.com", "@a8c.com"]
     }
 }
 
diff --git a/WordPress/WordPress.xcodeproj/project.pbxproj b/WordPress/WordPress.xcodeproj/project.pbxproj
index 92842d087308..a09fd05df2b9 100644
--- a/WordPress/WordPress.xcodeproj/project.pbxproj
+++ b/WordPress/WordPress.xcodeproj/project.pbxproj
@@ -440,6 +440,7 @@
 		4A2C73E42A943DEA00ACE79E /* TaggedManagedObjectIDTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 4A2C73E32A943DEA00ACE79E /* TaggedManagedObjectIDTests.swift */; };
 		4A2C73F72A9585B000ACE79E /* PostRepositoryTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 4A2C73F62A9585B000ACE79E /* PostRepositoryTests.swift */; };
 		4A5598852B05AC180083C220 /* PagesListTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 4A5598842B05AC180083C220 /* PagesListTests.swift */; };
+		4A5EDD812CE58B9900A605FC /* ZendeskUtilsTests+A8CEmail.swift in Sources */ = {isa = PBXBuildFile; fileRef = 4A5EDD802CE58B9900A605FC /* ZendeskUtilsTests+A8CEmail.swift */; };
 		4A690C152BA791B100A8E0C5 /* PrivacyInfo.xcprivacy in Resources */ = {isa = PBXBuildFile; fileRef = 4A690C142BA790BC00A8E0C5 /* PrivacyInfo.xcprivacy */; };
 		4A690C162BA791B200A8E0C5 /* PrivacyInfo.xcprivacy in Resources */ = {isa = PBXBuildFile; fileRef = 4A690C142BA790BC00A8E0C5 /* PrivacyInfo.xcprivacy */; };
 		4A690C182BA794C800A8E0C5 /* PrivacyInfo.xcprivacy in Resources */ = {isa = PBXBuildFile; fileRef = 4A690C172BA794C300A8E0C5 /* PrivacyInfo.xcprivacy */; };
@@ -2329,6 +2330,7 @@
 		4A2C73F62A9585B000ACE79E /* PostRepositoryTests.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = PostRepositoryTests.swift; sourceTree = "<group>"; };
 		4A2F746A2C2198CB00E72117 /* WordPressAuthenticatorTests.xcconfig */ = {isa = PBXFileReference; lastKnownFileType = text.xcconfig; path = WordPressAuthenticatorTests.xcconfig; sourceTree = "<group>"; };
 		4A5598842B05AC180083C220 /* PagesListTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = PagesListTests.swift; sourceTree = "<group>"; };
+		4A5EDD802CE58B9900A605FC /* ZendeskUtilsTests+A8CEmail.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = "ZendeskUtilsTests+A8CEmail.swift"; sourceTree = "<group>"; };
 		4A690C142BA790BC00A8E0C5 /* PrivacyInfo.xcprivacy */ = {isa = PBXFileReference; lastKnownFileType = text.xml; path = PrivacyInfo.xcprivacy; sourceTree = "<group>"; };
 		4A690C172BA794C300A8E0C5 /* PrivacyInfo.xcprivacy */ = {isa = PBXFileReference; lastKnownFileType = text.xml; path = PrivacyInfo.xcprivacy; sourceTree = "<group>"; };
 		4A690C1A2BA7958F00A8E0C5 /* PrivacyInfo.xcprivacy */ = {isa = PBXFileReference; lastKnownFileType = text.xml; path = PrivacyInfo.xcprivacy; sourceTree = "<group>"; };
@@ -4733,6 +4735,7 @@
 			isa = PBXGroup;
 			children = (
 				3F751D452491A93D0008A2B1 /* ZendeskUtilsTests+Plans.swift */,
+				4A5EDD802CE58B9900A605FC /* ZendeskUtilsTests+A8CEmail.swift */,
 			);
 			name = Zendesk;
 			sourceTree = "<group>";
@@ -10085,6 +10088,7 @@
 				D81C2F6620F8ACCD002AE1F1 /* FormattableContentFormatterTests.swift in Sources */,
 				08AA640C2A8511FB0076E38D /* DashboardGoogleDomainsViewModelTests.swift in Sources */,
 				83EF3D7F2937F08C000AF9BF /* SharedDataIssueSolverTests.swift in Sources */,
+				4A5EDD812CE58B9900A605FC /* ZendeskUtilsTests+A8CEmail.swift in Sources */,
 				8070EB3E28D807CB005C6513 /* InMemoryUserDefaults.swift in Sources */,
 				F93735F822D53C3B00A3C312 /* LoggingURLRedactorTests.swift in Sources */,
 				C738CB1128626606001BE107 /* QRLoginVerifyCoordinatorTests.swift in Sources */,
diff --git a/WordPress/WordPressTest/ZendeskUtilsTests+A8CEmail.swift b/WordPress/WordPressTest/ZendeskUtilsTests+A8CEmail.swift
new file mode 100644
index 000000000000..95f8533f9ab5
--- /dev/null
+++ b/WordPress/WordPressTest/ZendeskUtilsTests+A8CEmail.swift
@@ -0,0 +1,35 @@
+import Foundation
+import Testing
+
+@testable import WordPress
+
+@Suite
+struct ZendeskUtilsA8CEmailTests {
+
+    @Test(
+        arguments: [
+            ("hello@automattic.com", "hello+testing@automattic.com"),
+            ("Hello@Automattic.com", "Hello+testing@Automattic.com"),
+            ("Hello@a8c.com", "Hello+testing@a8c.com"),
+            ("Hello@A8C.com", "Hello+testing@A8C.com"),
+        ]
+    )
+    func insertTesting(email: String, expected: String) {
+        let result = ZendeskUtils.a8cTestEmail(email)
+        #expect(result == expected)
+    }
+
+    @Test(
+        arguments: [
+            "Hello@world.com",
+            "Hello@World.com",
+            "Hello+Testing@automattic.com",
+            "Hello+World@a8c.com"
+        ]
+    )
+    func keepOriginalEmail(email: String) {
+        let result = ZendeskUtils.a8cTestEmail(email)
+        #expect(result == email)
+    }
+
+}