Skip to content

Commit 8c9b2cb

Browse files
added auth to get and tests
1 parent 9923b7f commit 8c9b2cb

4 files changed

Lines changed: 31 additions & 16 deletions

File tree

src/middleware/index.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ 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);
1012
User.authenticate(user.name, user.pass, (error, user) => {
1113
if(error || !user){
1214
return Utils.throwError(401, "Access Denied: Wrong email or password", next);

src/routes/course.js

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

7-
router.get('/', (req, res, next) => {
8-
res.send("hello!");
7+
router.get('/',authorizeUser, (req, res, next) => {
8+
let status = 200;
9+
return res.status(status).json({success: true, message: "Courses Successfully retrieved!", status: status});
910
});
1011

1112
router.post('/', authorizeUser, (req, res, next) => {
@@ -18,7 +19,7 @@ router.post('/', authorizeUser, (req, res, next) => {
1819
return next(err);
1920
}
2021
let status = 201;
21-
return res.status(status).json({success: true, message: "User Successfully added!", status: status, course});
22+
return res.status(status).json({success: true, message: "Course Successfully added!", status: status, course});
2223
});
2324
});
2425

test/course_test.js

Lines changed: 20 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@
33
process.env.NODE_ENV = 'test';
44

55
const Course = require('../src/models/course'),
6-
chai = require('chai'),
7-
chaiHttp = require('chai-http'),
8-
server = require('../src/app');
6+
chai = require('chai'),
7+
chaiHttp = require('chai-http'),
8+
server = require('../src/app');
99

1010

1111
const should = chai.should();
@@ -30,13 +30,24 @@ describe('Courses', () => {
3030

3131
describe('/GET course', () => {
3232
it('should GET all the courses', (done) => {
33+
let user = {
34+
emailAddress: validAuth.user,
35+
password: validAuth.pass,
36+
fullName: "Joe Smith"
37+
};
3338
chai.request(server)
34-
.get(courseIndexLink)
39+
.post('/api/users/')
40+
.send(user)
3541
.end((err, res) => {
36-
res.should.have.status(200);
37-
done();
42+
chai.request(server)
43+
.get(courseIndexLink)
44+
.auth(validAuth.user, validAuth.pass)
45+
.end((err, res) => {
46+
res.body.should.have.property('message').equal("Courses Successfully retrieved!");
47+
res.should.have.status(200);
48+
done();
49+
});
3850
});
39-
4051
});
4152
});
4253

@@ -104,7 +115,8 @@ describe('Courses', () => {
104115
};
105116
postCourse(course, done);
106117
});
107-
function postCourse(course, done){
118+
119+
function postCourse(course, done) {
108120
chai.request(server)
109121
.post(courseIndexLink)
110122
.auth(validAuth.user, validAuth.pass)

test/user_test.js

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -68,11 +68,11 @@ describe('Users', () => {
6868
});
6969

7070
describe('/POST users', () => {
71-
beforeEach((done) => {
72-
User.remove({}, (err) => {
73-
done();
74-
});
75-
});
71+
// beforeEach((done) => {
72+
// User.remove({}, (err) => {
73+
// done();
74+
// });
75+
// });
7676
let user400 = {
7777
emailAddress: 'wre23sd',
7878
password: 'password',

0 commit comments

Comments
 (0)