half comments addressed

Author: aditi-khare-mongoDB

src/mongo_client.ts

@@ -657,7 +657,8 @@ export class MongoClient extends TypedEventEmitter<MongoClientEvents> implements
     }
   }
 
-  async _close(force = false): Promise<void> {
+  /* @internal */
+  private async _close(force = false): Promise<void> {
     // There's no way to set hasBeenClosed back to false
     Object.defineProperty(this.s, 'hasBeenClosed', {
       value: true,

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

@@ -737,6 +737,26 @@ describe('class MongoClient', function () {
           ).to.not.throw;
         });
       });
+
+      it('when connect rejects lock is released regardless', async function () {
+        await client.connect();
+        expect(client.topology?.isConnected()).to.be.true;
+
+        const closeStub = sinon.stub(client, 'close');
+        closeStub.onFirstCall().rejects(new Error('cannot close'));
+
+        // first call rejected to simulate a close failure
+        const error = await client.close().catch(error => error);
+        expect(error).to.match(/cannot close/);
+
+        expect(client.topology?.isConnected()).to.be.true;
+        closeStub.restore();
+
+        // second call should close
+        await client.close();
+
+        expect(client.topology).to.be.undefined;
+      });
     });
   });