Skip to content

Commit 21ef0b1

Browse files
authored
fix(amazonq): remove flaky unit test (aws#8116)
## Problem ## Solution --- - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent ac6649e commit 21ef0b1

File tree

1 file changed

+28
-30
lines changed

1 file changed

+28
-30
lines changed

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

Lines changed: 28 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,8 @@ import {
1111
session,
1212
AuthUtil,
1313
DefaultCodeWhispererClient,
14-
RecommendationsList,
1514
ConfigurationEntry,
1615
RecommendationHandler,
17-
CodeWhispererCodeCoverageTracker,
1816
supplementalContextUtil,
1917
} from 'aws-core-vscode/codewhisperer'
2018
import {
@@ -54,35 +52,35 @@ describe('recommendationHandler', function () {
5452
sinon.restore()
5553
})
5654

57-
it('should assign correct recommendations given input', async function () {
58-
assert.strictEqual(CodeWhispererCodeCoverageTracker.instances.size, 0)
59-
assert.strictEqual(
60-
CodeWhispererCodeCoverageTracker.getTracker(mockEditor.document.languageId)?.serviceInvocationCount,
61-
0
62-
)
55+
// it('should assign correct recommendations given input', async function () {
56+
// assert.strictEqual(CodeWhispererCodeCoverageTracker.instances.size, 0)
57+
// assert.strictEqual(
58+
// CodeWhispererCodeCoverageTracker.getTracker(mockEditor.document.languageId)?.serviceInvocationCount,
59+
// 0
60+
// )
6361

64-
const mockServerResult = {
65-
recommendations: [{ content: "print('Hello World!')" }, { content: '' }],
66-
$response: {
67-
requestId: 'test_request',
68-
httpResponse: {
69-
headers: {
70-
'x-amzn-sessionid': 'test_request',
71-
},
72-
},
73-
},
74-
}
75-
const handler = new RecommendationHandler()
76-
sinon.stub(handler, 'getServerResponse').resolves(mockServerResult)
77-
await handler.getRecommendations(mockClient, mockEditor, 'AutoTrigger', config, 'Enter', false)
78-
const actual = session.recommendations
79-
const expected: RecommendationsList = [{ content: "print('Hello World!')" }, { content: '' }]
80-
assert.deepStrictEqual(actual, expected)
81-
assert.strictEqual(
82-
CodeWhispererCodeCoverageTracker.getTracker(mockEditor.document.languageId)?.serviceInvocationCount,
83-
1
84-
)
85-
})
62+
// const mockServerResult = {
63+
// recommendations: [{ content: "print('Hello World!')" }, { content: '' }],
64+
// $response: {
65+
// requestId: 'test_request',
66+
// httpResponse: {
67+
// headers: {
68+
// 'x-amzn-sessionid': 'test_request',
69+
// },
70+
// },
71+
// },
72+
// }
73+
// const handler = new RecommendationHandler()
74+
// sinon.stub(handler, 'getServerResponse').resolves(mockServerResult)
75+
// await handler.getRecommendations(mockClient, mockEditor, 'AutoTrigger', config, 'Enter', false)
76+
// const actual = session.recommendations
77+
// const expected: RecommendationsList = [{ content: "print('Hello World!')" }, { content: '' }]
78+
// assert.deepStrictEqual(actual, expected)
79+
// assert.strictEqual(
80+
// CodeWhispererCodeCoverageTracker.getTracker(mockEditor.document.languageId)?.serviceInvocationCount,
81+
// 1
82+
// )
83+
// })
8684

8785
it('should assign request id correctly', async function () {
8886
const mockServerResult = {

0 commit comments

Comments
 (0)