feat: Improve handling aggregate query errors

Author: dplewis

spec/ParseQuery.Aggregate.spec.js

@@ -1500,4 +1500,21 @@ describe('Parse.Query Aggregate testing', () => {
     expect(results.length).toEqual(3);
     await database.adapter.deleteAllClasses(false);
   });
+
+  it_only_db('mongo')('aggregate handle mongodb errors', async () => {
+    const pipeline = [
+      {
+        $search: {
+          index: "default",
+          text: {
+            path: ["name"],
+            query: 'foo',
+          },
+        },
+      },
+    ];
+    await expectAsync(new Parse.Query(TestObject).aggregate(pipeline)).toBeRejectedWith(
+      new Parse.Error(Parse.Error.INVALID_QUERY, 'Using $search and $vectorSearch aggregation stages requires additional configuration. Please connect to Atlas or an AtlasCLI local deployment to enable.For more information on how to connect, see https://dochub.mongodb.org/core/atlas-cli-deploy-local-reqs.')
+    );
+  });
 });

src/Routers/AggregateRouter.js

@@ -5,7 +5,7 @@ import ClassesRouter from './ClassesRouter';
 import UsersRouter from './UsersRouter';
 
 export class AggregateRouter extends ClassesRouter {
-  handleFind(req) {
+  async handleFind(req) {
     const body = Object.assign(req.body || {}, ClassesRouter.JSONFromQuery(req.query));
     const options = {};
     if (body.distinct) {
@@ -31,24 +31,25 @@ export class AggregateRouter extends ClassesRouter {
     if (typeof body.where === 'string') {
       body.where = JSON.parse(body.where);
     }
-    return rest
-      .find(
+    try {
+      const response = await rest.find(
         req.config,
         req.auth,
         this.className(req),
         body.where,
         options,
         req.info.clientSDK,
         req.info.context
-      )
-      .then(response => {
-        for (const result of response.results) {
-          if (typeof result === 'object') {
-            UsersRouter.removeHiddenProperties(result);
-          }
+      );
+      for (const result of response.results) {
+        if (typeof result === 'object') {
+          UsersRouter.removeHiddenProperties(result);
         }
-        return { response };
-      });
+      }
+      return { response };
+    } catch (e) {
+      throw new Parse.Error(Parse.Error.INVALID_QUERY, e.message);
+    }
   }
 
   /* Builds a pipeline from the body. Originally the body could be passed as a single object,