Do not throw error if user provide a pointer like index onMongo (#6923)

* Do not throw error if user provide a pointer like index on mongo * Add test
parse-community · Oct 1, 2020 · 929c4e1 · 929c4e1
1 parent 66f7af9
commit 929c4e1
Show file tree

Hide file tree

Showing 2 changed files with 183 additions and 105 deletions.
diff --git a/spec/schemas.spec.js b/spec/schemas.spec.js
@@ -2540,7 +2540,7 @@ describe('schemas', () => {
 
   it('unset field in beforeSave should not stop object creation', done => {
     const hook = {
-      method: function(req) {
+      method: function (req) {
         if (req.object.get('undesiredField')) {
           req.object.unset('undesiredField');
         }
@@ -3110,6 +3110,77 @@ describe('schemas', () => {
       });
     });
 
+    it_only_db('mongo')(
+      'lets you add index with with pointer like structure',
+      done => {
+        request({
+          url: 'http://localhost:8378/1/schemas/NewClass',
+          method: 'POST',
+          headers: masterKeyHeaders,
+          json: true,
+          body: {},
+        }).then(() => {
+          request({
+            url: 'http://localhost:8378/1/schemas/NewClass',
+            method: 'PUT',
+            headers: masterKeyHeaders,
+            json: true,
+            body: {
+              fields: {
+                aPointer: { type: 'Pointer', targetClass: 'NewClass' },
+              },
+              indexes: {
+                pointer: { _p_aPointer: 1 },
+              },
+            },
+          }).then(response => {
+            expect(
+              dd(response.data, {
+                className: 'NewClass',
+                fields: {
+                  ACL: { type: 'ACL' },
+                  createdAt: { type: 'Date' },
+                  updatedAt: { type: 'Date' },
+                  objectId: { type: 'String' },
+                  aPointer: { type: 'Pointer', targetClass: 'NewClass' },
+                },
+                classLevelPermissions: defaultClassLevelPermissions,
+                indexes: {
+                  _id_: { _id: 1 },
+                  pointer: { _p_aPointer: 1 },
+                },
+              })
+            ).toEqual(undefined);
+            request({
+              url: 'http://localhost:8378/1/schemas/NewClass',
+              headers: masterKeyHeaders,
+              json: true,
+            }).then(response => {
+              expect(response.data).toEqual({
+                className: 'NewClass',
+                fields: {
+                  ACL: { type: 'ACL' },
+                  createdAt: { type: 'Date' },
+                  updatedAt: { type: 'Date' },
+                  objectId: { type: 'String' },
+                  aPointer: { type: 'Pointer', targetClass: 'NewClass' },
+                },
+                classLevelPermissions: defaultClassLevelPermissions,
+                indexes: {
+                  _id_: { _id: 1 },
+                  pointer: { _p_aPointer: 1 },
+                },
+              });
+              config.database.adapter.getIndexes('NewClass').then(indexes => {
+                expect(indexes.length).toEqual(2);
+                done();
+              });
+            });
+          });
+        });
+      }
+    );
+
     it('lets you add multiple indexes', done => {
       request({
         url: 'http://localhost:8378/1/schemas/NewClass',