finish migrating Q

Author: Hweinstock

packages/amazonq/src/inlineChat/controller/inlineChatController.ts

@@ -156,7 +156,7 @@ export class InlineChatController {
     }
 
     private async reset() {
-        this.listeners.forEach((listener) => listener.dispose())
+        await Promise.all(this.listeners.map((l) => l.dispose()))
         this.listeners = []
 
         this.task = undefined

packages/amazonq/src/inlineChat/output/computeDiff.ts

@@ -2,7 +2,7 @@
  * Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
  * SPDX-License-Identifier: Apache-2.0
  */
-import { type LinesOptions, diffLines, Change } from 'diff'
+import { type LinesOptions, diffLines } from 'diff'
 import * as vscode from 'vscode'
 import { InlineTask, TextDiff } from '../controller/inlineTask'
 
@@ -24,8 +24,7 @@ export function computeDiff(response: string, inlineTask: InlineTask, isPartialD
 
     const textDiff: TextDiff[] = []
     let startLine = selectedRange.start.line
-
-    diffs.forEach((part: Change) => {
+    for (const part of diffs) {
         const count = part.count ?? 0
         if (part.removed) {
             if (part.value !== '\n') {
@@ -49,7 +48,7 @@ export function computeDiff(response: string, inlineTask: InlineTask, isPartialD
             }
         }
         startLine += count
-    })
+    }
     inlineTask.diff = textDiff
     return textDiff
 }

packages/amazonq/test/unit/codewhisperer/service/keyStrokeHandler.test.ts

@@ -212,7 +212,7 @@ describe('keyStrokeHandler', function () {
             ['function suggestedByIntelliSense():', DocumentChangedSource.Unknown],
         ]
 
-        cases.forEach((tuple) => {
+        for (const tuple of cases) {
             const input = tuple[0]
             const expected = tuple[1]
             it(`test input ${input} should return ${expected}`, function () {
@@ -221,7 +221,7 @@ describe('keyStrokeHandler', function () {
                 ).checkChangeSource()
                 assert.strictEqual(actual, expected)
             })
-        })
+        }
 
         function createFakeDocumentChangeEvent(str: string): ReadonlyArray<vscode.TextDocumentContentChangeEvent> {
             return [

packages/amazonq/test/unit/codewhisperer/util/crossFileContextUtil.test.ts

@@ -241,11 +241,12 @@ describe('crossFileContextUtil', function () {
             const actual = await crossFile.getCrossFileCandidates(editor)
 
             assert.ok(actual.length === 5)
-            actual.forEach((actualFile, index) => {
+            for (const item of actual.map((v: string, i: number) => [i, v] as const)) {
+                const [index, actualFile] = item
                 const expectedFile = path.join(tempFolder, expectedFilePaths[index])
                 assert.strictEqual(normalize(expectedFile), normalize(actualFile))
                 assert.ok(areEqual(tempFolder, actualFile, expectedFile))
-            })
+            }
         })
     })
 
@@ -264,7 +265,7 @@ describe('crossFileContextUtil', function () {
             await closeAllEditors()
         })
 
-        fileExtLists.forEach((fileExt) => {
+        for (const fileExt of fileExtLists) {
             it('should be empty if userGroup is control', async function () {
                 const editor = await toTextEditor('content-1', `file-1.${fileExt}`, tempFolder)
                 await toTextEditor('content-2', `file-2.${fileExt}`, tempFolder, { preview: false })
@@ -277,7 +278,7 @@ describe('crossFileContextUtil', function () {
 
                 assert.ok(actual && actual.supplementalContextItems.length === 0)
             })
-        })
+        }
     })
 
     describe.skip('partial support - crossfile group', function () {
@@ -294,8 +295,7 @@ describe('crossFileContextUtil', function () {
         afterEach(async function () {
             await closeAllEditors()
         })
-
-        fileExtLists.forEach((fileExt) => {
+        for (const fileExt of fileExtLists) {
             it('should be non empty if usergroup is Crossfile', async function () {
                 const editor = await toTextEditor('content-1', `file-1.${fileExt}`, tempFolder)
                 await toTextEditor('content-2', `file-2.${fileExt}`, tempFolder, { preview: false })
@@ -308,7 +308,7 @@ describe('crossFileContextUtil', function () {
 
                 assert.ok(actual && actual.supplementalContextItems.length !== 0)
             })
-        })
+        }
     })
 
     describe('full support', function () {
@@ -329,7 +329,7 @@ describe('crossFileContextUtil', function () {
             await closeAllEditors()
         })
 
-        fileExtLists.forEach((fileExt) => {
+        for (const fileExt of fileExtLists) {
             it(`supplemental context for file ${fileExt} should be non empty`, async function () {
                 sinon.stub(FeatureConfigProvider.instance, 'getProjectContextGroup').returns('control')
                 sinon
@@ -353,7 +353,7 @@ describe('crossFileContextUtil', function () {
 
                 assert.ok(actual && actual.supplementalContextItems.length !== 0)
             })
-        })
+        }
     })
 
     describe('splitFileToChunks', function () {

packages/amazonq/test/unit/codewhisperer/util/editorContext.test.ts

@@ -94,13 +94,12 @@ describe('editorContext', function () {
                 ['typescript', 'ts'],
                 ['c', 'c'],
             ])
-
-            languageToExtension.forEach((extension, language) => {
+            for (const language of languageToExtension.keys()) {
                 const editor = createMockTextEditor('', 'test.ipynb', language, 1, 17)
                 const actual = EditorContext.getFileRelativePath(editor)
-                const expected = 'test.' + extension
+                const expected = 'test.' + languageToExtension.get(language)
                 assert.strictEqual(actual, expected)
-            })
+            }
         })
 
         it('Should return relative path', async function () {

packages/amazonq/test/unit/codewhisperer/util/runtimeLanguageContext.test.ts

@@ -51,16 +51,15 @@ describe('runtimeLanguageContext', function () {
         beforeEach(async function () {
             await resetCodeWhispererGlobalVariables()
         })
-
-        cases.forEach((tuple) => {
+        for (const tuple of cases) {
             const languageId = tuple[0]
             const expected = tuple[1]
 
             it(`should ${expected ? '' : 'not'} support ${languageId}`, function () {
                 const actual = languageContext.isLanguageSupported(languageId)
                 assert.strictEqual(actual, expected)
             })
-        })
+        }
 
         describe('test isLanguageSupported with document as the argument', function () {
             const cases: [string, boolean][] = [
@@ -105,7 +104,7 @@ describe('runtimeLanguageContext', function () {
                 ['helloFoo.foo', false],
             ]
 
-            cases.forEach((tuple) => {
+            for (const tuple of cases) {
                 const fileName = tuple[0]
                 const expected = tuple[1]
 
@@ -114,7 +113,7 @@ describe('runtimeLanguageContext', function () {
                     const actual = languageContext.isLanguageSupported(doc)
                     assert.strictEqual(actual, expected)
                 })
-            })
+            }
         })
     })
 
@@ -148,14 +147,14 @@ describe('runtimeLanguageContext', function () {
             [undefined, 'plaintext'],
         ]
 
-        cases.forEach((tuple) => {
+        for (const tuple of cases) {
             const vscLanguageId = tuple[0]
             const expectedCwsprLanguageId = tuple[1]
             it(`given vscLanguage ${vscLanguageId} should return ${expectedCwsprLanguageId}`, function () {
                 const result = runtimeLanguageContext.getLanguageContext(vscLanguageId)
                 assert.strictEqual(result.language as string, expectedCwsprLanguageId)
             })
-        })
+        }
     })
 
     describe('normalizeLanguage', function () {

packages/amazonq/test/unit/codewhisperer/util/securityScanLanguageContext.test.ts

@@ -51,15 +51,15 @@ describe('securityScanLanguageContext', function () {
             await resetCodeWhispererGlobalVariables()
         })
 
-        cases.forEach((tuple) => {
+        for (const tuple of cases) {
             const languageId = tuple[0]
             const expected = tuple[1]
 
             it(`should ${expected ? '' : 'not'} support ${languageId}`, function () {
                 const actual = languageContext.isLanguageSupported(languageId)
                 assert.strictEqual(actual, expected)
             })
-        })
+        }
     })
 
     describe('normalizeLanguage', function () {

packages/amazonq/test/web/testRunner.ts

@@ -35,6 +35,7 @@ function setupMocha() {
 
 function gatherTestFiles() {
     // Bundles all files in the current directory matching `*.test`
+    // eslint-disable-next-line aws-toolkits/no-foreach
     const importAll = (r: __WebpackModuleApi.RequireContext) => r.keys().forEach(r)
     importAll(require.context('.', true, /\.test$/))
 }