test updates

Author: W-A-James

test/integration/crud/find.test.js

@@ -36,9 +36,7 @@ describe('Find', function () {
       const collection = db.collection('test_find_simple');
       const docs = [{ a: 2 }, { b: 3 }];
 
-      console.log('before insert');
       await collection.insert(docs, configuration.writeConcernMax());
-      console.log('after insert');
 
       const insertedDocs = await collection.find().toArray();
       expect(insertedDocs).to.have.length(2);

test/unit/operations/find.test.ts

@@ -41,7 +41,7 @@ describe('FindOperation', function () {
 
       it('should build basic find command with filter', async () => {
         const findOperation = new FindOperation(undefined, namespace, filter);
-        const stub = sinon.stub(server, 'command');
+        const stub = sinon.stub(server, 'command').resolves({});
         await findOperation.execute(server, undefined);
         expect(stub).to.have.been.calledOnceWith(namespace, {
           find: namespace.collection,
@@ -54,7 +54,7 @@ describe('FindOperation', function () {
           oplogReplay: true
         };
         const findOperation = new FindOperation(undefined, namespace, {}, options);
-        const stub = sinon.stub(server, 'command');
+        const stub = sinon.stub(server, 'command').resolves({});
         await findOperation.execute(server, undefined);
         expect(stub).to.have.been.calledOnceWith(
           namespace,

test/unit/operations/get_more.test.ts

@@ -54,7 +54,7 @@ describe('GetMoreOperation', function () {
         );
         const opts = { ...options, documentsReturnedIn: 'nextBatch', returnFieldSelector: null };
         const operation = new GetMoreOperation(namespace, cursorId, server, opts);
-        const stub = sinon.stub(server, 'command').returns(Promise.resolve({}));
+        const stub = sinon.stub(server, 'command').resolves({});
 
         const expectedGetMoreCommand = {
           getMore: cursorId,
@@ -104,7 +104,7 @@ describe('GetMoreOperation', function () {
 
       it('should build basic getMore command with cursorId and collection', async () => {
         const getMoreOperation = new GetMoreOperation(namespace, cursorId, server, {});
-        const stub = sinon.stub(server, 'command');
+        const stub = sinon.stub(server, 'command').resolves({});
         await getMoreOperation.execute(server, undefined);
         expect(stub).to.have.been.calledOnceWith(namespace, {
           getMore: cursorId,
@@ -117,7 +117,7 @@ describe('GetMoreOperation', function () {
           batchSize: 234
         };
         const getMoreOperation = new GetMoreOperation(namespace, cursorId, server, options);
-        const stub = sinon.stub(server, 'command');
+        const stub = sinon.stub(server, 'command').resolves({});
         await getMoreOperation.execute(server, undefined);
         expect(stub).to.have.been.calledOnceWith(
           namespace,
@@ -130,7 +130,7 @@ describe('GetMoreOperation', function () {
           maxAwaitTimeMS: 234
         };
         const getMoreOperation = new GetMoreOperation(namespace, cursorId, server, options);
-        const stub = sinon.stub(server, 'command');
+        const stub = sinon.stub(server, 'command').resolves({});
         await getMoreOperation.execute(server, undefined);
         expect(stub).to.have.been.calledOnceWith(
           namespace,
@@ -188,7 +188,7 @@ describe('GetMoreOperation', function () {
               maxWireVersion: serverVersion
             };
             const operation = new GetMoreOperation(namespace, cursorId, server, optionsWithComment);
-            const stub = sinon.stub(server, 'command');
+            const stub = sinon.stub(server, 'command').resolves({});
             await operation.execute(server, undefined);
             expect(stub).to.have.been.calledOnceWith(namespace, getMore);
           });
@@ -202,7 +202,7 @@ describe('GetMoreOperation', function () {
         new ServerDescription('a:1'),
         {} as any
       );
-      sinon.stub(server, 'command');
+      sinon.stub(server, 'command').resolves({});
 
       it('should throw if the cursorId is undefined', async () => {
         const getMoreOperation = new GetMoreOperation(

test/unit/operations/kill_cursors.test.ts

@@ -91,7 +91,7 @@ describe('class KillCursorsOperation', () => {
         server,
         options
       ) as any;
-      const stub = sinon.stub(server, 'command').returns(Promise.resolve({}));
+      const stub = sinon.stub(server, 'command').resolves({});
       await killCursorsOperation.execute(server, undefined);
       expect(stub).to.have.been.calledOnceWith(namespace, {
         killCursors: namespace.collection,