fix: make inspection result of BSON types evaluable (#416)

Make sure that evaluating the inspection result of a BSON type
returns a result that can be evaluated to give an object equivalent
to the original.

NODE-2947

Author: addaleax

src/binary.ts

@@ -263,7 +263,7 @@ export class Binary {
 
   inspect(): string {
     const asBuffer = this.value(true);
-    return `Binary("${asBuffer.toString('hex')}", ${this.sub_type})`;
+    return `new Binary(Buffer.from("${asBuffer.toString('hex')}", "hex"), ${this.sub_type})`;
   }
 }
 

src/code.ts

@@ -50,7 +50,9 @@ export class Code {
 
   inspect(): string {
     const codeJson = this.toJSON();
-    return `Code("${codeJson.code}"${codeJson.scope ? `, ${JSON.stringify(codeJson.scope)}` : ''})`;
+    return `new Code("${codeJson.code}"${
+      codeJson.scope ? `, ${JSON.stringify(codeJson.scope)}` : ''
+    })`;
   }
 }
 

src/db_ref.ts

@@ -108,7 +108,9 @@ export class DBRef {
     // NOTE: if OID is an ObjectId class it will just print the oid string.
     const oid =
       this.oid === undefined || this.oid.toString === undefined ? this.oid : this.oid.toString();
-    return `DBRef("${this.namespace}", "${oid}"${this.db ? `, "${this.db}"` : ''})`;
+    return `new DBRef("${this.namespace}", new ObjectId("${oid}")${
+      this.db ? `, "${this.db}"` : ''
+    })`;
   }
 }
 

src/decimal128.ts

@@ -794,7 +794,7 @@ export class Decimal128 {
   }
 
   inspect(): string {
-    return `Decimal128("${this.toString()}")`;
+    return `Decimal128.fromString("${this.toString()}")`;
   }
 }
 

src/double.ts

@@ -78,7 +78,7 @@ export class Double {
 
   inspect(): string {
     const eJSON = this.toExtendedJSON() as DoubleExtended;
-    return `Double(${eJSON.$numberDouble})`;
+    return `new Double(${eJSON.$numberDouble})`;
   }
 }
 

src/int_32.ts

@@ -57,7 +57,7 @@ export class Int32 {
   }
 
   inspect(): string {
-    return `Int32(${this.valueOf()})`;
+    return `new Int32(${this.valueOf()})`;
   }
 }
 

src/long.ts

@@ -992,7 +992,7 @@ export class Long {
   }
 
   inspect(): string {
-    return `Long("${this.toString()}")`;
+    return `new Long("${this.toString()}")`;
   }
 }
 

src/max_key.ts

@@ -26,7 +26,7 @@ export class MaxKey {
   }
 
   inspect(): string {
-    return 'MaxKey()';
+    return 'new MaxKey()';
   }
 }
 

src/min_key.ts

@@ -26,7 +26,7 @@ export class MinKey {
   }
 
   inspect(): string {
-    return 'MinKey()';
+    return 'new MinKey()';
   }
 }
 

src/objectid.ts

@@ -350,7 +350,7 @@ export class ObjectId {
   }
 
   inspect(): string {
-    return `ObjectId("${this.toHexString()}")`;
+    return `new ObjectId("${this.toHexString()}")`;
   }
 }