fix(tests): sso token cache race condition #3001

Problem: Unit tests would sporadically fail due
to cache not being cleared when expected.
Certain methods called async clear() without
waiting, and the checks in the test executed before
the clear() could complete.

Solution: Add 'await' to the impacted clear() calls.

Signed-off-by: Nikolas Komonen <[email protected]>

Author: nkomonen-amazon

src/credentials/sso/ssoAccessTokenProvider.ts

@@ -104,7 +104,7 @@ export class SsoAccessTokenProvider {
             return await this.authorize(registration)
         } catch (err) {
             if (err instanceof SSOOIDCServiceException && isClientFault(err)) {
-                this.cache.registration.clear(cacheKey)
+                await this.cache.registration.clear(cacheKey)
             }
 
             throw err
@@ -119,7 +119,7 @@ export class SsoAccessTokenProvider {
             return this.formatToken(response, registration)
         } catch (err) {
             if (err instanceof SSOOIDCServiceException && isClientFault(err)) {
-                this.cache.token.clear(this.tokenCacheKey)
+                await this.cache.token.clear(this.tokenCacheKey)
             }
 
             throw err