chore(auth): refractor handleSignedIn method in AuthenticatorViewModel (#192)

Co-authored-by: Edison Zhang <edisonzz@amazon.com>
Co-authored-by: Matt Creaser <mattcreaser@gmail.com>

Author: 3 people

authenticator/src/main/java/com/amplifyframework/ui/authenticator/AuthenticatorViewModel.kt

@@ -582,7 +582,6 @@ internal class AuthenticatorViewModel(
                     logger.error(result.error.toString())
                     logger.error("Current signed in user session has expired, signing out.")
                     signOut()
-                    moveTo(AuthenticatorStep.SignIn)
                 } else {
                     handleGeneralFailure(result.error)
                 }

authenticator/src/test/java/com/amplifyframework/ui/authenticator/AuthenticatorViewModelTest.kt

@@ -133,7 +133,7 @@ class AuthenticatorViewModelTest {
     }
 
     @Test
-    fun `getCurrentUser error with session expired exception during start results in SignIn state`() = runTest {
+    fun `getCurrentUser error with session expired exception during start results in being signed out`() = runTest {
         coEvery { authProvider.fetchAuthSession() } returns Success(mockAuthSession(isSignedIn = true))
         coEvery { authProvider.getCurrentUser() } returns AmplifyResult.Error(SessionExpiredException())
 
@@ -143,8 +143,8 @@ class AuthenticatorViewModelTest {
         coVerify(exactly = 1) {
             authProvider.fetchAuthSession()
             authProvider.getCurrentUser()
+            authProvider.signOut()
         }
-        viewModel.currentStep shouldBe AuthenticatorStep.SignIn
     }
 
     @Test