pr comments

Author: aditi-khare-mongoDB

src/binary.ts

@@ -278,7 +278,7 @@ export class Binary {
     if (!data) {
       throw new BSONTypeError(`Unexpected Binary Extended JSON format ${JSON.stringify(doc)}`);
     }
-    return type === 4 ? new UUID(data) : new Binary(data, type);
+    return type === BSON_BINARY_SUBTYPE_UUID_NEW ? new UUID(data) : new Binary(data, type);
   }
 
   /** @internal */

src/parser/deserializer.ts

@@ -445,12 +445,10 @@ function deserializeObject(
 
         if (promoteBuffers && promoteValues) {
           value = _buffer;
+        } else if (subType === constants.BSON_BINARY_SUBTYPE_UUID_NEW) {
+          value = new Binary(buffer.slice(index, index + binarySize), subType).toUUID();
         } else {
-          value = new Binary(_buffer, subType);
           value = new Binary(buffer.slice(index, index + binarySize), subType);
-          if (subType === constants.BSON_BINARY_SUBTYPE_UUID_NEW) {
-            value = value.toUUID();
-          }
         }
       }
 

test/node/extended_json_tests.js

@@ -779,7 +779,7 @@ Converting circular structure to EJSON:
       const stringifiedUUID = EJSON.stringify({ uuid: exampleUUID });
       const parsedUUID = EJSON.parse(stringifiedUUID);
       const expectedResult = {
-        uuid: new UUID('878dac1201cc4830b271cbc8518e63ad')
+        uuid: new UUID('878dac12-01cc-4830-b271-cbc8518e63ad')
       };
       expect(parsedUUID).to.deep.equal(expectedResult);
     });

test/node/uuid_tests.js

@@ -198,7 +198,7 @@ describe('UUID', () => {
       const serializedUUID = BSON.serialize({ uuid: exampleUUID });
       const deserializedUUID = BSON.deserialize(serializedUUID);
       const expectedResult = {
-        uuid: new UUID('878dac1201cc4830b271cbc8518e63ad')
+        uuid: new UUID('878dac12-01cc-4830-b271-cbc8518e63ad')
       };
       expect(deserializedUUID).to.deep.equal(expectedResult);
     });