From d00f4ce0cc5190394664f08a8882269d0c1a683b Mon Sep 17 00:00:00 2001
From: loulin <lin.lou@hotmail.com>
Date: Fri, 19 Jun 2015 12:09:15 +0800
Subject: [PATCH] should return done(err) in article server route test

---
 .../server/article.server.routes.tests.js     | 40 ++++++++++++++-----
 1 file changed, 30 insertions(+), 10 deletions(-)

diff --git a/modules/articles/tests/server/article.server.routes.tests.js b/modules/articles/tests/server/article.server.routes.tests.js
index c8c35c8718..aaa8f2937a 100644
--- a/modules/articles/tests/server/article.server.routes.tests.js
+++ b/modules/articles/tests/server/article.server.routes.tests.js
@@ -60,7 +60,9 @@ describe('Article CRUD tests', function() {
 			.expect(200)
 			.end(function(signinErr, signinRes) {
 				// Handle signin error
-				if (signinErr) done(signinErr);
+				if (signinErr) {
+					return done(signinErr);
+				}
 
 				// Get the userId
 				var userId = user.id;
@@ -71,13 +73,17 @@ describe('Article CRUD tests', function() {
 					.expect(200)
 					.end(function(articleSaveErr, articleSaveRes) {
 						// Handle article save error
-						if (articleSaveErr) done(articleSaveErr);
+						if (articleSaveErr) {
+							return done(articleSaveErr);
+						}
 
 						// Get a list of articles
 						agent.get('/api/articles')
 							.end(function(articlesGetErr, articlesGetRes) {
 								// Handle article save error
-								if (articlesGetErr) done(articlesGetErr);
+								if (articlesGetErr) {
+									return done(articlesGetErr);
+								}
 
 								// Get articles list
 								var articles = articlesGetRes.body;
@@ -112,7 +118,9 @@ describe('Article CRUD tests', function() {
 			.expect(200)
 			.end(function(signinErr, signinRes) {
 				// Handle signin error
-				if (signinErr) done(signinErr);
+				if (signinErr) {
+					return done(signinErr);
+				}
 
 				// Get the userId
 				var userId = user.id;
@@ -137,7 +145,9 @@ describe('Article CRUD tests', function() {
 			.expect(200)
 			.end(function(signinErr, signinRes) {
 				// Handle signin error
-				if (signinErr) done(signinErr);
+				if (signinErr) {
+					return done(signinErr);
+				}
 
 				// Get the userId
 				var userId = user.id;
@@ -148,7 +158,9 @@ describe('Article CRUD tests', function() {
 					.expect(200)
 					.end(function(articleSaveErr, articleSaveRes) {
 						// Handle article save error
-						if (articleSaveErr) done(articleSaveErr);
+						if (articleSaveErr) {
+							return done(articleSaveErr);
+						}
 
 						// Update article title
 						article.title = 'WHY YOU GOTTA BE SO MEAN?';
@@ -159,7 +171,9 @@ describe('Article CRUD tests', function() {
 							.expect(200)
 							.end(function(articleUpdateErr, articleUpdateRes) {
 								// Handle article update error
-								if (articleUpdateErr) done(articleUpdateErr);
+								if (articleUpdateErr) {
+									return done(articleUpdateErr);
+								}
 
 								// Set assertions
 								(articleUpdateRes.body._id).should.equal(articleSaveRes.body._id);
@@ -215,7 +229,9 @@ describe('Article CRUD tests', function() {
 			.expect(200)
 			.end(function(signinErr, signinRes) {
 				// Handle signin error
-				if (signinErr) done(signinErr);
+				if (signinErr) {
+					return done(signinErr);
+				}
 
 				// Get the userId
 				var userId = user.id;
@@ -226,7 +242,9 @@ describe('Article CRUD tests', function() {
 					.expect(200)
 					.end(function(articleSaveErr, articleSaveRes) {
 						// Handle article save error
-						if (articleSaveErr) done(articleSaveErr);
+						if (articleSaveErr) {
+							return done(articleSaveErr);
+						}
 
 						// Delete an existing article
 						agent.delete('/api/articles/' + articleSaveRes.body._id)
@@ -234,7 +252,9 @@ describe('Article CRUD tests', function() {
 							.expect(200)
 							.end(function(articleDeleteErr, articleDeleteRes) {
 								// Handle article error error
-								if (articleDeleteErr) done(articleDeleteErr);
+								if (articleDeleteErr) {
+									return done(articleDeleteErr);
+								}
 
 								// Set assertions
 								(articleDeleteRes.body._id).should.equal(articleSaveRes.body._id);