Merge pull request #971 from jloveland/session-tests-return

Add return done() to session secret configuration tests
meanjs · Oct 9, 2015 · 9c78c63 · 9c78c63
2 parents 3454cf8 + b07af94
commit 9c78c63
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/modules/core/tests/server/core.server.config.tests.js b/modules/core/tests/server/core.server.config.tests.js
@@ -17,14 +17,14 @@ describe('Configuration Tests:', function () {
     before(function(done) {
       User.remove(function(err) {
         should.not.exist(err);
-        done();
+        return done();
       });
     });
 
     after(function(done) {
       User.remove(function(err) {
         should.not.exist(err);
-        done();
+        return done();
       });
     });
 
@@ -128,7 +128,7 @@ describe('Configuration Tests:', function () {
       config.utils.validateSessionSecret(conf, true).should.equal(false);
       // set env back to test
       process.env.NODE_ENV = 'test';
-      done();
+      return done();
     });
 
     it('should accept non-default session secret when running in production', function (done) {
@@ -138,7 +138,7 @@ describe('Configuration Tests:', function () {
       config.utils.validateSessionSecret(conf, true).should.equal(true);
       // set env back to test
       process.env.NODE_ENV = 'test';
-      done();
+      return done();
     });
 
     it('should accept default session secret when running in development', function (done) {
@@ -148,13 +148,13 @@ describe('Configuration Tests:', function () {
       config.utils.validateSessionSecret(conf, true).should.equal(true);
       // set env back to test
       process.env.NODE_ENV = 'test';
-      done();
+      return done();
     });
 
     it('should accept default session secret when running in test', function (done) {
       var conf = { sessionSecret: 'MEAN' };
       config.utils.validateSessionSecret(conf, true).should.equal(true);
-      done();
+      return done();
     });
   });
 });