remove comment from commands pre-4.4

Author: baileympearson

src/operations/command.ts

@@ -179,10 +179,6 @@ export abstract class CommandOperation<T> extends AbstractOperation<T> {
       cmd.maxTimeMS = options.maxTimeMS;
     }
 
-    if (typeof options.comment === 'string') {
-      cmd.comment = options.comment;
-    }
-
     if (this.hasAspect(Aspect.EXPLAINABLE) && this.explain) {
       if (serverWireVersion < 6 && cmd.aggregate) {
         // Prior to 3.6, with aggregate, verbosity is ignored, and we must pass in "explain: true"

test/integration/enumerate_collections.test.ts

@@ -25,7 +25,8 @@ const testSuite = new UnifiedTestSuiteBuilder('listCollections with comment opti
           {
             commandStartedEvent: {
               command: {
-                listCollections: 1
+                listCollections: 1,
+                comment: { $$exists: false }
               }
             }
           }

test/integration/enumerate_databases.test.ts

@@ -156,7 +156,8 @@ const testSuite = new UnifiedTestSuiteBuilder('listDatabases with comment option
       .operation({
         name: 'listDatabases',
         arguments: {
-          filter: {}
+          filter: {},
+          comment: 'string value'
         },
         object: 'client0'
       })
@@ -166,7 +167,8 @@ const testSuite = new UnifiedTestSuiteBuilder('listDatabases with comment option
           {
             commandStartedEvent: {
               command: {
-                listDatabases: 1
+                listDatabases: 1,
+                comment: { $$exists: false }
               }
             }
           }

test/integration/enumerate_indexes.test.ts

@@ -24,7 +24,10 @@ const testSuite = new UnifiedTestSuiteBuilder('listIndexes with comment option')
         events: [
           {
             commandStartedEvent: {
-              command: {}
+              command: {
+                listIndexes: 'coll0',
+                comment: { $$exists: false }
+              }
             }
           }
         ]

test/tools/unified-spec-runner/match.ts

@@ -253,7 +253,12 @@ export function specialCheck(
   } else if (isExistsOperator(expected)) {
     // $$exists
     const actualExists = actual !== undefined && actual !== null;
-    expect((expected.$$exists && actualExists) || (!expected.$$exists && !actualExists)).to.be.true;
+    expect(
+      (expected.$$exists && actualExists) || (!expected.$$exists && !actualExists),
+      `expected value at ${path.join('')} to not exist, but received ${JSON.stringify(
+        actual
+      )} instead`
+    ).to.be.true;
   } else {
     expect.fail(`Unknown special operator: ${JSON.stringify(expected)}`);
   }