ci: Find duplicate and slow tests

.nvmrc

@@ -1,2 +1 @@
-10.14.2
-
+20.15.0

spec/CloudCode.Validator.spec.js

@@ -194,27 +194,6 @@ describe('cloud validator', () => {
       });
   });
 
-  it('set params on cloud functions', done => {
-    Parse.Cloud.define(
-      'hello',
-      () => {
-        return 'Hello world!';
-      },
-      {
-        fields: ['a'],
-      }
-    );
-    Parse.Cloud.run('hello', {})
-      .then(() => {
-        fail('function should have failed.');
-      })
-      .catch(error => {
-        expect(error.code).toEqual(Parse.Error.VALIDATION_ERROR);
-        expect(error.message).toEqual('Validation failed. Please specify data for a.');
-        done();
-      });
-  });
-
   it('allow params on cloud functions', done => {
     Parse.Cloud.define(
       'hello',
@@ -1629,7 +1608,7 @@ describe('cloud validator', () => {
     }
   });
 
-  it('Logs on invalid config', () => {
+  it('Logs on multiple invalid configs', () => {
     const fields = [
       {
         field: 'otherKey',

spec/CloudCode.spec.js

@@ -49,6 +49,7 @@ describe('Cloud Code', () => {
   });
 
   it('cloud code must be valid type', async () => {
+    spyOn(console, 'error').and.callFake(() => {});
     await expectAsync(reconfigureServer({ cloud: true })).toBeRejectedWith(
       "argument 'cloud' must either be a string or a function"
     );

spec/ParseFile.spec.js

@@ -842,30 +842,6 @@ describe('Parse.File testing', () => {
       expect(file.headers['content-range']).toBe('bytes 0-1212/1212');
     });
 
-    it('supports bytes range if end is greater than size', async () => {
-      const headers = {
-        'Content-Type': 'application/octet-stream',
-        'X-Parse-Application-Id': 'test',
-        'X-Parse-REST-API-Key': 'rest',
-      };
-      const response = await request({
-        method: 'POST',
-        headers: headers,
-        url: 'http://localhost:8378/1//files/file.txt ',
-        body: repeat('argle bargle', 100),
-      });
-      const b = response.data;
-      const file = await request({
-        url: b.url,
-        headers: {
-          'Content-Type': 'application/octet-stream',
-          'X-Parse-Application-Id': 'test',
-          Range: 'bytes=0-2000',
-        },
-      }).catch(e => e);
-      expect(file.headers['content-range']).toBe('bytes 0-1212/1212');
-    });
-
     it('supports bytes range with 0 length', async () => {
       const headers = {
         'Content-Type': 'application/octet-stream',

spec/ParseLiveQuery.spec.js

@@ -382,7 +382,6 @@ describe('ParseLiveQuery', function () {
     await obj2.save();
     obj2.set('foo', 'bart');
     await obj2.save();
-    await sleep(2000);
     expect(createSpy).toHaveBeenCalledTimes(1);
     expect(updateSpy).toHaveBeenCalledTimes(1);
   });

spec/ParseLiveQueryServer.spec.js

@@ -1633,35 +1633,6 @@ describe('ParseLiveQueryServer', function () {
         });
     });
 
-    it('matches CLP when find is restricted to userIds', done => {
-      const parseLiveQueryServer = new ParseLiveQueryServer({});
-      const acl = new Parse.ACL();
-      acl.setReadAccess(testUserId, true);
-      // Mock sessionTokenCache will return false when sessionToken is undefined
-      const client = {
-        sessionToken: 'sessionToken',
-        getSubscriptionInfo: jasmine.createSpy('getSubscriptionInfo').and.returnValue({
-          sessionToken: 'userId',
-        }),
-      };
-      const requestId = 0;
-
-      parseLiveQueryServer
-        ._matchesCLP(
-          {
-            find: { userId: true },
-          },
-          { className: 'Yolo' },
-          client,
-          requestId,
-          'find'
-        )
-        .then(isMatched => {
-          expect(isMatched).toBe(true);
-          done();
-        });
-    });
-
     it('matches CLP when find is restricted to userIds', done => {
       const parseLiveQueryServer = new ParseLiveQueryServer({});
       const acl = new Parse.ACL();

spec/ParseQuery.Aggregate.spec.js

@@ -402,7 +402,7 @@ describe('Parse.Query Aggregate testing', () => {
   });
 
   it_exclude_dbs(['postgres'])(
-    'can group by any date field (it does not work if you have dirty data)', // rows in your collection with non date data in the field that is supposed to be a date
+    'can group by any date field (it does not work if you have dirty data) (Mongo)', // rows in your collection with non date data in the field that is supposed to be a date
     done => {
       const obj1 = new TestObject({ dateField2019: new Date(1990, 11, 1) });
       const obj2 = new TestObject({ dateField2019: new Date(1990, 5, 1) });
@@ -440,7 +440,7 @@ describe('Parse.Query Aggregate testing', () => {
   );
 
   it_only_db('postgres')(
-    'can group by any date field (it does not work if you have dirty data)', // rows in your collection with non date data in the field that is supposed to be a date
+    'can group by any date field (it does not work if you have dirty data) (Postgres)', // rows in your collection with non date data in the field that is supposed to be a date
     done => {
       const obj1 = new TestObject({ dateField2019: new Date(1990, 11, 1) });
       const obj2 = new TestObject({ dateField2019: new Date(1990, 5, 1) });

spec/ParseQuery.hint.spec.js

@@ -27,7 +27,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     await TestUtils.destroyAllDataPermanently(false);
   });
 
-  it_only_mongodb_version('<5.1>=6')('query find with hint string', async () => {
+  it_only_mongodb_version('<5.1>=6')('query find with hint string (Mongo <5.1>=6)', async () => {
     const object = new TestObject();
     await object.save();
 
@@ -39,7 +39,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(explain.queryPlanner.winningPlan.inputStage.indexName).toBe('_id_');
   });
 
-  it_only_mongodb_version('>=5.1<6')('query find with hint string', async () => {
+  it_only_mongodb_version('>=5.1<6')('query find with hint string (Mongo >=5.1<6)', async () => {
     const object = new TestObject();
     await object.save();
 
@@ -78,7 +78,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(explain.queryPlanner.winningPlan.queryPlan.inputStage.keyPattern).toEqual({ _id: 1 });
   });
 
-  it_only_mongodb_version('<4.4')('query aggregate with hint string', async () => {
+  it_only_mongodb_version('<4.4')('query aggregate with hint string (Mongo <4.4)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
 
@@ -100,7 +100,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.inputStage.indexName).toBe('_id_');
   });
 
-  it_only_mongodb_version('>=4.4<5.1')('query aggregate with hint string', async () => {
+  it_only_mongodb_version('>=4.4<5.1')('query aggregate with hint string (Mongo >=4.4<5.1)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
 
@@ -124,7 +124,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.inputStage.inputStage.indexName).toBe('_id_');
   });
 
-  it_only_mongodb_version('>=5.1<5.2')('query aggregate with hint string', async () => {
+  it_only_mongodb_version('>=5.1<5.2')('query aggregate with hint string (Mongo >=5.1<5.2)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
 
@@ -148,7 +148,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.queryPlan.inputStage.inputStage.indexName).toBe('_id_');
   });
 
-  it_only_mongodb_version('>=5.2')('query aggregate with hint string', async () => {
+  it_only_mongodb_version('>=5.2')('query aggregate with hint string (Mongo >=5.2)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
 
@@ -172,7 +172,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.queryPlan.inputStage.inputStage.indexName).toBe('_id_');
   });
 
-  it_only_mongodb_version('<4.4')('query aggregate with hint object', async () => {
+  it_only_mongodb_version('<4.4')('query aggregate with hint object (Mongo <4.4)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
 
@@ -192,7 +192,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.inputStage.keyPattern).toEqual({ _id: 1 });
   });
 
-  it_only_mongodb_version('>=4.4<5.1')('query aggregate with hint object', async () => {
+  it_only_mongodb_version('>=4.4<5.1')('query aggregate with hint object (Mongo >=4.4<5.1)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
 
@@ -217,7 +217,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.inputStage.inputStage.keyPattern).toEqual({ _id: 1 });
   });
 
-  it_only_mongodb_version('>=5.1<5.2')('query aggregate with hint object', async () => {
+  it_only_mongodb_version('>=5.1<5.2')('query aggregate with hint object (Mongo >=5.1<5.2)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
 
@@ -242,7 +242,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.queryPlan.inputStage.inputStage.keyPattern).toEqual({ _id: 1 });
   });
 
-  it_only_mongodb_version('>=5.2')('query aggregate with hint object', async () => {
+  it_only_mongodb_version('>=5.2')('query aggregate with hint object (Mongo >=5.2)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
 
@@ -267,7 +267,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.queryPlan.inputStage.inputStage.keyPattern).toEqual({ _id: 1 });
   });
 
-  it_only_mongodb_version('<5.1>=6')('query find with hint (rest)', async () => {
+  it_only_mongodb_version('<5.1>=6')('query find with hint (rest) (Mongo <5.1>=6)', async () => {
     const object = new TestObject();
     await object.save();
     let options = Object.assign({}, masterKeyOptions, {
@@ -292,7 +292,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(explain.queryPlanner.winningPlan.inputStage.inputStage.indexName).toBe('_id_');
   });
 
-  it_only_mongodb_version('>=5.1<6')('query find with hint (rest)', async () => {
+  it_only_mongodb_version('>=5.1<6')('query find with hint (rest) (Mongo >=5.1<6)', async () => {
     const object = new TestObject();
     await object.save();
     let options = Object.assign({}, masterKeyOptions, {
@@ -317,7 +317,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(explain.queryPlanner.winningPlan.queryPlan.inputStage.inputStage.indexName).toBe('_id_');
   });
 
-  it_only_mongodb_version('<4.4')('query aggregate with hint (rest)', async () => {
+  it_only_mongodb_version('<4.4')('query aggregate with hint (rest) (Mongo <4.4)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
     let options = Object.assign({}, masterKeyOptions, {
@@ -344,7 +344,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.inputStage.keyPattern).toEqual({ _id: 1 });
   });
 
-  it_only_mongodb_version('>=4.4<5.1')('query aggregate with hint (rest)', async () => {
+  it_only_mongodb_version('>=4.4<5.1')('query aggregate with hint (rest) (Mongo >=4.4<5.1)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
     let options = Object.assign({}, masterKeyOptions, {
@@ -377,7 +377,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.inputStage.inputStage.keyPattern).toEqual({ _id: 1 });
   });
 
-  it_only_mongodb_version('>=5.1<5.2')('query aggregate with hint (rest)', async () => {
+  it_only_mongodb_version('>=5.1<5.2')('query aggregate with hint (rest) (Mongo >=5.1<5.2)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
     let options = Object.assign({}, masterKeyOptions, {
@@ -410,7 +410,7 @@ describe_only_db('mongo')('Parse.Query hint', () => {
     expect(queryPlanner.winningPlan.queryPlan.inputStage.inputStage.keyPattern).toEqual({ _id: 1 });
   });
 
-  it_only_mongodb_version('>=5.2')('query aggregate with hint (rest)', async () => {
+  it_only_mongodb_version('>=5.2')('query aggregate with hint (rest) (Mongo >=5.2)', async () => {
     const object = new TestObject({ foo: 'bar' });
     await object.save();
     let options = Object.assign({}, masterKeyOptions, {

spec/PostgresStorageAdapter.spec.js

@@ -362,7 +362,7 @@ describe_only_db('postgres')('PostgresStorageAdapter', () => {
     await dropTable(client, tableName);
   });
 
-  it('should use index for caseInsensitive query', async () => {
+  it('should use index for caseInsensitive query with user', async () => {
     await adapter.deleteAllClasses();
     const config = Config.get('test');
     config.schemaCache.clear();

spec/PushWorker.spec.js

@@ -120,7 +120,7 @@ describe('PushWorker', () => {
       });
     });
 
-    it('transforms body appropriately', () => {
+    it('transforms body appropriately with title locale', () => {
       const cleanBody = PushUtils.transformPushBodyForLocale(
         {
           data: {

spec/helper.js

@@ -286,6 +286,10 @@ afterEach(function (done) {
     .then(afterLogOut);
 });
 
+afterAll(() => {
+  global.displaySlowTests();
+});
+
 const TestObject = Parse.Object.extend({
   className: 'TestObject',
 });

spec/support/CurrentSpecReporter.js

@@ -1,15 +1,45 @@
 // Sets a global variable to the current test spec
 // ex: global.currentSpec.description
-
+const { performance } = require('perf_hooks');
 global.currentSpec = null;
 
+const timerMap = {};
+const duplicates = [];
+/** The min. execution time in seconds for a test to be considered slow. */
+const slowTestLimit = 2;
+
 class CurrentSpecReporter {
   specStarted(spec) {
+    if (timerMap[spec.fullName]) {
+      console.log('Duplicate spec: ' + spec.fullName);
+      duplicates.push(spec.fullName);
+    }
+    timerMap[spec.fullName] = performance.now();
     global.currentSpec = spec;
   }
-  specDone() {
+  specDone(result) {
+    if (result.status === 'excluded') {
+      delete timerMap[result.fullName];
+      return;
+    }
+    timerMap[result.fullName] = (performance.now() - timerMap[result.fullName]) / 1000;
     global.currentSpec = null;
   }
 }
+global.displaySlowTests = function() {
+  const times = Object.values(timerMap).sort((a,b) => b - a);
+  if (times.length > 0) {
+    console.log('Slow tests with execution time >=${slowTestLimit}s:');
+  }
+  times.forEach((time) => {
+    if (time >= slowTestLimit) {
+      console.warn(`${time.toFixed(3)}s:`, Object.keys(timerMap).find(key => timerMap[key] === time));
+    }
+  });
+  console.log('\n');
+  duplicates.forEach((spec) => {
+    console.warn('Duplicate spec: ' + spec);
+  });
+};
 
 module.exports = CurrentSpecReporter;