Skip to content

Commit 6223893

Browse files
committed
fix(tests): remove Q 3 -> 5 scope migration tech debt
Cleans up remainder of 9525cdf
1 parent a71d1e9 commit 6223893

File tree

2 files changed

+0
-18
lines changed

2 files changed

+0
-18
lines changed

packages/core/src/codewhisperer/util/authUtil.ts

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -223,20 +223,6 @@ export class AuthUtil {
223223
return this.secondaryAuth.useNewConnection(conn)
224224
}
225225

226-
/**
227-
* HACK: Use the connection, but then mark it as expired.
228-
* We currently only need this to handle an edge case with the transition
229-
* from the old to new standalone extension. This should eventually be removed.
230-
*/
231-
public async useConnectionButExpire(conn: Connection) {
232-
await this.secondaryAuth.useNewConnection(conn)
233-
if (conn.type !== 'sso') {
234-
return
235-
}
236-
await this.auth.expireConnection(conn)
237-
await this.notifyReauthenticate()
238-
}
239-
240226
public static get instance() {
241227
if (this.#instance !== undefined) {
242228
return this.#instance

packages/core/src/test/techdebt.test.ts

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,6 @@ describe('tech debt', function () {
4040
)
4141
})
4242

43-
it('remove missing Amazon Q scopes edge case handling', async function () {
44-
fixByDate('2024-07-30', 'Remove the edge case code from the commit that this test is a part of.')
45-
})
46-
4743
it('remove separate sessions login edge cases', async function () {
4844
// src/auth/auth.ts:SessionSeparationPrompt
4945
// forgetConnection() function and calls

0 commit comments

Comments
 (0)