test(amazonq): add more tests for /transform#6183
Conversation
| authType = 'awsId' | ||
| } | ||
| return authType | ||
| } |
There was a problem hiding this comment.
This above function was previously in transformApiHandler.ts, just moving it here so it's in a shared utils file
| assert.strictEqual(usesHttpsAndExpiresAfter60Seconds, true) | ||
| }) | ||
|
|
||
| it('WHEN zipCode THEN ZIP contains all expected files and no unexpected files', async function () { |
There was a problem hiding this comment.
Moved this test to the file above with the other zipCode tests
| @@ -35,10 +34,6 @@ describe('transformByQ', async function () { | |||
| await fs.writeFile(tempFilePath, 'sample content for the test file') | |||
| transformByQState.setProjectPath(tempDir) | |||
| const zipCodeResult = await zipCode({ | |||
| dependenciesFolder: { | |||
There was a problem hiding this comment.
Not used for these tests
packages/core/src/testE2E/amazonQTransform/transformByQ.test.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
How often will these tests be run & do they have to be manually triggered?
There was a problem hiding this comment.
/runIntegrationTests to trigger them
There was a problem hiding this comment.
They are also run automatically after merging.
|
|
| export { default as MessengerUtils } from './chat/controller/messenger/messengerUtils' | ||
| export { GumbyController } from './chat/controller/controller' | ||
| export { TabsStorage } from '../amazonq/webview/ui/storages/tabsStorage' | ||
| export * as startTransformByQ from '../../src/codewhisperer/commands/startTransformByQ' |
There was a problem hiding this comment.
I'm unable to run the tests in packages/amazonq/test/e2e/amazonq/transformByQ.test.ts by importing these things from the core module directly where they're defined, as I get an import error when doing that. So instead I have to export them here and then (in the test file, line 11) import them from here.
|
/runIntegrationTests |
|
/runIntegrationTests |
|
/runIntegrationTests |
|
/runIntegrationTests |
|
/runIntegrationTests |
|
/runIntegrationTests |
|
/runIntegrationTests |
3 participants
Problem
/transformcould benefit from more test coverage.Solution
Add tests.
feature/xbranches will not be squash-merged at release time.License: I confirm that my contribution is made under the terms of the Apache 2.0 license.