test: remaining tests

Author: durran

test/integration/crud/insert.test.ts

@@ -569,39 +569,6 @@ describe('crud - insert', function () {
       expect(err).to.be.instanceOf(MongoServerError);
     });
 
-    // TODO(7219): remove as it's redundant (custom ids are used in multiple other tests: "with UUID", "Date object as _id")
-    // it('shouldCorrectlyInsertDocWithCustomId', {
-    //   // Add a tag that our runner can trigger on
-    //   // 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'] }
-    //   },
-    //
-    //   test: function (done) {
-    //     const configuration = this.configuration;
-    //     const client = configuration.newClient(configuration.writeConcernMax(), { maxPoolSize: 1 });
-    //     client.connect(function (err, client) {
-    //       const db = client.db(configuration.db);
-    //       const collection = db.collection('shouldCorrectlyInsertDocWithCustomId');
-    //       // Insert the update
-    //       collection.insert(
-    //         { _id: 0, test: 'hello' },
-    //         configuration.writeConcernMax(),
-    //         function (err, result) {
-    //           expect(err).to.not.exist;
-    //           test.ok(result);
-    //
-    //           collection.findOne({ _id: 0 }, function (err, item) {
-    //             test.equal(0, item._id);
-    //             test.equal('hello', item.test);
-    //             client.close(done);
-    //           });
-    //         }
-    //       );
-    //     });
-    //   }
-    // });
-
     it('should correctly perform upsert against new document and existing one', async function () {
       const configuration = this.configuration;
       const db = client.db(configuration.db);

test/integration/node-specific/bson-options/promote_values.test.ts

@@ -31,65 +31,6 @@ describe('Promote Values', function () {
     await client.close();
   });
 
-  // TODO(NODE-7192): remove as it duplicates "should correctly honor promoteValues when creating an instance using Db"
-  // it('should correctly honor promoteValues when creating an instance using MongoClient', {
-  //   // Add a tag that our runner can trigger on
-  //   // 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'] }
-  //   },
-  //
-  //   test: async function () {
-  //     const configuration = this.configuration;
-  //     const client = configuration.newClient({}, { promoteValues: false });
-  //     await client.connect();
-  //     const db = client.db(configuration.db);
-  //
-  //     await db.collection('shouldCorrectlyHonorPromoteValues').insertOne({
-  //       doc: Long.fromNumber(10),
-  //       int: 10,
-  //       double: 2.2222,
-  //       array: [[Long.fromNumber(10)]]
-  //     });
-  //
-  //     const doc = await db.collection('shouldCorrectlyHonorPromoteValues').findOne();
-  //     expect(Long.fromNumber(10)).deep.equals(doc.doc);
-  //     expect(new Int32(10)).deep.equals(doc.int);
-  //     expect(new Double(2.2222)).deep.equals(doc.double);
-  //
-  //     await client.close();
-  //   }
-  // });
-
-  // TODO(NODE-7192): remove as it duplicates "should correctly honor promoteValues when creating an instance using Db"
-  // it('should correctly honor promoteValues at cursor level', {
-  //   // Add a tag that our runner can trigger on
-  //   // 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'] }
-  //   },
-  //
-  //   test: async function () {
-  //     const configuration = this.configuration;
-  //     const client = configuration.newClient({}, { promoteValues: false });
-  //     await client.connect();
-  //     const db = client.db(configuration.db);
-  //     await db.collection('shouldCorrectlyHonorPromoteValues').insertOne({
-  //       doc: Long.fromNumber(10),
-  //       int: 10,
-  //       double: 2.2222,
-  //       array: [[Long.fromNumber(10)]]
-  //     });
-  //
-  //     const doc = await db.collection('shouldCorrectlyHonorPromoteValues').find().next();
-  //     expect(doc.doc).to.deep.equal(Long.fromNumber(10));
-  //     expect(doc.int).to.deep.equal(new Int32(10));
-  //     expect(doc.double).to.deep.equal(new Double(2.2222));
-  //
-  //     await client.close();
-  //   }
-  // });
-
   it('should correctly honor promoteValues at cursor find level', async function () {
     const configuration = this.configuration;
     const client = configuration.newClient();

test/integration/node-specific/db.test.ts

@@ -66,28 +66,6 @@ describe('Db', function () {
     await client.close();
   });
 
-  // TODO(NODE-7192): remove test as it doesn't test anything
-  // it.skip('shouldCorrectlyThrowWhenTryingToReOpenConnection', {
-  //   metadata: {
-  //     requires: { topology: ['single', 'replicaset', 'sharded'] }
-  //   },
-  //
-  //   test: function (done) {
-  //     var configuration = this.configuration;
-  //     var client = configuration.newClient(configuration.writeConcernMax(), { maxPoolSize: 1 });
-  //     client.connect(err => {
-  //       expect(err).to.not.exist;
-  //
-  //       try {
-  //         client.connect(function () {});
-  //         test.ok(false);
-  //       } catch {
-  //         client.close(done);
-  //       }
-  //     });
-  //   }
-  // });
-
   it('should not cut collection name when it is the same as the database', async function () {
     const configuration = this.configuration;
     const client = configuration.newClient(configuration.writeConcernMax(), { maxPoolSize: 1 });