Added done callback to explicitly let mocha know about before and after methods.

Author: Vikas Agarwal

src/routes/scopeChangeRequests/create.spec.js

@@ -94,7 +94,7 @@ describe('Create Scope Change Rquest', () => {
   let projectWithPendingChange;
   let projectWithApprovedChange;
 
-  before(() => {
+  before((done) => {
     const projectStatuses = [
       PROJECT_STATUS.DRAFT,
       PROJECT_STATUS.IN_REVIEW,
@@ -106,11 +106,12 @@ describe('Create Scope Change Rquest', () => {
       .then(_projects => _projects.map((project, i) => [projectStatuses[i], project]))
       .then((_projectStatusPairs) => {
         projects = _.fromPairs(_projectStatusPairs);
-      });
+      })
+      .then(() => done());
   });
 
-  after(() => {
-    testUtil.clearDb();
+  after((done) => {
+    testUtil.clearDb(done);
   });
 
   describe('POST projects/{projectId}/scopeChangeRequests', () => {

src/routes/scopeChangeRequests/update.spec.js

@@ -99,7 +99,7 @@ describe('Update Scope Change Rquest', () => {
   let project;
   let scopeChangeRequest;
 
-  before(() =>
+  before((done) =>
     testUtil
       .clearDb()
       .then(() => createProject(PROJECT_STATUS.REVIEWED))
@@ -110,11 +110,12 @@ describe('Update Scope Change Rquest', () => {
       .then(_project => createScopeChangeRequest(_project))
       .then((_scopeChangeRequest) => {
         scopeChangeRequest = _scopeChangeRequest;
-      }),
+      })
+      .then(() => done()),
   );
 
-  after(() => {
-    testUtil.clearDb();
+  after((done) => {
+    testUtil.clearDb(done);
   });
 
   describe('PATCH projects/{projectId}/scopeChangeRequests/{requestId}', () => {