fix(NODE-4887): serializeInto does not check for the presence of a toBSON method for values in Map entries (#555)

Co-authored-by: Bailey Pearson <[email protected]>

Author: W-A-James

src/parser/serializer.ts

@@ -757,7 +757,11 @@ export function serializeInto(
 
       // Get the entry values
       const key = entry.value[0];
-      const value = entry.value[1];
+      let value = entry.value[1];
+
+      if (typeof value?.toBSON === 'function') {
+        value = value.toBSON();
+      }
 
       // Check the type of the value
       const type = typeof value;

test/node/to_bson_test.js

@@ -183,6 +183,15 @@ describe('toBSON', function () {
       const sizeNestedToBSON = BSON.calculateObjectSize({ a: [0] });
       expect(sizeNestedToBSON).to.equal(33);
     });
+
+    it('uses toBSON on values contained in a map', () => {
+      const map = new Map();
+      map.set('a', 100);
+
+      const serializedData = BSON.serialize(map);
+      const deserializedData = BSON.deserialize(serializedData);
+      expect(deserializedData).to.have.property('a', 'hello number');
+    });
   });
 
   it('should use toBSON in calculateObjectSize', () => {