Merge pull request #5401 from hayemaxi/tech-debt

fix tech debt tests

Author: hayemaxi

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

@@ -223,20 +223,6 @@ export class AuthUtil {
         return this.secondaryAuth.useNewConnection(conn)
     }
 
-    /**
-     * HACK: Use the connection, but then mark it as expired.
-     * We currently only need this to handle an edge case with the transition
-     * from the old to new standalone extension. This should eventually be removed.
-     */
-    public async useConnectionButExpire(conn: Connection) {
-        await this.secondaryAuth.useNewConnection(conn)
-        if (conn.type !== 'sso') {
-            return
-        }
-        await this.auth.expireConnection(conn)
-        await this.notifyReauthenticate()
-    }
-
     public static get instance() {
         if (this.#instance !== undefined) {
             return this.#instance

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

@@ -40,13 +40,9 @@ describe('tech debt', function () {
         )
     })
 
-    it('remove missing Amazon Q scopes edge case handling', async function () {
-        fixByDate('2024-07-30', 'Remove the edge case code from the commit that this test is a part of.')
-    })
-
     it('remove separate sessions login edge cases', async function () {
         // src/auth/auth.ts:SessionSeparationPrompt
         // forgetConnection() function and calls
-        fixByDate('2024-07-30', 'Remove the edge case code from the commit that this test is a part of.')
+        fixByDate('2024-08-30', 'Remove the edge case code from the commit that this test is a part of.')
     })
 })