committing before adding new logic

Author: Jrschellenberg

src/routes/course.js

@@ -5,23 +5,31 @@ import {authorizeUser} from "../middleware/index";
 import Utils from '../utils';
 
 router.get('/', (req, res, next) => {
-	Course.find({}, (err, docs) => {
+	Course.find({}, (err, courses) => {
 		if(err){
 			return next(err);
 		}
-		let courses = [];
-		for(let i=0; i<docs.length; i++){
+		let coursesArray = [];
+		for(let i=0; i<courses.length; i++){
 			let obj = {
-				_id: docs[i]._id,
-				title: docs[i].title
+				_id: courses[i]._id,
+				title: courses[i].title
 			};
-			courses.push(obj);
+			coursesArray.push(obj);
 		}
 		let status = 200;
-		return res.status(status).json({success: true, message: "Courses Successfully retrieved!", status: status, courses: courses});
+		return res.status(status).json({success: true, message: "Courses Successfully retrieved!", status: status, courses: coursesArray});
 	});
 });
 
+// router.get('/:courseId', (req, res, next) => {
+// 	Course.findbyId(req.params.courseId, (err, course) => {
+//		
+//		
+// 	});
+//	
+// });
+
 router.post('/', authorizeUser, (req, res, next) => {
 	if(!req.body.title || !req.body.description){
 		return Utils.throwError(400, "Bad Request", next );

test/course_test.js

@@ -18,12 +18,11 @@ const validAuth = {
 };
 
 describe('Courses', () => {
-	beforeEach((done) => {
-		Course.remove({}, (err) => {
-			done();
-		});
-	});
-	
+	// beforeEach((done) => {
+	// 	Course.remove({}, (err) => {
+	// 		done();
+	// 	});
+	// });
 	/*
 	Our GET Tests
 	 */
@@ -40,6 +39,15 @@ describe('Courses', () => {
 		});
 	});
 
+	// describe('/GET course', () => {
+	// 	it('should GET a course when provided with valid ID', (done) => {
+	//		
+	//		
+	//		
+	// 	});
+	//	
+	// });
+	
 	describe('/POST courses', () => {
 		it('should POST a course with proper fields and proper auth', (done) => {
 			let course = {
@@ -60,6 +68,7 @@ describe('Courses', () => {
 				.auth(validAuth.user, validAuth.pass)
 				.send(course)
 				.end((err, res) => {
+					res.body.should.have.property('message').equal('test');
 					res.should.have.status(201);
 					res.body.should.be.a('object');
 					res.body.course.should.have.property('title');

test/user_test.js

@@ -68,11 +68,6 @@ describe('Users', () => {
 	});
 
 	describe('/POST users', () => {
-		// beforeEach((done) => {
-		// 	User.remove({}, (err) => {
-		// 		done();
-		// 	});
-		// });
 		let user400 = {
 			emailAddress: 'wre23sd',
 			password: 'password',