Makes sure tests dont yeild unhandled promise rejections

Author: flovilmart

spec/ParseACL.spec.js

@@ -2,7 +2,6 @@
 // hungry/js/test/parse_acl_test.js
 var rest = require('../src/rest');
 var Config = require('../src/Config');
-var config = new Config('test');
 var auth = require('../src/Auth');
 
 describe('Parse.ACL', () => {
@@ -1235,6 +1234,7 @@ describe('Parse.ACL', () => {
   });
 
   it('regression test #701', done => {
+    const config = new Config('test');
     var anonUser = {
       authData: {
         anonymous: {
@@ -1248,6 +1248,7 @@ describe('Parse.ACL', () => {
         var user = req.object;
         var acl = new Parse.ACL(user);
         user.setACL(acl);
+        console.log('IN AFTER SAVE!');
         user.save(null, {useMasterKey: true}).then(user => {
           new Parse.Query('_User').get(user.objectId).then(() => {
             fail('should not have fetched user without public read enabled');
@@ -1256,7 +1257,7 @@ describe('Parse.ACL', () => {
             expect(error.code).toEqual(Parse.Error.OBJECT_NOT_FOUND);
             done();
           });
-        });
+        }, done.fail);
       }
     });
 

spec/helper.js

@@ -118,44 +118,40 @@ if (process.env.PARSE_SERVER_TEST_CACHE === 'redis') {
 const openConnections = {};
 
 // Set up a default API server for testing with default configuration.
-var app = express();
-var api = new ParseServer(defaultConfiguration);
-app.use('/1', api);
-app.use('/1', () => {
-  fail('should not call next');
-});
-var server = app.listen(port);
-server.on('connection', connection => {
-  const key = `${connection.remoteAddress}:${connection.remotePort}`;
-  openConnections[key] = connection;
-  connection.on('close', () => { delete openConnections[key] });
-});
+var app;
+var api;
+var server;
+
 // Allows testing specific configurations of Parse Server
 const reconfigureServer = changedConfiguration => {
   return new Promise((resolve, reject) => {
-    server.close(() => {
-      try {
-        const newConfiguration = Object.assign({}, defaultConfiguration, changedConfiguration, {
-          __indexBuildCompletionCallbackForTests: indexBuildPromise => indexBuildPromise.then(resolve, reject)
-        });
-        cache.clear();
-        app = express();
-        api = new ParseServer(newConfiguration);
-        api.use(require('./testing-routes').router);
-        app.use('/1', api);
-        app.use('/1', () => {
-          fail('should not call next');
-        });
-        server = app.listen(port);
-        server.on('connection', connection => {
-          const key = `${connection.remoteAddress}:${connection.remotePort}`;
-          openConnections[key] = connection;
-          connection.on('close', () => { delete openConnections[key] });
-        });
-      } catch(error) {
-        reject(error);
-      }
-    });
+    if (server) {
+      return server.close(() => {
+        server = undefined;
+        reconfigureServer(changedConfiguration).then(resolve, reject);
+      });
+    }
+    try {
+      const newConfiguration = Object.assign({}, defaultConfiguration, changedConfiguration, {
+        __indexBuildCompletionCallbackForTests: indexBuildPromise => indexBuildPromise.then(resolve, reject)
+      });
+      cache.clear();
+      app = express();
+      api = new ParseServer(newConfiguration);
+      api.use(require('./testing-routes').router);
+      app.use('/1', api);
+      app.use('/1', () => {
+        fail('should not call next');
+      });
+      server = app.listen(port);
+      server.on('connection', connection => {
+        const key = `${connection.remoteAddress}:${connection.remotePort}`;
+        openConnections[key] = connection;
+        connection.on('close', () => { delete openConnections[key] });
+      });
+    } catch(error) {
+      reject(error);
+    }
   });
 }
 

spec/rest.spec.js

@@ -6,13 +6,14 @@ var Parse = require('parse/node').Parse;
 var rest = require('../src/rest');
 var request = require('request');
 
-var config = new Config('test');
-const database = config.database;
+let config;
+let database;
 
 describe('rest create', () => {
 
   beforeEach(() => {
     config = new Config('test');
+    database = config.database;
   });
 
   it('handles _id', done => {

spec/schemas.spec.js

@@ -5,7 +5,7 @@ var request = require('request');
 var dd = require('deep-diff');
 var Config = require('../src/Config');
 
-var config = new Config('test');
+var config;
 
 var hasAllPODobject = () => {
   var obj = new Parse.Object('HasAllPOD');
@@ -131,8 +131,11 @@ var masterKeyHeaders = {
 };
 
 describe('schemas', () => {
-
   beforeEach(() => {
+    config = new Config('test');
+  });
+
+  afterEach(() => {
     config.database.schemaCache.clear();
   });