Skip to content

Commit 1547c7f

Browse files
committed
move await before telemetry assertion to avoid race cond
1 parent 1fb2337 commit 1547c7f

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

packages/core/src/test/credentials/sso/ssoAccessTokenProvider.test.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -320,19 +320,24 @@ describe('SsoAccessTokenProvider', function () {
320320
const resp = sut
321321
.createToken()
322322
.then(() => assert.fail('Should not resolve'))
323-
.catch((e) => assert.ok(e instanceof ToolkitError))
323+
.catch((e) => {
324+
assert.ok(
325+
e instanceof ToolkitError &&
326+
e.message === 'Timed-out waiting for browser login flow to complete'
327+
)
328+
})
324329

325330
const progress = await getTestWindow().waitForMessage(/login page opened/i)
326331
await clock.tickAsync(750)
327332
assert.ok(progress.visible)
328333
await clock.tickAsync(750)
329334
assert.ok(!progress.visible)
335+
await resp
330336
assertTelemetry('aws_loginWithBrowser', {
331337
result: 'Failed',
332338
isReAuth: undefined,
333339
credentialStartUrl: startUrl,
334340
})
335-
await resp
336341
})
337342

338343
/**

0 commit comments

Comments
 (0)