Refactor alerts to be presented on their respective coordinator

ios/.swiftlint.yml

@@ -31,6 +31,8 @@ line_length:
   ignores_interpolated_strings: true
   warning: 120
   error: 300
+cyclomatic_complexity:
+  ignores_case_statements: true
 
 type_name:
   min_length: 4

ios/MullvadVPN/Classes/AppRoutes.swift

@@ -94,7 +94,7 @@ enum AppRoute: AppRouteProtocol {
     /**
      Alert route.
      */
-    case alert(AlertPresentation)
+    case alert(String)
 
     /**
      Routes that are part of primary horizontal navigation group.

ios/MullvadVPN/Coordinators/AccountCoordinator.swift

@@ -29,10 +29,6 @@ final class AccountCoordinator: Coordinator, Presentable, Presenting {
         navigationController
     }
 
-    var presentationContext: UIViewController {
-        navigationController
-    }
-
     var didFinish: ((AccountCoordinator, AccountDismissReason) -> Void)?
 
     init(
@@ -48,7 +44,7 @@ final class AccountCoordinator: Coordinator, Presentable, Presenting {
 
         let accountController = AccountViewController(
             interactor: interactor,
-            errorPresenter: PaymentAlertPresenter(coordinator: self)
+            errorPresenter: PaymentAlertPresenter(alertContext: self)
         )
 
         accountController.actionHandler = handleViewControllerAction
@@ -133,18 +129,25 @@ final class AccountCoordinator: Coordinator, Presentable, Presenting {
     // MARK: - Alerts
 
     private func logOut() {
-        let presentation = AlertPresentation(icon: .spinner, message: nil, buttons: [])
+        let presentation = AlertPresentation(
+            id: "account-logout-alert",
+            icon: .spinner,
+            message: nil,
+            buttons: []
+        )
+
+        let alertPresenter = AlertPresenter(context: self)
 
         interactor.logout {
             DispatchQueue.main.asyncAfter(deadline: .now() + .seconds(1)) { [weak self] in
                 guard let self else { return }
 
-                applicationRouter?.dismiss(.alert(presentation), animated: true)
+                alertPresenter.dismissAlert(presentation: presentation, animated: true)
                 self.didFinish?(self, .userLoggedOut)
             }
         }
 
-        applicationRouter?.present(.alert(presentation))
+        alertPresenter.showAlert(presentation: presentation, animated: true)
     }
 
     private func showAccountDeviceInfo() {
@@ -164,6 +167,7 @@ final class AccountCoordinator: Coordinator, Presentable, Presenting {
         )
 
         let presentation = AlertPresentation(
+            id: "account-device-info-alert",
             message: message,
             buttons: [AlertAction(
                 title: NSLocalizedString(
@@ -176,6 +180,7 @@ final class AccountCoordinator: Coordinator, Presentable, Presenting {
             )]
         )
 
-        applicationRouter?.present(.alert(presentation), animated: true)
+        let presenter = AlertPresenter(context: self)
+        presenter.showAlert(presentation: presentation, animated: true)
     }
 }

ios/MullvadVPN/Coordinators/ApplicationCoordinator.swift

@@ -16,8 +16,7 @@ import UIKit
  Application coordinator managing split view and two navigation contexts.
  */
 final class ApplicationCoordinator: Coordinator, Presenting, RootContainerViewControllerDelegate,
-    UISplitViewControllerDelegate, ApplicationRouterDelegate,
-    NotificationManagerDelegate {
+    UISplitViewControllerDelegate, ApplicationRouterDelegate, NotificationManagerDelegate {
     typealias RouteType = AppRoute
 
     /**
@@ -120,11 +119,11 @@ final class ApplicationCoordinator: Coordinator, Presenting, RootContainerViewCo
 
     func applicationRouter(
         _ router: ApplicationRouter<RouteType>,
-        route: AppRoute,
+        presentWithContext context: RoutePresentationContext<RouteType>,
         animated: Bool,
         completion: @escaping (Coordinator) -> Void
     ) {
-        switch route {
+        switch context.route {
         case .account:
             presentAccount(animated: animated, completion: completion)
 
@@ -155,8 +154,8 @@ final class ApplicationCoordinator: Coordinator, Presenting, RootContainerViewCo
         case .welcome:
             presentWelcome(animated: animated, completion: completion)
 
-        case let .alert(presentation):
-            presentAlert(presentation: presentation, animated: animated, completion: completion)
+        case .alert:
+            presentAlert(animated: animated, context: context, completion: completion)
         }
     }
 
@@ -564,11 +563,10 @@ final class ApplicationCoordinator: Coordinator, Presenting, RootContainerViewCo
         )
 
         coordinator.didFinishPayment = { [weak self] _ in
-            guard let self else { return }
+            guard let self = self else { return }
 
             if shouldDismissOutOfTime() {
                 router.dismiss(.outOfTime, animated: true)
-
                 continueFlow(animated: true)
             }
         }
@@ -589,7 +587,7 @@ final class ApplicationCoordinator: Coordinator, Presenting, RootContainerViewCo
         )
 
         coordinator.didFinish = { [weak self] _ in
-            guard let self else { return }
+            guard let self = self else { return }
             appPreferences.isShownOnboarding = true
             router.dismiss(.welcome, animated: false)
             continueFlow(animated: false)
@@ -642,19 +640,24 @@ final class ApplicationCoordinator: Coordinator, Presenting, RootContainerViewCo
     }
 
     private func presentAlert(
-        presentation: AlertPresentation,
         animated: Bool,
+        context: RoutePresentationContext<RouteType>,
         completion: @escaping (Coordinator) -> Void
     ) {
-        let coordinator = AlertCoordinator(presentation: presentation)
+        guard let metadata = context.metadata as? AlertMetadata else {
+            assertionFailure("Could not get AlertMetadata from RoutePresentationContext.")
+            return
+        }
+
+        let coordinator = AlertCoordinator(presentation: metadata.presentation)
 
         coordinator.didFinish = { [weak self] in
-            self?.router.dismiss(.alert(presentation))
+            self?.router.dismiss(.alert(metadata.presentation.id))
         }
 
         coordinator.start()
 
-        presentChild(coordinator, animated: animated) {
+        metadata.context.presentChild(coordinator, animated: animated) {
             completion(coordinator)
         }
     }
@@ -996,4 +999,6 @@ private protocol Poppable: Presentable {
         animated: Bool,
         completion: () -> Void
     )
+
+    // swiftlint:disable:next file_length
 }

ios/MullvadVPN/Coordinators/InAppPurchaseCoordinator.swift

@@ -11,7 +11,7 @@ import Routing
 import StoreKit
 import UIKit
 
-class InAppPurchaseCoordinator: Coordinator, Presentable {
+class InAppPurchaseCoordinator: Coordinator, Presenting, Presentable {
     private let navigationController: RootContainerViewController
     private let interactor: InAppPurchaseInteractor
 
@@ -50,6 +50,7 @@ class InAppPurchaseCoordinator: Coordinator, Presentable {
 
             case let .failure(failure):
                 let presentation = AlertPresentation(
+                    id: "in-app-purchase-error-alert",
                     icon: .alert,
                     message: failure.error.localizedDescription,
                     buttons: [
@@ -69,7 +70,8 @@ class InAppPurchaseCoordinator: Coordinator, Presentable {
                     ]
                 )
 
-                applicationRouter?.present(.alert(presentation), animated: true)
+                let presenter = AlertPresenter(context: self)
+                presenter.showAlert(presentation: presentation, animated: true)
             }
         }
     }

ios/MullvadVPN/Coordinators/LoginCoordinator.swift

@@ -12,7 +12,7 @@ import Operations
 import Routing
 import UIKit
 
-final class LoginCoordinator: Coordinator, DeviceManagementViewControllerDelegate {
+final class LoginCoordinator: Coordinator, Presenting, DeviceManagementViewControllerDelegate {
     private let tunnelManager: TunnelManager
     private let devicesProxy: REST.DevicesProxy
 
@@ -22,6 +22,10 @@ final class LoginCoordinator: Coordinator, DeviceManagementViewControllerDelegat
     var didFinish: ((LoginCoordinator) -> Void)?
     var didCreateAccount: (() -> Void)?
 
+    var presentationContext: UIViewController {
+        navigationController
+    }
+
     let navigationController: RootContainerViewController
 
     init(
@@ -107,11 +111,12 @@ final class LoginCoordinator: Coordinator, DeviceManagementViewControllerDelegat
         )
         let controller = DeviceManagementViewController(
             interactor: interactor,
-            alertPresenter: AlertPresenter(coordinator: self)
+            alertPresenter: AlertPresenter(context: self)
         )
         controller.delegate = self
+
         controller.fetchDevices(animateUpdates: false) { [weak self] result in
-            guard let self else { return }
+            guard let self = self else { return }
 
             switch result {
             case .success:

ios/MullvadVPN/Coordinators/OutOfTimeCoordinator.swift

@@ -9,13 +9,17 @@
 import Routing
 import UIKit
 
-class OutOfTimeCoordinator: Coordinator, OutOfTimeViewControllerDelegate {
+class OutOfTimeCoordinator: Coordinator, Presenting, OutOfTimeViewControllerDelegate {
     let navigationController: RootContainerViewController
     let storePaymentManager: StorePaymentManager
     let tunnelManager: TunnelManager
 
     var didFinishPayment: ((OutOfTimeCoordinator) -> Void)?
 
+    var presentationContext: UIViewController {
+        navigationController
+    }
+
     private(set) var isMakingPayment = false
     private var viewController: OutOfTimeViewController?
 
@@ -42,7 +46,7 @@ class OutOfTimeCoordinator: Coordinator, OutOfTimeViewControllerDelegate {
 
         let controller = OutOfTimeViewController(
             interactor: interactor,
-            errorPresenter: PaymentAlertPresenter(coordinator: self)
+            errorPresenter: PaymentAlertPresenter(alertContext: self)
         )
 
         controller.delegate = self

ios/MullvadVPN/Coordinators/SettingsCoordinator.swift

@@ -32,10 +32,6 @@ final class SettingsCoordinator: Coordinator, Presentable, Presenting, SettingsV
         navigationController
     }
 
-    var presentationContext: UIViewController {
-        navigationController
-    }
-
     var willNavigate: ((
         _ coordinator: SettingsCoordinator,
         _ from: SettingsNavigationRoute?,
@@ -159,13 +155,13 @@ final class SettingsCoordinator: Coordinator, Presentable, Presenting, SettingsV
         case .preferences:
             return PreferencesViewController(
                 interactor: interactorFactory.makePreferencesInteractor(),
-                alertPresenter: AlertPresenter(coordinator: self)
+                alertPresenter: AlertPresenter(context: self)
             )
 
         case .problemReport:
             return ProblemReportViewController(
                 interactor: interactorFactory.makeProblemReportInteractor(),
-                alertPresenter: AlertPresenter(coordinator: self)
+                alertPresenter: AlertPresenter(context: self)
             )
 
         case .faq:

ios/MullvadVPN/Coordinators/TunnelCoordinator.swift

@@ -9,12 +9,16 @@
 import Routing
 import UIKit
 
-class TunnelCoordinator: Coordinator {
+class TunnelCoordinator: Coordinator, Presenting {
     private let tunnelManager: TunnelManager
     private let controller: TunnelViewController
 
     private var tunnelObserver: TunnelObserver?
 
+    var presentationContext: UIViewController {
+        controller
+    }
+
     var rootViewController: UIViewController {
         controller
     }
@@ -59,6 +63,7 @@ class TunnelCoordinator: Coordinator {
 
     private func showCancelTunnelAlert() {
         let presentation = AlertPresentation(
+            id: "main-cancel-tunnel-alert",
             icon: .alert,
             message: NSLocalizedString(
                 "CANCEL_TUNNEL_ALERT_MESSAGE",
@@ -91,6 +96,7 @@ class TunnelCoordinator: Coordinator {
             ]
         )
 
-        applicationRouter?.present(.alert(presentation), animated: true)
+        let presenter = AlertPresenter(context: self)
+        presenter.showAlert(presentation: presentation, animated: true)
     }
 }

ios/MullvadVPN/Coordinators/WelcomeCoordinator.swift

@@ -26,10 +26,6 @@ final class WelcomeCoordinator: Coordinator, Presentable, Presenting {
         navigationController
     }
 
-    var presentationContext: UIViewController {
-        navigationController
-    }
-
     init(
         navigationController: RootContainerViewController,
         storePaymentManager: StorePaymentManager,
@@ -102,6 +98,7 @@ extension WelcomeCoordinator: WelcomeViewControllerDelegate {
         )
 
         let presentation = AlertPresentation(
+            id: "welcome-device-name-alert",
             icon: .info,
             message: message,
             buttons: [
@@ -117,7 +114,8 @@ extension WelcomeCoordinator: WelcomeViewControllerDelegate {
             ]
         )
 
-        applicationRouter?.present(.alert(presentation), animated: true)
+        let presenter = AlertPresenter(context: self)
+        presenter.showAlert(presentation: presentation, animated: true)
     }
 
     func didRequestToPurchaseCredit(controller: WelcomeViewController, accountNumber: String, product: SKProduct) {
@@ -150,7 +148,7 @@ extension WelcomeCoordinator: WelcomeViewControllerDelegate {
         )
 
         coordinator.didCancel = { [weak self] coordinator in
-            guard let self else { return }
+            guard let self = self else { return }
             navigationController.popViewController(animated: true)
             coordinator.removeFromParent()
         }

ios/MullvadVPN/SceneDelegate.swift

@@ -27,6 +27,7 @@ class SceneDelegate: UIResponder, UIWindowSceneDelegate, SettingsMigrationUIHand
     private var tunnelObserver: TunnelObserver?
 
     private var appDelegate: AppDelegate {
+        // swiftlint:disable:next force_cast
         UIApplication.shared.delegate as! AppDelegate
     }
 
@@ -185,7 +186,13 @@ class SceneDelegate: UIResponder, UIWindowSceneDelegate, SettingsMigrationUIHand
     // MARK: - SettingsMigrationUIHandler
 
     func showMigrationError(_ error: Error, completionHandler: @escaping () -> Void) {
+        guard let appCoordinator else {
+            completionHandler()
+            return
+        }
+
         let presentation = AlertPresentation(
+            id: "settings-migration-error-alert",
             title: NSLocalizedString(
                 "ALERT_TITLE",
                 tableName: "SettingsMigrationUI",
@@ -204,7 +211,8 @@ class SceneDelegate: UIResponder, UIWindowSceneDelegate, SettingsMigrationUIHand
             ]
         )
 
-        appCoordinator?.router.present(.alert(presentation), animated: true) ?? completionHandler()
+        let presenter = AlertPresenter(context: appCoordinator)
+        presenter.showAlert(presentation: presentation, animated: true)
     }
 
     private static func migrationErrorReason(_ error: Error) -> String {