Fix navigationStackCoordinator getting torn down when being reset on … (#613)

* Fix navigationStackCoordinator getting torn down when being reset on the navigationSplitCoordinator, added unit tests for it

* Guard against using the same coordinator more than once

ElementX/Sources/Application/Navigation/NavigationCoordinators.swift

@@ -138,6 +138,10 @@ class NavigationSplitCoordinator: CoordinatorProtocol, ObservableObject, CustomS
             return
         }
         
+        if sidebarModule?.coordinator === coordinator {
+            fatalError("Cannot use the same coordinator more than once")
+        }
+        
         sidebarModule = NavigationModule(coordinator, dismissalCallback: dismissalCallback)
     }
     
@@ -151,6 +155,10 @@ class NavigationSplitCoordinator: CoordinatorProtocol, ObservableObject, CustomS
             return
         }
         
+        if detailModule?.coordinator === coordinator {
+            fatalError("Cannot use the same coordinator more than once")
+        }
+        
         detailModule = NavigationModule(coordinator, dismissalCallback: dismissalCallback)
     }
     
@@ -164,6 +172,10 @@ class NavigationSplitCoordinator: CoordinatorProtocol, ObservableObject, CustomS
             return
         }
         
+        if sheetModule?.coordinator === coordinator {
+            fatalError("Cannot use the same coordinator more than once")
+        }
+        
         sheetModule = NavigationModule(coordinator, dismissalCallback: dismissalCallback)
     }
     
@@ -177,6 +189,10 @@ class NavigationSplitCoordinator: CoordinatorProtocol, ObservableObject, CustomS
             return
         }
         
+        if fullScreenCoverModule?.coordinator === coordinator {
+            fatalError("Cannot use the same coordinator more than once")
+        }
+        
         fullScreenCoverModule = NavigationModule(coordinator, dismissalCallback: dismissalCallback)
     }
         
@@ -509,6 +525,10 @@ class NavigationStackCoordinator: ObservableObject, CoordinatorProtocol, CustomS
             return
         }
         
+        if rootModule?.coordinator === coordinator {
+            fatalError("Cannot use the same coordinator more than once")
+        }
+        
         popToRoot(animated: false)
         
         rootModule = NavigationModule(coordinator, dismissalCallback: dismissalCallback)
@@ -565,6 +585,10 @@ class NavigationStackCoordinator: ObservableObject, CoordinatorProtocol, CustomS
             return
         }
         
+        if sheetModule?.coordinator === coordinator {
+            fatalError("Cannot use the same coordinator more than once")
+        }
+        
         sheetModule = NavigationModule(coordinator, dismissalCallback: dismissalCallback)
     }
     
@@ -584,6 +608,10 @@ class NavigationStackCoordinator: ObservableObject, CoordinatorProtocol, CustomS
             return
         }
         
+        if fullScreenCoverModule?.coordinator === coordinator {
+            fatalError("Cannot use the same coordinator more than once")
+        }
+        
         fullScreenCoverModule = NavigationModule(coordinator, dismissalCallback: dismissalCallback)
     }
     

ElementX/Sources/Services/UserSession/UserSessionFlowCoordinator.swift

@@ -168,8 +168,7 @@ class UserSessionFlowCoordinator: CoordinatorProtocol {
                                                              emojiProvider: emojiProvider)
             let coordinator = RoomScreenCoordinator(parameters: parameters)
             
-            detailNavigationStackCoordinator.setRootCoordinator(coordinator)
+            detailNavigationStackCoordinator.setRootCoordinator(coordinator) { [weak self, roomIdentifier] in
-            navigationSplitCoordinator.setDetailCoordinator(detailNavigationStackCoordinator) { [weak self, roomIdentifier] in
                 guard let self else { return }
                 
                 // Move the state machine to no room selected if the room currently being dimissed
@@ -180,6 +179,10 @@ class UserSessionFlowCoordinator: CoordinatorProtocol {
                     self.detailNavigationStackCoordinator.setRootCoordinator(nil)
                 }
             }
+            
+            if navigationSplitCoordinator.detailCoordinator == nil {
+                navigationSplitCoordinator.setDetailCoordinator(detailNavigationStackCoordinator)
+            }
         }
     }
         

UnitTests/Sources/NavigationSplitCoordinatorTests.swift

@@ -259,9 +259,13 @@ class NavigationSplitCoordinatorTests: XCTestCase {
     // MARK: - Private
     
     private func assertCoordinatorsEqual(_ lhs: CoordinatorProtocol?, _ rhs: CoordinatorProtocol?) {
+        if lhs == nil, rhs == nil {
+            return
+        }
+        
         guard let lhs = lhs as? SomeTestCoordinator,
               let rhs = rhs as? SomeTestCoordinator else {
-            XCTFail("Coordinators are not the same")
+            XCTFail("Coordinators are not the same: \(String(describing: lhs)) != \(String(describing: rhs))")
             return
         }
         

changelog.d/pr-613.bugfix

@@ -0,0 +1 @@
+Fix broken split layout room navigation