test: remaining tests

Author: durran

test/integration/change-streams/change_stream.test.ts

@@ -1255,7 +1255,6 @@ describe('Change Streams', function () {
 
   UnifiedTestSuiteBuilder.describe('document shapes')
     .runOnRequirement({
-      minServerVersion: '4.0.0',
       auth: true,
       // Running on replicaset because other topologies are finiky with the cluster-wide events
       // Dropping and renaming and creating collections in order to achieve a clean slate isn't worth the goal of these tests

test/integration/change-streams/change_streams.prose.test.ts

@@ -332,11 +332,10 @@ describe('Change Stream prose tests', function () {
     }
 
     // 11. For a ChangeStream under these conditions:
-    //   Running against a server >=4.0.7.
     //   The batch is empty or has been iterated to the last document.
     // Expected result:
     //   getResumeToken must return the postBatchResumeToken from the current command response.
-    describe('for emptied batch on server >= 4.0.7', function () {
+    describe('for emptied batch on server', function () {
       it('must return the postBatchResumeToken from the current command response', function () {
         const manager = new MockServerManager(this.configuration, {
           aggregate: (function* () {

test/integration/connections-survive-step-down/connections_survive_step_down.prose.test.ts

@@ -140,7 +140,7 @@ describe('Connections Survive Primary Step Down - prose', function () {
 
   context(
     'Shutdown in progress - Reset Connection Pool',
-    { requires: { mongodb: '>=4.0', topology: ['replicaset'] } },
+    { requires: { topology: ['replicaset'] } },
     () => {
       // This test should be run on all server versions >= 4.0.
 
@@ -168,7 +168,7 @@ describe('Connections Survive Primary Step Down - prose', function () {
 
   context(
     'Interrupted at shutdown - Reset Connection Pool',
-    { requires: { mongodb: '>=4.0', topology: ['replicaset'] } },
+    { requires: { topology: ['replicaset'] } },
     () => {
       // This test should be run on all server versions >= 4.0.
 

test/integration/crud/aggregation.test.ts

@@ -72,26 +72,23 @@ describe('Aggregation', function () {
     });
   });
 
-  it('should correctly execute db.aggregate() with $currentOp', {
-    metadata: { requires: { mongodb: '>=4.0.0' } },
-    test: function (done) {
-      const client = this.configuration.newClient({ w: 1 }, { maxPoolSize: 1 });
+  it('should correctly execute db.aggregate() with $currentOp', function (done) {
+    const client = this.configuration.newClient({ w: 1 }, { maxPoolSize: 1 });
 
-      const db = client.db('admin');
-      const cursor = db.aggregate([{ $currentOp: { localOps: true } }]);
+    const db = client.db('admin');
+    const cursor = db.aggregate([{ $currentOp: { localOps: true } }]);
 
-      cursor.toArray((err, result) => {
-        expect(err).to.not.exist;
+    cursor.toArray((err, result) => {
+      expect(err).to.not.exist;
 
-        const aggregateOperation = result.filter(op => op.command && op.command.aggregate)[0];
-        expect(aggregateOperation.command.aggregate).to.equal(1);
-        expect(aggregateOperation.command.pipeline).to.eql([{ $currentOp: { localOps: true } }]);
-        expect(aggregateOperation.command.cursor).to.deep.equal({});
-        expect(aggregateOperation.command['$db']).to.equal('admin');
+      const aggregateOperation = result.filter(op => op.command && op.command.aggregate)[0];
+      expect(aggregateOperation.command.aggregate).to.equal(1);
+      expect(aggregateOperation.command.pipeline).to.eql([{ $currentOp: { localOps: true } }]);
+      expect(aggregateOperation.command.cursor).to.deep.equal({});
+      expect(aggregateOperation.command['$db']).to.equal('admin');
 
-        client.close(done);
-      });
-    }
+      client.close(done);
+    });
   });
 
   it('should fail when executing simple aggregation pipeline using arguments not an array', function (done) {

test/integration/crud/crud.prose.test.ts

@@ -32,7 +32,7 @@ describe('CRUD Prose Spec Tests', () => {
   });
 
   // TODO(NODE-3888): Implement this test
-  it.skip('1. WriteConcernError.details exposes writeConcernError.errInfo', {
+  it.skip('1. WriteConcernError.details exposes writeConcernError.errInfo', async function () {
     /**
      * Test that writeConcernError.errInfo in a command response is propagated as WriteConcernError.details (or equivalent) in the driver.
      * Using a 4.0+ server, set the following failpoint:
@@ -62,10 +62,7 @@ describe('CRUD Prose Spec Tests', () => {
      * Then, perform an insert operation and assert that a WriteConcernError occurs and that
      * its details property is both accessible and matches the errInfo object from the failpoint.
      */
-    metadata: { requires: { mongodb: '>=4.0.0' } },
-    async test() {
-      throw new Error('This test is not implemented!');
-    }
+    throw new Error('This test is not implemented!');
   }).skipReason = 'TODO(NODE-3888): Implement this test';
 
   describe('2. WriteError.details exposes writeErrors[].errInfo', () => {

test/integration/crud/find_and_modify.test.ts

@@ -347,27 +347,15 @@ describe('Collection (#findOneAnd...)', function () {
           await client?.close();
         });
 
-        it(
-          'the aggregation pipeline updates the matching document',
-          {
-            requires: {
-              mongodb: '>4.0'
-            }
-          },
-          async function () {
-            const {
-              value: { _id, ...document }
-            } = await collection.findOneAndUpdate(
-              { a: 1 },
-              [{ $set: { a: { $add: [1, '$a'] } } }],
-              {
-                includeResultMetadata: true,
-                returnDocument: 'after'
-              }
-            );
-            expect(document).to.deep.equal({ a: 2, b: 1 });
-          }
-        );
+        it('the aggregation pipeline updates the matching document', async function () {
+          const {
+            value: { _id, ...document }
+          } = await collection.findOneAndUpdate({ a: 1 }, [{ $set: { a: { $add: [1, '$a'] } } }], {
+            includeResultMetadata: true,
+            returnDocument: 'after'
+          });
+          expect(document).to.deep.equal({ a: 2, b: 1 });
+        });
       });
 
       context('when passing includeResultMetadata: false', function () {
@@ -385,22 +373,14 @@ describe('Collection (#findOneAnd...)', function () {
           await client?.close();
         });
 
-        it(
-          'the aggregation pipeline updates the matching document',
-          {
-            requires: {
-              mongodb: '>4.0'
-            }
-          },
-          async function () {
-            const { _id, ...document } = await collection.findOneAndUpdate(
-              { a: 1 },
-              [{ $set: { a: { $add: [1, '$a'] } } }],
-              { returnDocument: 'after' }
-            );
-            expect(document).to.deep.equal({ a: 2, b: 1 });
-          }
-        );
+        it('the aggregation pipeline updates the matching document', async function () {
+          const { _id, ...document } = await collection.findOneAndUpdate(
+            { a: 1 },
+            [{ $set: { a: { $add: [1, '$a'] } } }],
+            { returnDocument: 'after' }
+          );
+          expect(document).to.deep.equal({ a: 2, b: 1 });
+        });
       });
     });
   });

test/integration/enumerate_databases.test.ts

@@ -6,7 +6,6 @@ import { TestBuilder, UnifiedTestSuiteBuilder } from '../tools/utils';
 
 const metadata: MongoDBMetadataUI = {
   requires: {
-    mongodb: '>=4.0.6',
     auth: 'enabled',
 
     // 'TODO: NODE-3891 - fix tests broken when AUTH enabled'

test/integration/index_management.test.ts

@@ -358,18 +358,10 @@ describe('Indexes', function () {
         });
     });
 
-    it(
-      'should return false',
-      {
-        requires: {
-          mongodb: '>4.0'
-        }
-      },
-      async function () {
-        const result = await collection.dropIndexes();
-        expect(result).to.equal(false);
-      }
-    );
+    it('should return false', async function () {
+      const result = await collection.dropIndexes();
+      expect(result).to.equal(false);
+    });
   });
 
   context('indexExists', function () {

test/integration/node-specific/bson-options/promote_buffers.test.js

@@ -150,8 +150,7 @@ describe('Promote Buffers', function () {
     // in this case we are setting that node needs to be higher than 0.10.X to run
     metadata: {
       requires: {
-        topology: ['single', 'replicaset', 'sharded', 'ssl', 'heap', 'wiredtiger'],
-        mongodb: '>=2.4.0'
+        topology: ['single', 'replicaset', 'sharded', 'ssl', 'heap', 'wiredtiger']
       }
     },
 

test/integration/server-selection/operation_count.test.ts

@@ -11,15 +11,13 @@ import { type FailPoint } from '../../tools/utils';
 
 const testMetadata: MongoDBMetadataUI = {
   requires: {
-    topology: 'single',
-    mongodb: '>=4.0.0'
+    topology: 'single'
   }
 };
 
 const loadBalancedTestMetadata: MongoDBMetadataUI = {
   requires: {
-    topology: 'load-balanced',
-    mongodb: '>=4.0.0'
+    topology: 'load-balanced'
   }
 };