test: add test showing that bulkWrite strips out keys that arent in schema by default re: code review on #15036

vkarpov15 · vkarpov15 · commit c9cbea055905 · 2024-11-13T11:53:01.000-05:00
diff --git a/test/model.test.js b/test/model.test.js
@@ -4157,7 +4157,8 @@ describe('Model', function() {
             filter: { _id: r1._id },
             update: {
               $set: {
-                'testArray.$[element].field1': field1update
+                'testArray.$[element].field1': field1update,
+                'testArray.$[element].nonexistentProp': field1update
               }
             },
             arrayFilters: [
@@ -4168,8 +4169,9 @@ describe('Model', function() {
             ]
           }
         }]);
-        const r2 = await TestModel.findById(r1._id);
+        const r2 = await TestModel.findById(r1._id).lean();
         assert.equal(r2.testArray[0].field1, field1update);
+        assert.strictEqual(r2.testArray[0].nonexistentProp, undefined);
       });
 
       it('with child timestamps and array filters (gh-7032)', async function() {

Original file line number	Diff line number	Diff line change
`@@ -4157,7 +4157,8 @@ describe('Model', function() {`
`4157`	`4157`	`filter: { _id: r1._id },`
`4158`	`4158`	`update: {`
`4159`	`4159`	`$set: {`
`4160`		`- 'testArray.$[element].field1': field1update`
	`4160`	`+ 'testArray.$[element].field1': field1update,`
	`4161`	`+ 'testArray.$[element].nonexistentProp': field1update`
`4161`	`4162`	`}`
`4162`	`4163`	`},`
`4163`	`4164`	`arrayFilters: [`
`@@ -4168,8 +4169,9 @@ describe('Model', function() {`
`4168`	`4169`	`]`
`4169`	`4170`	`}`
`4170`	`4171`	`}]);`
`4171`		`- const r2 = await TestModel.findById(r1._id);`
	`4172`	`+ const r2 = await TestModel.findById(r1._id).lean();`
`4172`	`4173`	`assert.equal(r2.testArray[0].field1, field1update);`
	`4174`	`+ assert.strictEqual(r2.testArray[0].nonexistentProp, undefined);`
`4173`	`4175`	`});`
`4174`	`4176`
`4175`	`4177`	`it('with child timestamps and array filters (gh-7032)', async function() {`