Skip to content

Commit 69e6551

Browse files
and put course and more unit tests
1 parent ece30a8 commit 69e6551

2 files changed

Lines changed: 60 additions & 1 deletion

File tree

src/routes/course.js

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@ const Course = require('../models/course');
44
import {authorizeUser} from "../middleware/index";
55
import Utils from '../utils';
66

7+
/*
8+
GET routes
9+
*/
10+
711
router.get('/', (req, res, next) => {
812
Course.find({}, (err, courses) => {
913
if(err){
@@ -36,6 +40,22 @@ router.get('/:courseId', (req, res, next) => {
3640
});
3741
});
3842

43+
/*
44+
PUT Routes
45+
*/
46+
router.put('/:courseId', authorizeUser, (req, res, next) => {
47+
Course.findByIdAndUpdate(req.params.courseId, req.body, (err, course) => {
48+
console.log("hit this?!?");
49+
Utils.isError(err, next);
50+
let status = 204;
51+
console.log("how about here?!?!");
52+
return res.status(status).json({}); //Send status 204 and no content..
53+
});
54+
});
55+
56+
/*
57+
POST Routes
58+
*/
3959
router.post('/', authorizeUser, (req, res, next) => {
4060
if(!req.body.title || !req.body.description){
4161
return Utils.throwError(400, "Bad Request", next );

test/course_test.js

Lines changed: 40 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,46 @@ describe('Courses', () => {
5252
done();
5353
});
5454
});
55-
55+
});
56+
57+
describe('/PUT course', () => {
58+
let course = {
59+
title: "My first title!",
60+
description: "My course description",
61+
user: {
62+
_id: "57029ed4795118be119cc437"
63+
},
64+
steps: [
65+
{
66+
title: "step 1",
67+
description: "My first Updated Step!"
68+
}
69+
]
70+
};
71+
it('should PUT a course with proper fields and proper auth', (done) => {
72+
chai.request(server)
73+
.put(courseIndexLink+'57029ed4795118be119cc43d')
74+
.auth(validAuth.user, validAuth.pass)
75+
.send(course)
76+
.end((err, res) => {
77+
//res.body.should.have.property('message').equal('Course Successfully Updated!');
78+
res.should.have.status(204);
79+
done();
80+
});
81+
});
82+
83+
it('should not PUT a course if not authenticated user', (done) => {
84+
chai.request(server)
85+
.put(courseIndexLink+'57029ed4795118be119cc43d')
86+
.auth('InvalidUser'+validAuth.user, validAuth.pass)
87+
.send(course)
88+
.end((err, res) => {
89+
//res.body.should.have.property('message').equal('Course Successfully Updated!');
90+
res.should.have.status(401);
91+
done();
92+
});
93+
})
94+
5695
});
5796

5897
describe('/POST courses', () => {

0 commit comments

Comments
 (0)