Skip to content

Commit 93ce28d

Browse files
committed
Fix unit tests
1 parent c16a728 commit 93ce28d

File tree

3 files changed

+5
-9
lines changed

3 files changed

+5
-9
lines changed

packages/amazonq/test/unit/codewhisperer/region/regionProfileManager.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ describe('RegionProfileManager', async function () {
3737
}
3838
}
3939

40-
beforeEach(async function () {
40+
beforeEach(function () {
4141
regionProfileManager = new RegionProfileManager(AuthUtil.instance)
4242
})
4343

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import {
1515
RecommendationHandler,
1616
CodeWhispererCodeCoverageTracker,
1717
supplementalContextUtil,
18+
AuthUtil,
1819
} from 'aws-core-vscode/codewhisperer'
1920
import {
2021
assertTelemetryCurried,
@@ -24,8 +25,6 @@ import {
2425
} from 'aws-core-vscode/test'
2526
// import * as supplementalContextUtil from 'aws-core-vscode/codewhisperer'
2627

27-
const enterpriseSsoStartUrl = 'https://enterprise.awsapps.com/start'
28-
2928
describe('recommendationHandler', function () {
3029
const config: ConfigurationEntry = {
3130
isShowMethodsEnabled: true,
@@ -142,7 +141,7 @@ describe('recommendationHandler', function () {
142141
codewhispererLineNumber: 1,
143142
codewhispererCursorOffset: 38,
144143
codewhispererLanguage: 'python',
145-
credentialStartUrl: enterpriseSsoStartUrl,
144+
credentialStartUrl: AuthUtil.instance.connection?.startUrl,
146145
codewhispererSupplementalContextIsUtg: false,
147146
codewhispererSupplementalContextTimeout: false,
148147
codewhispererSupplementalContextLatency: 0,

packages/amazonq/test/unit/codewhisperer/tracker/codewhispererTracker.test.ts

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
import assert from 'assert'
77
import * as sinon from 'sinon'
8-
import { assertTelemetryCurried, createTestAuthUtil } from 'aws-core-vscode/test'
8+
import { assertTelemetryCurried } from 'aws-core-vscode/test'
99
import { AuthUtil, CodeWhispererTracker } from 'aws-core-vscode/codewhisperer'
1010
import { resetCodeWhispererGlobalVariables, createAcceptedSuggestionEntry } from 'aws-core-vscode/test'
1111
import { globals } from 'aws-core-vscode/shared'
@@ -82,9 +82,6 @@ describe('codewhispererTracker', function () {
8282

8383
describe('emitTelemetryOnSuggestion', function () {
8484
it('Should call recordCodewhispererUserModification with suggestion event', async function () {
85-
await createTestAuthUtil()
86-
const testStartUrl = 'testStartUrl'
87-
sinon.stub(AuthUtil.instance.connection!, 'startUrl').value(testStartUrl)
8885
const suggestion = createAcceptedSuggestionEntry()
8986
const assertTelemetry = assertTelemetryCurried('codewhisperer_userModification')
9087
await CodeWhispererTracker.getTracker().emitTelemetryOnSuggestion(suggestion)
@@ -96,7 +93,7 @@ describe('codewhispererTracker', function () {
9693
codewhispererModificationPercentage: 1,
9794
codewhispererCompletionType: 'Line',
9895
codewhispererLanguage: 'java',
99-
credentialStartUrl: testStartUrl,
96+
credentialStartUrl: AuthUtil.instance.connection?.startUrl,
10097
codewhispererCharactersAccepted: suggestion.originalString.length,
10198
codewhispererCharactersModified: 0,
10299
})

0 commit comments

Comments
 (0)