Add get course by Id logic. and fixed bug I accidently introduced into listening on proper port.

Author: Jrschellenberg

src/app.js

@@ -42,6 +42,7 @@ db.on('connected', function() {
 			// Callback to populate DB once collections have been cleared
 			seeder.populateModels(data, function() {
 				console.log("Finished seeding Database!");
+				
 				app.emit("appStarted"); //Emits an event to tell our tests it is ok to now test.
 			});
 		});

src/bin/www

@@ -16,7 +16,6 @@ var port = normalizePort(process.env.PORT || '3000');
 app.set('port', port);
 console.log("Listening on port", port);
 
-
 /**
  * Create HTTP server.
  */
@@ -27,11 +26,10 @@ var server = http.createServer(app);
  * Listen on provided port, on all network interfaces.
  */
 
+server.listen(port); //Dont delete this >.>
 server.on('error', onError);
 server.on('listening', onListening);
 
-
-
 /**
  * Normalize a port into a number, string, or false.
  */

src/models/course.js

@@ -1,5 +1,7 @@
 const mongoose = require('mongoose');
 const Schema = mongoose.Schema;
+const User = require('./user');
+const Review = require('./review');
 
 const CourseSchema = new mongoose.Schema({
 	user: {
@@ -44,6 +46,14 @@ const CourseSchema = new mongoose.Schema({
 	]
 });
 
+CourseSchema.methods.getUser = function(cb) {
+	return User.findById(this.user, cb);
+};
+
+CourseSchema.methods.getReview = function(index, cb) {
+	return Review.findById(this.reviews[index], cb);
+};
+
 
 const Course = mongoose.model('Course', CourseSchema);
 module.exports = Course;

src/models/review.js

@@ -1,5 +1,6 @@
 const mongoose = require('mongoose');
 const Schema = mongoose.Schema;
+const User = require('./user');
 
 const ReviewSchema = new Schema({
 	user: {
@@ -21,6 +22,10 @@ const ReviewSchema = new Schema({
 	}
 });
 
+ReviewSchema.methods.getUser = function(cb) {
+	return User.findById(this.user, cb);
+};
+
 
 const Review = mongoose.model('Review', ReviewSchema);
 module.exports = Review;

src/routes/course.js

@@ -22,13 +22,34 @@ router.get('/', (req, res, next) => {
 	});
 });
 
-// router.get('/:courseId', (req, res, next) => {
-// 	Course.findbyId(req.params.courseId, (err, course) => {
-//		
-//		
-// 	});
-//	
-// });
+router.get('/:courseId', (req, res, next) => {
+	Course.findById(req.params.courseId, (err, course) => {
+		Utils.isError(err, next);
+		course.getUser((err1, user) => {
+			Utils.isError(err1, next);
+			course.user = user;
+			let itemsProcessed = 0;
+			course.reviews.forEach((courseReview, index, array) => {
+				course.getReview(index, (err2, review) => {
+					Utils.isError(err2, next);
+					review.getUser((err3, reviewUser) => {
+						Utils.isError(err3, next);
+						review.user = reviewUser;
+						course.reviews[index] = review;
+						itemsProcessed++;
+						if(itemsProcessed === array.length){
+							res.locals.course = course;
+							next();
+						}
+					});
+				});
+			});
+		});
+	});
+}, (req, res) => {
+	let status = 200;
+	return res.status(status).json({success: true, message: "Course Successfully retrieved!", status: status, course: res.locals.course });
+});
 
 router.post('/', authorizeUser, (req, res, next) => {
 	if(!req.body.title || !req.body.description){

src/utils.js

@@ -4,4 +4,9 @@ export default class Utils {
 		err.status = status;
 		return next(err);
 	}
+	static isError(err, next){
+		if(err){
+			return next(err);
+		}
+	}
 }

test/course_test.js

@@ -42,14 +42,18 @@ describe('Courses', () => {
 		});
 	});
 
-	// describe('/GET course', () => {
-	// 	it('should GET a course when provided with valid ID', (done) => {
-	//		
-	//		
-	//		
-	// 	});
-	//	
-	// });
+	describe('/GET course', () => {
+		it('should GET a course when provided with valid ID', (done) => {
+			chai.request(server)
+				.get(courseIndexLink+'57029ed4795118be119cc43d')
+				.end((err, res) => {
+					res.body.should.have.property('message').equal("Course Successfully retrieved!");
+					res.should.have.status(200);
+					done();
+				});
+		});
+
+	});
 
 	describe('/POST courses', () => {
 		it('should POST a course with proper fields and proper auth', (done) => {