fix tests

Author: W-A-James

test/unit/assorted/polling_srv_records_for_mongos_discovery.prose.test.ts

@@ -115,7 +115,8 @@ describe('Polling Srv Records for Mongos Discovery', () => {
 
     afterEach(function (done) {
       if (context.topology) {
-        context.topology.close({}, done);
+        context.topology.close();
+        done();
       } else {
         done();
       }

test/unit/assorted/server_selection_spec_helper.js

@@ -110,7 +110,8 @@ function executeServerSelectionTest(testDefinition, testDone) {
     });
 
   function done(err) {
-    topology.close({}, e => testDone(e || err));
+    topology.close();
+    testDone(err);
   }
 
   topology.connect(err => {

test/unit/error.test.ts

@@ -359,7 +359,10 @@ describe('MongoErrors', () => {
 
       makeAndConnectReplSet((err, topology) => {
         // cleanup the server before calling done
-        const cleanup = err => topology.close({ force: true }, err2 => done(err || err2));
+        const cleanup = err => {
+          topology.close();
+          done();
+        };
 
         if (err) {
           return cleanup(err);
@@ -402,7 +405,10 @@ describe('MongoErrors', () => {
 
       makeAndConnectReplSet((err, topology) => {
         // cleanup the server before calling done
-        const cleanup = err => topology.close({}, err2 => done(err || err2));
+        const cleanup = err => {
+          topology.close();
+          done(err);
+        };
 
         if (err) {
           return cleanup(err);

test/unit/sdam/monitor.test.ts

@@ -93,7 +93,8 @@ describe('monitoring', function () {
         const serverDescription = Array.from(topology.description.servers.values())[0];
         expect(serverDescription).property('roundTripTime').to.be.greaterThan(0);
 
-        topology.close({}, done as any);
+        topology.close();
+        done();
       }, 500);
     });
   }).skipReason = 'TODO(NODE-3819): Unskip flaky tests';

test/unit/sdam/topology.test.js

@@ -26,7 +26,7 @@ describe('Topology (unit)', function () {
     }
 
     if (topology) {
-      topology.close({});
+      topology.close();
     }
   });
 
@@ -104,7 +104,8 @@ describe('Topology (unit)', function () {
             .then(expect.fail, err => {
               expect(err).to.exist;
               expect(err).to.match(/timed out/);
-              topology.close({}, done);
+              topology.close();
+              done();
             });
         });
       });
@@ -251,7 +252,8 @@ describe('Topology (unit)', function () {
             expect(err).to.exist;
             expect(err).to.eql(serverDescription.error);
             expect(poolCleared).to.be.false;
-            topology.close({}, done);
+            topology.close();
+            done();
           });
         });
       });
@@ -401,17 +403,12 @@ describe('Topology (unit)', function () {
 
         it('should clean up listeners on close', function (done) {
           topology.s.state = 'connected'; // fake state to test clean up logic
-          topology.close({}, e => {
-            const srvPollerListeners = topology.s.srvPoller.listeners(
-              SrvPoller.SRV_RECORD_DISCOVERY
-            );
-            expect(srvPollerListeners).to.have.lengthOf(0);
-            const topologyChangeListeners = topology.listeners(
-              Topology.TOPOLOGY_DESCRIPTION_CHANGED
-            );
-            expect(topologyChangeListeners).to.have.lengthOf(0);
-            done(e);
-          });
+          topology.close();
+          const srvPollerListeners = topology.s.srvPoller.listeners(SrvPoller.SRV_RECORD_DISCOVERY);
+          expect(srvPollerListeners).to.have.lengthOf(0);
+          const topologyChangeListeners = topology.listeners(Topology.TOPOLOGY_DESCRIPTION_CHANGED);
+          expect(topologyChangeListeners).to.have.lengthOf(0);
+          done();
         });
       });
 
@@ -481,7 +478,8 @@ describe('Topology (unit)', function () {
           // occurs `requestCheck` will be called for an immediate check.
           expect(requestCheck).property('callCount').to.equal(1);
 
-          topology.close({}, done);
+          topology.close();
+          done();
         });
       });
     });
@@ -493,12 +491,11 @@ describe('Topology (unit)', function () {
         this.emit('connect');
       });
 
-      topology.close({}, () => {
-        topology.selectServer(ReadPreference.primary, { serverSelectionTimeoutMS: 2000 }, err => {
-          expect(err).to.exist;
-          expect(err).to.match(/Topology is closed/);
-          done();
-        });
+      topology.close();
+      topology.selectServer(ReadPreference.primary, { serverSelectionTimeoutMS: 2000 }, err => {
+        expect(err).to.exist;
+        expect(err).to.match(/Topology is closed/);
+        done();
       });
     });