refactor(codewhisperer): remove duplicate code

[Will-ShaoHua]

Problem

testing

VSC

Auto trigger

vscode.auto.trigger.mov

Manual trigger

vscode.manual.trigger.mov

C9

Auto trigger

c9.auto.trigger.mov

Manual trigger

c9.manual.trigger.mov

Solution

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[Will-ShaoHua]

will wait until post re:invent

[justinmk3]

is this ready?

[Will-ShaoHua]

is this ready?

@justinmk3 yea it's ready but should we wait until next year given it's not a P0 change, considering it's late Dec and we might have lower coverage for CX's issues in the following 1-2 wks?

[Will-ShaoHua]

@justinmk3 this one is ready to go thanks!

think the failing test is not relevant to the change here?

  1) Auth
    at MultiReporters.done (d:\a\aws-toolkit-vscode\aws-toolkit-vscode\node_modules\mocha-multi-reporters\lib\MultiReporters.js:124:16)
       Shared ini files
    at done (d:\a\aws-toolkit-vscode\aws-toolkit-vscode\node_modules\mocha\lib\mocha.js:1007:16)
         does not cache if the credentials file changes:
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

Extension host test runner error Error: 1 tests failed.
      AssertionError [ERR_ASSERTION]: Expected values to be strictly deep-equal:
    at d:\a\aws-toolkit-vscode\aws-toolkit-vscode\src\test\testRunner.ts:90:28
+ actual - expected
    at MultiReporters.done (d:\a\aws-toolkit-vscode\aws-toolkit-vscode\node_modules\mocha-multi-reporters\lib\MultiReporters.js:124:16)

    at done (d:\a\aws-toolkit-vscode\aws-toolkit-vscode\node_modules\mocha\lib\mocha.js:1007:16)
  {
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
+   accessKeyId: 'x',
+   secretAccessKey: 'x',
-   accessKeyId: 'y',
-   secretAccessKey: 'y',
    sessionToken: undefined
  }
      + expected - actual

       {
      -  "accessKeyId": "x"
      -  "secretAccessKey": "x"
      +  "accessKeyId": "y"
      +  "secretAccessKey": "y"
         "sessionToken": [undefined]
       }