PM-14632: Revert to legacy create account flow if updated environment doesn't support email verification

BitwardenShared/UI/Auth/IntroCarousel/IntroCarouselProcessor.swift

@@ -70,4 +70,10 @@ extension IntroCarouselProcessor: StartRegistrationDelegate {
     func didChangeRegion() async {
         // No-op: the carousel doesn't show the region so there's nothing to do here.
     }
+
+    func switchToLegacyCreateAccountFlow() {
+        coordinator.navigate(to: .dismissWithAction(DismissAction {
+            self.coordinator.navigate(to: .createAccount)
+        }))
+    }
 }

BitwardenShared/UI/Auth/IntroCarousel/IntroCarouselProcessorTests.swift

@@ -71,4 +71,19 @@ class IntroCarouselProcessorTests: BitwardenTestCase {
         subject.receive(.logIn)
         XCTAssertEqual(coordinator.routes.last, .landing)
     }
+
+    /// `switchToLegacyCreateAccountFlow()` dismisses the currently presented view and navigates to
+    /// create account.
+    @MainActor
+    func test_switchToLegacyCreateAccountFlow() throws {
+        subject.switchToLegacyCreateAccountFlow()
+
+        let dismissRoute = try XCTUnwrap(coordinator.routes.last)
+        guard case let .dismissWithAction(action) = dismissRoute else {
+            return XCTFail("Expected route `.dismissWithAction` not found.")
+        }
+        action?.action()
+
+        XCTAssertEqual(coordinator.routes, [.dismissWithAction(action), .createAccount])
+    }
 }

BitwardenShared/UI/Auth/Landing/LandingProcessor.swift

@@ -219,6 +219,12 @@ extension LandingProcessor: StartRegistrationDelegate {
     func didChangeRegion() async {
         await regionHelper.loadRegion()
     }
+
+    func switchToLegacyCreateAccountFlow() {
+        coordinator.navigate(to: .dismissWithAction(DismissAction {
+            self.coordinator.navigate(to: .createAccount)
+        }))
+    }
 }
 
 // MARK: - RegionDelegate

BitwardenShared/UI/Auth/Landing/LandingProcessorTests.swift

@@ -875,4 +875,19 @@ class LandingProcessorTests: BitwardenTestCase { // swiftlint:disable:this type_
         subject.receive(.toastShown(nil))
         XCTAssertNil(subject.state.toast)
     }
+
+    /// `switchToLegacyCreateAccountFlow()` dismisses the currently presented view and navigates to
+    /// create account.
+    @MainActor
+    func test_switchToLegacyCreateAccountFlow() throws {
+        subject.switchToLegacyCreateAccountFlow()
+
+        let dismissRoute = try XCTUnwrap(coordinator.routes.last)
+        guard case let .dismissWithAction(action) = dismissRoute else {
+            return XCTFail("Expected route `.dismissWithAction` not found.")
+        }
+        action?.action()
+
+        XCTAssertEqual(coordinator.routes, [.dismissWithAction(action), .createAccount])
+    }
 } // swiftlint:disable:this file_length

BitwardenShared/UI/Auth/StartRegistration/StartRegistrationProcessor.swift

@@ -12,6 +12,11 @@ protocol StartRegistrationDelegate: AnyObject {
     /// Called when the user changes regions.
     ///
     func didChangeRegion() async
+
+    /// If the user changes environments and the environment doesn't support email verification,
+    /// the UI should switch to using the legacy create account flow.
+    ///
+    func switchToLegacyCreateAccountFlow()
 }
 
 // MARK: - StartRegistrationError
@@ -229,5 +234,16 @@ extension StartRegistrationProcessor: RegionDelegate {
         state.region = region
         state.showReceiveMarketingToggle = state.region != .selfHosted
         await delegate?.didChangeRegion()
+
+        if await !services.configService.getFeatureFlag(
+            .emailVerification,
+            defaultValue: false,
+            forceRefresh: true,
+            isPreAuth: true
+        ) {
+            // If email verification isn't enabled in the selected environment, switch to the
+            // legacy create account flow.
+            delegate?.switchToLegacyCreateAccountFlow()
+        }
     }
 }

BitwardenShared/UI/Auth/StartRegistration/StartRegistrationProcessorTests.swift

@@ -15,9 +15,9 @@ class StartRegistrationProcessorTests: BitwardenTestCase { // swiftlint:disable:
     var clientAuth: MockClientAuth!
     var configService: MockConfigService!
     var coordinator: MockCoordinator<AuthRoute, AuthEvent>!
+    var delegate: MockStartRegistrationDelegate!
     var errorReporter: MockErrorReporter!
     var subject: StartRegistrationProcessor!
-    var delegate: MockStartRegistrationDelegate!
     var stateService: MockStateService!
     var environmentService: MockEnvironmentService!
 
@@ -31,6 +31,7 @@ class StartRegistrationProcessorTests: BitwardenTestCase { // swiftlint:disable:
         clientAuth = MockClientAuth()
         configService = MockConfigService()
         coordinator = MockCoordinator<AuthRoute, AuthEvent>()
+        delegate = MockStartRegistrationDelegate()
         environmentService = MockEnvironmentService()
         errorReporter = MockErrorReporter()
         stateService = MockStateService()
@@ -629,12 +630,39 @@ class StartRegistrationProcessorTests: BitwardenTestCase { // swiftlint:disable:
         XCTAssertEqual(environmentService.setPreAuthEnvironmentUrlsData, .defaultEU)
         XCTAssertFalse(subject.state.isReceiveMarketingToggleOn)
     }
+
+    /// `setRegion(_:_:)` doesn't notify the delegate to switch to the legacy create account flow
+    /// if the selected region supports email verification.
+    @MainActor
+    func test_setRegion_emailVerificationEnabled() async {
+        configService.featureFlagsBoolPreAuth[.emailVerification] = true
+
+        await subject.setRegion(.unitedStates, .defaultUS)
+
+        XCTAssertFalse(delegate.switchToLegacyCreateAccountFlowCalled)
+    }
+
+    /// `setRegion(_:_:)` notifies the delegate to switch to the legacy create account flow if the
+    /// selected region doesn't support email verification.
+    @MainActor
+    func test_setRegion_emailVerificationDisabled() async {
+        configService.featureFlagsBoolPreAuth[.emailVerification] = false
+
+        await subject.setRegion(.unitedStates, .defaultUS)
+
+        XCTAssertTrue(delegate.switchToLegacyCreateAccountFlowCalled)
+    }
 }
 
 class MockStartRegistrationDelegate: StartRegistrationDelegate {
     var didChangeRegionCalled: Bool = false
+    var switchToLegacyCreateAccountFlowCalled = false
 
     func didChangeRegion() async {
         didChangeRegionCalled = true
     }
+
+    func switchToLegacyCreateAccountFlow() {
+        switchToLegacyCreateAccountFlowCalled = true
+    }
 } // swiftlint:disable:this file_length