eslint and build fixes

Author: W-A-James

src/cmap/connect.ts

@@ -44,7 +44,7 @@ export async function connect(options: ConnectionOptions): Promise<Connection> {
     await performInitialHandshake(connection, options);
     return connection;
   } catch (error) {
-    connection?.destroy({ force: false });
+    connection?.destroy();
     throw error;
   }
 }

src/mongo_client.ts

@@ -552,7 +552,7 @@ export class MongoClient extends TypedEventEmitter<MongoClientEvents> {
       try {
         await promisify(callback => this.topology?.connect(options, callback))();
       } catch (error) {
-        this.topology?.close({ force: true });
+        this.topology?.close();
         throw error;
       }
     };

src/sdam/monitor.ts

@@ -214,7 +214,7 @@ function resetMonitorState(monitor: Monitor) {
 
   monitor[kCancellationToken].emit('cancel');
 
-  monitor.connection?.destroy({ force: true });
+  monitor.connection?.destroy();
   monitor.connection = null;
 }
 
@@ -247,7 +247,7 @@ function checkServer(monitor: Monitor, callback: Callback<Document | null>) {
   );
 
   function onHeartbeatFailed(err: Error) {
-    monitor.connection?.destroy({ force: true });
+    monitor.connection?.destroy();
     monitor.connection = null;
 
     monitor.emitAndLogHeartbeat(
@@ -366,13 +366,13 @@ function checkServer(monitor: Monitor, callback: Callback<Document | null>) {
       await performInitialHandshake(connection, monitor.connectOptions);
       return connection;
     } catch (error) {
-      connection.destroy({ force: false });
+      connection.destroy();
       throw error;
     }
   })().then(
     connection => {
       if (isInCloseState(monitor)) {
-        connection.destroy({ force: true });
+        connection.destroy();
         return;
       }
 
@@ -479,7 +479,7 @@ export class RTTPinger {
     this.closed = true;
     clearTimeout(this[kMonitorId]);
 
-    this.connection?.destroy({ force: true });
+    this.connection?.destroy();
     this.connection = undefined;
   }
 }
@@ -495,7 +495,7 @@ function measureRoundTripTime(rttPinger: RTTPinger, options: RTTPingerOptions) {
 
   function measureAndReschedule(conn?: Connection) {
     if (rttPinger.closed) {
-      conn?.destroy({ force: true });
+      conn?.destroy();
       return;
     }
 
@@ -529,7 +529,7 @@ function measureRoundTripTime(rttPinger: RTTPinger, options: RTTPingerOptions) {
   connection.command(ns('admin.$cmd'), { [commandName]: 1 }, undefined).then(
     () => measureAndReschedule(),
     () => {
-      rttPinger.connection?.destroy({ force: true });
+      rttPinger.connection?.destroy();
       rttPinger.connection = undefined;
       rttPinger[kRoundTripTime] = 0;
       return;

test/unit/error.test.ts

@@ -361,7 +361,7 @@ describe('MongoErrors', () => {
         // cleanup the server before calling done
         const cleanup = err => {
           topology.close();
-          done();
+          done(err);
         };
 
         if (err) {