test: fix failing null check test

Author: nbbeeken

test/functional/crud_api.test.js

@@ -1,7 +1,7 @@
 'use strict';
 const test = require('./shared').assert;
 const { expect } = require('chai');
-const { ReturnDocument } = require('../../src');
+const { ReturnDocument, ObjectId } = require('../../src');
 const setupDatabase = require('./shared').setupDatabase;
 
 // instanceof cannot be use reliably to detect the new models in js due to scoping and new
@@ -14,6 +14,27 @@ describe('CRUD API', function () {
     return setupDatabase(this.configuration);
   });
 
+  it('should correctly execute findOne method using crud api', async function () {
+    const configuration = this.configuration;
+    const client = configuration.newClient();
+    await client.connect();
+    const db = client.db(configuration.db);
+    const collection = db.collection('t');
+
+    await collection.insertOne({ findOneTest: 1 });
+
+    const findOneResult = await collection.findOne({ findOneTest: 1 });
+
+    expect(findOneResult).to.have.property('findOneTest', 1);
+    expect(findOneResult).to.have.property('_id').that.is.instanceOf(ObjectId);
+
+    const findNoneResult = await collection.findOne({ findOneTest: 2 });
+    expect(findNoneResult).to.be.null;
+
+    await collection.drop();
+    await client.close();
+  });
+
   it('should correctly execute find method using crud api', {
     // Add a tag that our runner can trigger on
     // in this case we are setting that node needs to be higher than 0.10.X to run

test/functional/sessions.test.js

@@ -301,8 +301,8 @@ describe('Sessions', function () {
         controlSession = client.startSession();
 
         // set up sessions with two sets of cluster times
-        expect(await collection.findOne({}, { session: controlSession })).to.be.undefined;
-        expect(await collection.findOne({}, { session: testSession })).to.be.undefined;
+        expect(await collection.findOne({}, { session: controlSession })).to.be.null;
+        expect(await collection.findOne({}, { session: testSession })).to.be.null;
         await collection.insertOne({ apple: 'green' });
         expect(await collection.findOne({}, { session: otherSession }))
           .property('apple')