Skip to content

Commit 08014fc

Browse files
Add logic and test to retrieve all courses _id and title only
1 parent 1befb32 commit 08014fc

5 files changed

Lines changed: 17 additions & 13 deletions

File tree

package-lock.json

Lines changed: 0 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
"author": "",
1414
"license": "ISC",
1515
"devDependencies": {
16-
"babel": "^6.23.0",
1716
"babel-cli": "^6.26.0",
1817
"babel-plugin-istanbul": "^4.1.5",
1918
"babel-preset-es2015": "^6.24.1",

src/middleware/index.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@ export function authorizeUser(req, res, next) {
77
if(!user || !user.name || !user.pass){
88
return Utils.throwError(401, "Access Denied: Please supply login credentials!", next);
99
}
10-
console.log(user.name);
11-
console.log(user.pass);
1210
User.authenticate(user.name, user.pass, (error, user) => {
1311
if(error || !user){
1412
return Utils.throwError(401, "Access Denied: Wrong email or password", next);

src/routes/course.js

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,21 @@ import {authorizeUser} from "../middleware/index";
55
import Utils from '../utils';
66

77
router.get('/', (req, res, next) => {
8-
let status = 200;
9-
return res.status(status).json({success: true, message: "Courses Successfully retrieved!", status: status});
8+
Course.find({}, (err, docs) => {
9+
if(err){
10+
return next(err);
11+
}
12+
let courses = [];
13+
for(let i=0; i<docs.length; i++){
14+
let obj = {
15+
_id: docs[i]._id,
16+
title: docs[i].title
17+
};
18+
courses.push(obj);
19+
}
20+
let status = 200;
21+
return res.status(status).json({success: true, message: "Courses Successfully retrieved!", status: status, courses: courses});
22+
});
1023
});
1124

1225
router.post('/', authorizeUser, (req, res, next) => {

test/course_test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ describe('Courses', () => {
2828
Our GET Tests
2929
*/
3030

31-
describe('/GET course', () => {
31+
describe('/GET courses', () => {
3232
it('should GET all the courses', (done) => {
3333
chai.request(server)
3434
.get(courseIndexLink)
@@ -40,7 +40,7 @@ describe('Courses', () => {
4040
});
4141
});
4242

43-
describe('/POST course', () => {
43+
describe('/POST courses', () => {
4444
it('should POST a course with proper fields and proper auth', (done) => {
4545
let course = {
4646
title: "My first title!",

0 commit comments

Comments
 (0)