Skip to content

Commit 3888ac5

Browse files
committed
test: adjust telemetry to mirror CI reality
1 parent d286070 commit 3888ac5

File tree

1 file changed

+2
-13
lines changed

1 file changed

+2
-13
lines changed

packages/amazonq/test/e2e/lsp/lspInstaller.test.ts

Lines changed: 2 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -123,22 +123,17 @@ describe('AmazonQLSPInstaller', () => {
123123
assert.ok(semver.satisfies(fallback.version, supportedLspServerVersions))
124124

125125
// Exclude version numbers so that this test doesn't have to be updated on each update.
126+
// Fetching manifest fails locally, but passes in CI. Therefore, this test will fail locally.
126127
assertTelemetry('languageServer_setup', [
127128
/* First Try Telemetry
128-
getManifest: remote fails, then cache succeeds.
129+
getManifest: remote succeeds
129130
getServer: cache fails then remote succeeds.
130131
validate: succeeds.
131132
*/
132133
{
133134
id: 'AmazonQ',
134135
manifestLocation: 'remote',
135136
languageServerSetupStage: 'getManifest',
136-
result: 'Failed',
137-
},
138-
{
139-
id: 'AmazonQ',
140-
manifestLocation: 'cache',
141-
languageServerSetupStage: 'getManifest',
142137
result: 'Succeeded',
143138
},
144139
{
@@ -168,12 +163,6 @@ describe('AmazonQLSPInstaller', () => {
168163
id: 'AmazonQ',
169164
manifestLocation: 'remote',
170165
languageServerSetupStage: 'getManifest',
171-
result: 'Failed',
172-
},
173-
{
174-
id: 'AmazonQ',
175-
manifestLocation: 'cache',
176-
languageServerSetupStage: 'getManifest',
177166
result: 'Succeeded',
178167
},
179168
{

0 commit comments

Comments
 (0)