Skip to content

Commit e3c044c

Browse files
author
David Hasani
committed
update comment
1 parent d7fd12a commit e3c044c

File tree

2 files changed

+1
-9
lines changed

2 files changed

+1
-9
lines changed

packages/core/src/test/codewhisperer/commands/transformByQ.test.ts

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -166,14 +166,6 @@ describe('transformByQ', function () {
166166
transformByQState.setJobDefaults() // reset error messages to undefined
167167
})
168168

169-
it('WHEN finalizeTransformationJob on successful job THEN error not thrown and error message fields are not set', async function () {
170-
await assert.doesNotReject(async () => {
171-
await finalizeTransformationJob('COMPLETED')
172-
})
173-
assert.strictEqual(transformByQState.getJobFailureErrorChatMessage(), undefined)
174-
assert.strictEqual(transformByQState.getJobFailureErrorNotification(), undefined)
175-
})
176-
177169
it('WHEN polling completed job THEN returns status as completed', async function () {
178170
const mockJobResponse = {
179171
$response: {

packages/core/src/testE2E/amazonQTransform/transformByQ.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ describe('transformByQ', async function () {
7272
await fs.writeFile(tempFilePath, pomXmlContents)
7373
})
7474

75-
// TODO: this test 1) is skipped in GitHub CI due to no valid connection (see line 60 above) and
75+
// TODO: this test 1) is skipped in GitHub CI due to no valid connection (see line 62 above) and
7676
// 2) even locally, fails due to the max test duration being set to 30s (this test takes ~5m)
7777
// Once both of the above are resolved, this test will pass
7878
// You can manually override the 30s limit (in setupUtil.ts) to confirm that the test passes locally

0 commit comments

Comments
 (0)