remove before done

Author: dplewis

spec/MongoStorageAdapter.spec.js

@@ -15,8 +15,8 @@ const fakeClient = {
 // These tests are specific to the mongo storage adapter + mongo storage format
 // and will eventually be moved into their own repo
 describe_only_db('mongo')('MongoStorageAdapter', () => {
-  beforeEach(done => {
-    new MongoStorageAdapter({ uri: databaseURI }).deleteAllClasses().then(done, fail);
+  beforeEach(async () => {
+    await new MongoStorageAdapter({ uri: databaseURI }).deleteAllClasses();
     Config.get(Parse.applicationId).schemaCache.clear();
   });
 

spec/ParseGlobalConfig.spec.js

@@ -4,7 +4,7 @@ const request = require('../lib/request');
 const Config = require('../lib/Config');
 
 describe('a GlobalConfig', () => {
-  beforeEach(done => {
+  beforeEach(async () => {
     const config = Config.get('test');
     const query = on_db(
       'mongo',
@@ -16,7 +16,7 @@ describe('a GlobalConfig', () => {
         return { objectId: '1' };
       }
     );
-    config.database.adapter
+    await config.database.adapter
       .upsertOneObject(
         '_GlobalConfig',
         {
@@ -31,11 +31,7 @@ describe('a GlobalConfig', () => {
           params: { companies: ['US', 'DK'], counter: 20, internalParam: 'internal' },
           masterKeyOnly: { internalParam: true },
         }
-      )
-      .then(done, err => {
-        jfail(err);
-        done();
-      });
+      );
   });
 
   const headers = {

spec/ParseQuery.Aggregate.spec.js

@@ -69,8 +69,8 @@ const get = function (url, options) {
 };
 
 describe('Parse.Query Aggregate testing', () => {
-  beforeEach(done => {
-    loadTestData().then(done, done);
+  beforeEach(async () => {
+    await loadTestData();
   });
 
   it('should only query aggregate with master key', done => {

spec/PublicAPI.spec.js

@@ -111,8 +111,8 @@ describe('public API', () => {
 });
 
 describe('public API without publicServerURL', () => {
-  beforeEach(done => {
-    reconfigureServer({ appName: 'unused' }).then(done, fail);
+  beforeEach(async () => {
+    await reconfigureServer({ appName: 'unused' });
   });
   it('should get 404 on verify_email', done => {
     request('http://localhost:8378/1/apps/test/verify_email', (err, httpResponse) => {
@@ -137,8 +137,8 @@ describe('public API without publicServerURL', () => {
 });
 
 describe('public API supplied with invalid application id', () => {
-  beforeEach(done => {
-    reconfigureServer({ appName: 'unused' }).then(done, fail);
+  beforeEach(async () => {
+    await reconfigureServer({ appName: 'unused' });
   });
 
   it('should get 403 on verify_email', done => {

spec/PurchaseValidation.spec.js

@@ -23,13 +23,8 @@ function createProduct() {
 }
 
 describe('test validate_receipt endpoint', () => {
-  beforeEach(done => {
-    createProduct()
-      .then(done)
-      .catch(function (err) {
-        console.error({ err });
-        done();
-      });
+  beforeEach(async () => {
+    await createProduct();
   });
 
   it('should bypass appstore validation', async () => {

spec/PushController.spec.js

@@ -1224,14 +1224,11 @@ describe('PushController', () => {
       },
     };
 
-    beforeEach(done => {
-      reconfigureServer({
+    beforeEach(async () => {
+      await reconfigureServer({
         push: { adapter: pushAdapter },
-      })
-        .then(() => {
-          config = Config.get(Parse.applicationId);
-        })
-        .then(done, done.fail);
+      });
+      config = Config.get(Parse.applicationId);
     });
 
     it('should throw if both expiration_time and expiration_interval are set', () => {

spec/RevocableSessionsUpgrade.spec.js

@@ -15,9 +15,9 @@ function createUser() {
 }
 
 describe_only_db('mongo')('revocable sessions', () => {
-  beforeEach(done => {
+  beforeEach(async () => {
     // Create 1 user with the legacy
-    createUser().then(done);
+    await createUser();
   });
 
   it('should upgrade legacy session token', done => {

spec/UserPII.spec.js

@@ -243,8 +243,8 @@ describe('Personally Identifiable Information', () => {
   });
 
   describe('with deprecated configured sensitive fields', () => {
-    beforeEach(done => {
-      return reconfigureServer({ userSensitiveFields: ['ssn', 'zip'] }).then(done);
+    beforeEach(async () => {
+      await reconfigureServer({ userSensitiveFields: ['ssn', 'zip'] });
     });
 
     it('should be able to get own PII via API with object', done => {
@@ -691,12 +691,12 @@ describe('Personally Identifiable Information', () => {
   });
 
   describe('with configured sensitive fields via CLP', () => {
-    beforeEach(done => {
-      reconfigureServer({
+    beforeEach(async () => {
+      await reconfigureServer({
         protectedFields: {
           _User: { '*': ['ssn', 'zip'], 'role:Administrator': [] },
         },
-      }).then(done);
+      });
     });
 
     it('should be able to get own PII via API with object', done => {