fix: also allow setting nested field to undefined re: #14205

Author: vkarpov15

lib/document.js

@@ -1153,7 +1153,7 @@ Document.prototype.$set = function $set(path, val, type, options) {
           } else {
             throw new StrictModeError(key);
           }
-        } else if (pathtype === 'nested' && valForKey === null) {
+        } else if (pathtype === 'nested' && valForKey == null) {
           this.$set(pathName, valForKey, constructing, options);
         }
       } else if (valForKey !== void 0) {

test/document.test.js

@@ -12456,6 +12456,25 @@ describe('document', function() {
     doc.set({ nested: null });
     assert.strictEqual(doc.toObject().nested, null);
   });
+
+  it('handles setting nested path to undefined (gh-14205)', function() {
+    const schema = new mongoose.Schema({
+      nested: {
+        key1: String,
+        key2: String
+      }
+    });
+
+    const Model = db.model('Test', schema);
+
+    const doc = new Model();
+    doc.init({
+      nested: { key1: 'foo', key2: 'bar' }
+    });
+
+    doc.set({ nested: void 0 });
+    assert.strictEqual(doc.toObject().nested, void 0);
+  });
 });
 
 describe('Check if instance function that is supplied in schema option is availabe', function() {