fix(auth): Remove online check from getCurrentUser (#3122)

Author: mattcreaser

aws-auth-cognito/src/main/java/com/amplifyframework/auth/cognito/helpers/SessionHelper.kt

@@ -77,3 +77,5 @@ internal object SessionHelper {
             )
     }
 }
+
+internal fun CognitoUserPoolTokens.isValid() = SessionHelper.isValidTokens(this)

aws-auth-cognito/src/main/java/com/amplifyframework/auth/cognito/usecases/AuthUseCaseFactory.kt

@@ -93,7 +93,6 @@ internal class AuthUseCaseFactory(
     )
 
     fun getCurrentUser() = GetCurrentUserUseCase(
-        fetchAuthSession = fetchAuthSession(),
         stateMachine = stateMachine
     )
 

aws-auth-cognito/src/main/java/com/amplifyframework/auth/cognito/usecases/GetCurrentUserUseCase.kt

@@ -18,20 +18,12 @@ package com.amplifyframework.auth.cognito.usecases
 import com.amplifyframework.auth.AuthUser
 import com.amplifyframework.auth.cognito.AuthStateMachine
 import com.amplifyframework.auth.cognito.requireSignedInState
-import com.amplifyframework.auth.exceptions.SessionExpiredException
-import com.amplifyframework.util.throwIf
 
 internal class GetCurrentUserUseCase(
-    private val fetchAuthSession: FetchAuthSessionUseCase,
     private val stateMachine: AuthStateMachine
 ) {
     suspend fun execute(): AuthUser {
         val state = stateMachine.requireSignedInState()
-
-        // Throw exception if session is expired
-        val result = fetchAuthSession.execute().userPoolTokensResult
-        result.error.throwIf<SessionExpiredException>()
-
         return AuthUser(
             state.signedInData.userId,
             state.signedInData.username

aws-auth-cognito/src/test/java/com/amplifyframework/auth/cognito/usecases/GetCurrentUserUseCaseTest.kt

@@ -18,7 +18,6 @@ package com.amplifyframework.auth.cognito.usecases
 import com.amplifyframework.auth.cognito.AuthStateMachine
 import com.amplifyframework.auth.cognito.mockSignedInData
 import com.amplifyframework.auth.exceptions.InvalidStateException
-import com.amplifyframework.auth.exceptions.SessionExpiredException
 import com.amplifyframework.auth.exceptions.SignedOutException
 import com.amplifyframework.statemachine.codegen.states.AuthenticationState
 import io.kotest.assertions.throwables.shouldThrow
@@ -29,10 +28,6 @@ import kotlinx.coroutines.test.runTest
 import org.junit.Test
 
 class GetCurrentUserUseCaseTest {
-    private val fetchAuthSession: FetchAuthSessionUseCase = mockk {
-        coEvery { execute().userPoolTokensResult.error } returns null
-    }
-
     private val stateMachine: AuthStateMachine = mockk {
         coEvery { getCurrentState().authNState } returns AuthenticationState.SignedIn(
             signedInData = mockSignedInData(username = "username", userId = "sub"),
@@ -41,7 +36,6 @@ class GetCurrentUserUseCaseTest {
     }
 
     private val useCase = GetCurrentUserUseCase(
-        fetchAuthSession = fetchAuthSession,
         stateMachine = stateMachine
     )
 
@@ -68,13 +62,4 @@ class GetCurrentUserUseCaseTest {
             useCase.execute()
         }
     }
-
-    @Test
-    fun `throws exception for expired session`() = runTest {
-        val error = SessionExpiredException()
-        coEvery { fetchAuthSession.execute().userPoolTokensResult.error } returns error
-        shouldThrow<SessionExpiredException> {
-            useCase.execute()
-        } shouldBe error
-    }
 }