test: remove .only and add back constructor logic

Author: Grace Chong

src/objectid.ts

@@ -57,9 +57,7 @@ export class ObjectId {
         if ('toHexString' in id && typeof id.toHexString === 'function') {
           this[kId] = Buffer.from(id.toHexString(), 'hex');
         } else {
-          throw new TypeError(
-            'Object passed in does not have toHexString() function'
-          );
+          this[kId] = typeof id.id === 'string' ? Buffer.from(id.id) : id.id;
         }
       } else if (id == null || typeof id === 'number') {
         // The most common use case (blank id, new objectId instance)

test/node/object_id_tests.js

@@ -5,7 +5,7 @@ const BSON = require('../register-bson');
 const util = require('util');
 const ObjectId = BSON.ObjectId;
 
-describe.only('ObjectId', function () {
+describe('ObjectId', function () {
   /**
    * @ignore
    */
@@ -48,16 +48,6 @@ describe.only('ObjectId', function () {
     expect(() => new ObjectId(objectIdLike)).to.throw(TypeError);
   });
 
-  it('should throw error if object without toHexString function is passed in', function () {
-    var tmp = new ObjectId();
-    var objectIdLike = {
-      id: tmp.id
-    };
-    objectIdLike.id.toHexString = null;
-
-    expect(() => new ObjectId(objectIdLike).toHexString()).to.throw(TypeError);
-  });
-
   it('should correctly create ObjectId from number', function () {
     expect(() => new ObjectId(42)).to.not.throw();
     expect(() => new ObjectId(0x2a)).to.not.throw();