fix(castArrayFilters): handle casting on all fields of array filter

lib/helpers/update/castArrayFilters.js

@@ -38,40 +38,41 @@ module.exports = function castArrayFilters(query) {
     if (filter == null) {
       throw new Error(`Got null array filter in ${arrayFilters}`);
     }
-    const firstKey = Object.keys(filter)[0];
+    for (const key in filter) {
 
-    if (filter[firstKey] == null) {
-      continue;
-    }
+      if (filter[key] == null) {
+        continue;
+      }
 
-    const dot = firstKey.indexOf('.');
-    let filterPath = dot === -1 ?
-      updatedPathsByFilter[firstKey] + '.0' :
-      updatedPathsByFilter[firstKey.substr(0, dot)] + '.0' + firstKey.substr(dot);
+      const dot = key.indexOf('.');
+      let filterPath = dot === -1 ?
+        updatedPathsByFilter[key] + '.0' :
+        updatedPathsByFilter[key.substr(0, dot)] + '.0' + key.substr(dot);
 
-    if (filterPath == null) {
-      throw new Error(`Filter path not found for ${firstKey}`);
-    }
+      if (filterPath == null) {
+        throw new Error(`Filter path not found for ${key}`);
+      }
 
-    // If there are multiple array filters in the path being updated, make sure
-    // to replace them so we can get the schema path.
-    filterPath = cleanPositionalOperators(filterPath);
+      // If there are multiple array filters in the path being updated, make sure
+      // to replace them so we can get the schema path.
+      filterPath = cleanPositionalOperators(filterPath);
 
-    const schematype = getPath(schema, filterPath);
-    if (schematype == null) {
-      if (!strictQuery) {
-        return;
+      const schematype = getPath(schema, filterPath);
+      if (schematype == null) {
+        if (!strictQuery) {
+          return;
+        }
+        // For now, treat `strictQuery = true` and `strictQuery = 'throw'` as
+        // equivalent for casting array filters. `strictQuery = true` doesn't
+        // quite work in this context because we never want to silently strip out
+        // array filters, even if the path isn't in the schema.
+        throw new Error(`Could not find path "${filterPath}" in schema`);
+      }
+      if (typeof filter[key] === 'object') {
+        filter[key] = castFilterPath(query, schematype, filter[key]);
+      } else {
+        filter[key] = schematype.castForQuery(filter[key]);
       }
-      // For now, treat `strictQuery = true` and `strictQuery = 'throw'` as
-      // equivalent for casting array filters. `strictQuery = true` doesn't
-      // quite work in this context because we never want to silently strip out
-      // array filters, even if the path isn't in the schema.
-      throw new Error(`Could not find path "${filterPath}" in schema`);
-    }
-    if (typeof filter[firstKey] === 'object') {
-      filter[firstKey] = castFilterPath(query, schematype, filter[firstKey]);
-    } else {
-      filter[firstKey] = schematype.castForQuery(filter[firstKey]);
     }
   }
 };

test/helpers/update.castArrayFilters.test.js

@@ -42,6 +42,31 @@ describe('castArrayFilters', function() {
     done();
   });
 
+  it('casts on multiple fields', function(done) {
+    const schema = new Schema({
+      comments: [{
+        text: String,
+        replies: [{
+          beginAt: Date,
+          endAt: Date
+        }]
+      }]
+    });
+    const q = new Query();
+    q.schema = schema;
+
+    q.updateOne({}, { $set: { 'comments.$[x].replies.$[y].endAt': '2019-01-01' } }, {
+      arrayFilters: [{ 'x.text': 123 }, { 'y.beginAt': { $gte: '2018-01-01' }, 'y.endAt': { $lt: '2020-01-01' } }]
+    });
+    castArrayFilters(q);
+
+    assert.strictEqual(q.options.arrayFilters[0]['x.text'], '123');
+    assert.ok(q.options.arrayFilters[1]['y.beginAt'].$gte instanceof Date);
+    assert.ok(q.options.arrayFilters[1]['y.endAt'].$lt instanceof Date);
+
+    done();
+  });
+
   it('sane error on same filter twice', function(done) {
     const schema = new Schema({
       comments: [{