[LOOP-4597] updated handling of pump deactivation as apart of reset (#562)

Author: nhamming

Loop/Managers/Alerts/AlertManager.swift

@@ -778,16 +778,24 @@ extension AlertManager: AlertPermissionsCheckerDelegate {
 }
 
 extension AlertManager {
-    func presentConfirmCrashAlert(confirmAction: @escaping () -> Void) {
+    func presentConfirmCrashAlert(confirmAction: @escaping (@escaping () -> Void) -> Void) {
         let alert = UIAlertController(title: "New Study Product Detected", message: "We've detected a new study product is selected. In order to show use this study product, Tidepool Loop will need to restart.", preferredStyle: .alert)
         alert.addAction(UIAlertAction(title: "Confirm", style: .default, handler: { _ in
-            confirmAction()
-            fatalError("DEBUG: Resetting Loop")
+            confirmAction() {
+                fatalError("DEBUG: Resetting Loop")
+            }
         }))
         alert.addAction(UIAlertAction(title: "Cancel", style: .cancel))
 
         alertPresenter.present(alert, animated: true)
     }
+    
+    func presentCouldNotResetLoopAlert(error: Error) {
+        let alert = UIAlertController(title: "Could Not Restart Tidepool Loop", message: "While trying to restart Tidepool Loop an error occured.\n\n\(error.localizedDescription)", preferredStyle: .alert)
+        alert.addAction(UIAlertAction(title: "Cancel", style: .cancel))
+        
+        alertPresenter.present(alert, animated: true)
+    }
 }
 
 fileprivate extension UserDefaults {

Loop/Managers/LoopAppManager.swift

@@ -374,9 +374,21 @@ class LoopAppManager: NSObject {
         deviceDataManager.pluginManager.availableSupports.forEach { supportUI in
             if supportUI.loopNeedsReset {
                 supportUI.loopNeedsReset = false
-                alertManager.presentConfirmCrashAlert() { [weak self] in
-                    self?.deviceDataManager.pumpManager?.prepareForDeactivation({ _ in })
-                    supportUI.resetLoop()
+                alertManager.presentConfirmCrashAlert() { [weak self] completion in
+                    guard let pumpManager = self?.deviceDataManager.pumpManager else {
+                        supportUI.resetLoop()
+                        completion()
+                        return
+                    }
+                    
+                    pumpManager.prepareForDeactivation() { [weak self] error in
+                        guard let error = error else {
+                            supportUI.resetLoop()
+                            completion()
+                            return
+                        }
+                        self?.alertManager.presentCouldNotResetLoopAlert(error: error)
+                    }
                 }
             }
         }