cleanup

mongodb-js · Jul 3, 2024 · d3c2cf4 · d3c2cf4
1 parent bab80f2
commit d3c2cf4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 16 deletions.
diff --git a/packages/compass-e2e-tests/tests/collection-bulk-update.test.ts b/packages/compass-e2e-tests/tests/collection-bulk-update.test.ts
@@ -117,8 +117,8 @@ describe('Bulk Update', () => {
     // Check the telemetry
     const executedEvent = await telemetryEntry('Bulk Update Executed');
 
-    expect(openedEvent.connection_id).to.exist;
-    delete openedEvent.connection_id; // connection_id varies
+    expect(executedEvent.connection_id).to.exist;
+    delete executedEvent.connection_id; // connection_id varies
 
     expect(executedEvent).to.deep.equal({
       isUpdatePreviewSupported: true,
@@ -165,10 +165,8 @@ describe('Bulk Update', () => {
     // Check the telemetry
     const favoritedEvent = await telemetryEntry('Bulk Update Favorited');
 
-    // this id is always different, because the connection is not a saved one
-    // so we just check it exists for simplicity
     expect(favoritedEvent.connection_id).to.exist;
-    delete favoritedEvent.connection_id;
+    delete favoritedEvent.connection_id; // connection_id varies
 
     expect(favoritedEvent).to.deep.equal({
       isUpdatePreviewSupported: true,

diff --git a/packages/compass-e2e-tests/tests/collection-documents-tab.test.ts b/packages/compass-e2e-tests/tests/collection-documents-tab.test.ts
@@ -156,10 +156,8 @@ describe('Collection documents tab', function () {
     // Check the telemetry
     const queryExecutedEvent = await telemetryEntry('Query Executed');
 
-    // this id is always different, because the connection is not a saved one
-    // so we just check it exists for simplicity
     expect(queryExecutedEvent.connection_id).to.exist;
-    delete queryExecutedEvent.connection_id;
+    delete queryExecutedEvent.connection_id; // connection_id varies
 
     expect(queryExecutedEvent).to.deep.equal({
       changed_maxtimems: false,
@@ -197,10 +195,8 @@ describe('Collection documents tab', function () {
     // Check the telemetry
     const queryExecutedEvent = await telemetryEntry('Query Executed');
 
-    // this id is always different, because the connection is not a saved one
-    // so we just check it exists for simplicity
     expect(queryExecutedEvent.connection_id).to.exist;
-    delete queryExecutedEvent.connection_id;
+    delete queryExecutedEvent.connection_id; // connection_id varies
 
     expect(queryExecutedEvent).to.deep.equal({
       changed_maxtimems: false,

diff --git a/packages/compass-import-export/src/modules/import.ts b/packages/compass-import-export/src/modules/import.ts
@@ -386,11 +386,6 @@ export const startImport = (): ImportThunkAction<Promise<void>> => {
       errorLogWriteStream?.close();
     }
 
-    console.log(
-      '*** TRACK ***',
-      connectionRepository.getConnectionInfoById(connectionId)
-    );
-
     track(
       'Import Completed',
       {