fix(AuthenticatorState): Making move(to:) public

aws-amplify · Apr 22, 2024 · 2a28367 · 2a28367
1 parent 71fb556
commit 2a28367
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/Sources/Authenticator/Models/AuthenticatorState.swift b/Sources/Authenticator/Models/AuthenticatorState.swift
@@ -48,12 +48,19 @@ public class AuthenticatorState: ObservableObject, AuthenticatorStateProtocol {
         }
     }
 
-    func move(to initialStep: AuthenticatorInitialStep) {
+    /// Manually moves the Authenticator to an initial step
+    /// - Parameter initialStep: The desired ``AuthenticatorInitialStep``
+    public func move(to initialStep: AuthenticatorInitialStep) {
         if case .signedIn(_) = step {
             log.error("Cannot move to \(initialStep), the user is currently signed in. Call signOut first.")
             return
         }
 
+        guard step != .init(from: initialStep) else {
+            log.warn("Attempted to move to \(initialStep), but the Authenticator is already in that step.")
+            return
+        }
+
         setCurrentStep(.init(from: initialStep))
     }
 

diff --git a/Sources/Authenticator/Models/AuthenticatorStep.swift b/Sources/Authenticator/Models/AuthenticatorStep.swift
@@ -28,7 +28,6 @@ public struct AuthenticatorInitialStep: Equatable {
 /// An `AuthenticatorStep` represents a "state" or "view" for the Authenticator component within its lifecycle.
 public struct AuthenticatorStep: Equatable {
     private let name: String
-    //private(set) public var state: State = .signedOut
 
     private init(_ name: String) {
         self.name = name