Skip to content

Commit 801c9b7

Browse files
fix: disable saved Connections Loaded event (#585)
1 parent 896ac37 commit 801c9b7

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

src/connectionController.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,9 @@ export default class ConnectionController {
222222
this.eventEmitter.emit(DataServiceEventTypes.CONNECTIONS_DID_CHANGE);
223223
}
224224

225-
this._telemetryService.trackSavedConnectionsLoaded({
225+
// TODO: re-enable with fewer 'Saved Connections Loaded' events
226+
// https://jira.mongodb.org/browse/VSCODE-462
227+
/* this._telemetryService.trackSavedConnectionsLoaded({
226228
saved_connections: globalAndWorkspaceConnections.length,
227229
loaded_connections: loadedConnections.length,
228230
connections_with_secrets_in_keytar: loadedConnections.filter(
@@ -234,7 +236,7 @@ export default class ConnectionController {
234236
connection.secretStorageLocation ===
235237
SecretStorageLocation.SecretStorage
236238
).length,
237-
});
239+
}); */
238240

239241
if (connectionsThatDidNotMigrate.length) {
240242
log.error(

src/test/suite/connectionController.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1762,7 +1762,7 @@ suite('Connection Controller Test Suite', function () {
17621762
assert.strictEqual(trackStub.notCalled, true);
17631763
});
17641764

1765-
test('should track SAVED_CONNECTIONS_LOADED event on load of saved connections', async () => {
1765+
test.skip('should track SAVED_CONNECTIONS_LOADED event on load of saved connections', async () => {
17661766
testSandbox.replace(testStorageController, 'get', (key, storage) => {
17671767
if (
17681768
storage === StorageLocation.WORKSPACE ||

src/test/suite/telemetry/telemetryService.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -717,7 +717,7 @@ suite('Telemetry Controller Test Suite', () => {
717717
});
718718
});
719719

720-
test('track saved connections loaded', () => {
720+
test.skip('track saved connections loaded', () => {
721721
testTelemetryService.trackSavedConnectionsLoaded({
722722
saved_connections: 3,
723723
loaded_connections: 3,

0 commit comments

Comments
 (0)