[SDK-1938] Update the user/isAuthenticated state after getting a token

[adamjmcgrath]

Description

Everytime we call getAccessToken*, update the user from the spa client - since it may have been updated.

With the caveat that, if you're relying on the user as a dependant key of some memoized operation, the reference to the user might change when the user properties haven't. eg

// DON'T
useEffect(() => {
  (async () => {
    setProfile(await fetch(`/profile/{user.sub}`));
  })()
}, [user]) // <= ref might change

// DO
useEffect(() => {
  (async () => {
    setProfile(await fetch(`/profile/{user.sub}`));
  })()
}, [user.sub]) // <= primitive won't change

References

Fixes #109

Testing

Given I login to the auth0-react playground
And I update the user's claims
When I click getAccessTokenSilently
Then the user's claims should be updated

Checklist

• I have added documentation for new/changed functionality in this PR or in auth0.com/docs
• All active GitHub checks for tests, formatting, and security are passing
• The correct base branch is being used, if not master

[frederikprijck]

Is there any reason we went for a shared wrappedGetToken utils to a duplicated yet identical (besides the actual token call) code?

[adamjmcgrath]

The method signatures are different too (the old wrappedGetToken didn't account for PopupConfigOptions ) that felt like a good enough reason to split them into 2

[frederikprijck]

Wow so that means the second argument in case of popup was never passed along? That is a nice catch.

[adamjmcgrath]

yeah - @ygist caught that one