broken test + lint

Author: baileympearson

drivers-evergreen-tools

@@ -6,7 +6,7 @@ import * as CONSTANTS from './constants';
 import { AggregationCursor } from './cursor/aggregation_cursor';
 import { ListCollectionsCursor } from './cursor/list_collections_cursor';
 import { RunCommandCursor, type RunCursorCommandOptions } from './cursor/run_command_cursor';
-import { MONGODB_ERROR_CODES, MongoInvalidArgumentError, MongoServerError } from './error';
+import { MongoInvalidArgumentError } from './error';
 import type { MongoClient, PkFactory } from './mongo_client';
 import type { Abortable, TODO_NODE_3286 } from './mongo_types';
 import type { AggregateOptions } from './operations/aggregate';

src/db.ts

@@ -543,7 +543,12 @@ export type {
 export type { InsertManyResult, InsertOneOptions, InsertOneResult } from './operations/insert';
 export type { CollectionInfo, ListCollectionsOptions } from './operations/list_collections';
 export type { ListDatabasesOptions, ListDatabasesResult } from './operations/list_databases';
-export type { AbstractOperation, Hint, OperationOptions } from './operations/operation';
+export type {
+  AbstractOperation,
+  Hint,
+  ModernizedOperation,
+  OperationOptions
+} from './operations/operation';
 export type { ProfilingLevelOptions } from './operations/profiling_level';
 export type { RemoveUserOptions } from './operations/remove_user';
 export type { RenameOptions } from './operations/rename';

src/index.ts

@@ -185,6 +185,7 @@ export abstract class CommandOperation<T> extends AbstractOperation<T> {
   }
 }
 
+/** @internal */
 export abstract class ModernizedCommandOperation<T> extends ModernizedOperation<T> {
   override options: CommandOperationOptions;
   readConcern?: ReadConcern;

src/operations/command.ts

@@ -1,6 +1,8 @@
+import { MongoServerError } from '..';
 import type { Document } from '../bson';
 import { CursorTimeoutContext } from '../cursor/abstract_cursor';
 import type { Db } from '../db';
+import { MONGODB_ERROR_CODES } from '../error';
 import type { Server } from '../sdam/server';
 import type { ClientSession } from '../sessions';
 import { TimeoutContext } from '../timeout';

src/operations/drop.ts

@@ -131,6 +131,7 @@ export abstract class AbstractOperation<TResult = any> {
   }
 }
 
+/** @internal */
 export abstract class ModernizedOperation<TResult> extends AbstractOperation<TResult> {
   abstract SERVER_COMMAND_RESPONSE_TYPE: typeof MongoDBResponse;
 

src/operations/operation.ts

@@ -162,23 +162,25 @@ describe('Handshake Prose Tests', function () {
     let stubCalled = false;
     beforeEach(() => {
       // Mock the server response in a way that saslSupportedMechs array in the hello command response contains an arbitrary string.
-      sinon.stub(Connection.prototype, 'command').callsFake(async function (ns, cmd, options) {
-        // @ts-expect-error: sinon will place wrappedMethod there
-        const command = Connection.prototype.command.wrappedMethod.bind(this);
-        if (cmd.hello || cmd[LEGACY_HELLO_COMMAND]) {
-          return stub();
-        }
-        return command(ns, cmd, options);
-
-        async function stub() {
-          stubCalled = true;
-          const response = await command(ns, cmd, options);
-          return {
-            ...response,
-            saslSupportedMechs: [...(response.saslSupportedMechs ?? []), 'random string']
-          };
-        }
-      });
+      sinon
+        .stub(Connection.prototype, 'command')
+        .callsFake(async function (ns, cmd, options, responseType) {
+          // @ts-expect-error: sinon will place wrappedMethod there
+          const command = Connection.prototype.command.wrappedMethod.bind(this);
+          if (cmd.hello || cmd[LEGACY_HELLO_COMMAND]) {
+            return stub();
+          }
+          return command(ns, cmd, options, responseType);
+
+          async function stub() {
+            stubCalled = true;
+            const response = await command(ns, cmd, options, responseType);
+            return {
+              ...response,
+              saslSupportedMechs: [...(response.saslSupportedMechs ?? []), 'random string']
+            };
+          }
+        });
     });
 
     afterEach(() => sinon.restore());