Merge pull request #421 from vepasto/0.4.0

Wait for async savings and removing in mocha tests
meanjs · Feb 23, 2015 · 709c27f · 709c27f
2 parents d1b4bde + 1897ef9
commit 709c27f
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 12 deletions.
diff --git a/modules/articles/tests/server/article.server.model.tests.js b/modules/articles/tests/server/article.server.model.tests.js
@@ -57,8 +57,8 @@ describe('Article Model Unit Tests:', function() {
 	});
 
 	afterEach(function(done) {
-		Article.remove().exec();
-		User.remove().exec();
-		done();
+		Article.remove().exec(function() {
+			User.remove().exec(done);
+		});
 	});
 });
diff --git a/modules/articles/tests/server/article.server.routes.tests.js b/modules/articles/tests/server/article.server.routes.tests.js
@@ -270,8 +270,8 @@ describe('Article CRUD tests', function() {
 	});
 
 	afterEach(function(done) {
-		User.remove().exec();
-		Article.remove().exec();
-		done();
+		User.remove().exec(function() {
+			Article.remove().exec(done);
+		});
 	});
 });
diff --git a/modules/users/tests/server/user.server.model.tests.js b/modules/users/tests/server/user.server.model.tests.js
@@ -52,10 +52,11 @@ describe('User Model Unit Tests:', function() {
 		});
 
 		it('should fail to save an existing user again', function(done) {
-			user.save();
-			return user2.save(function(err) {
-				should.exist(err);
-				done();
+			user.save(function() {
+				user2.save(function(err) {
+					should.exist(err);
+					done();
+				});	
 			});
 		});
 
@@ -69,7 +70,6 @@ describe('User Model Unit Tests:', function() {
 	});
 
 	after(function(done) {
-		User.remove().exec();
-		done();
+		User.remove().exec(done);
 	});
 });