diff --git a/src/libs/routes/user.js b/src/libs/routes/user.js
index a6b0525a7318eac8d86f239c9fdf89c310b24155..4e4dd72f2cf3ef2dd854f7ccaa4dacf9a642d9d2 100644
--- a/src/libs/routes/user.js
+++ b/src/libs/routes/user.js
@@ -23,7 +23,8 @@ function emailSyntax(email) {
userApp.post('/', (req, res, next) => {
if(req.body.email){
if(!emailSyntax(req.body.email)){
- res.json({success: false, msg: ['O email informado é inválido.']});
+ res.status(400);
+ res.json({success: false, msg: 'O email informado é inválido.'});
} else {
next();
}
@@ -35,10 +36,12 @@ userApp.post('/', (req, res, next) => {
User.count({'email': req.body.email}, function(err, count){
if (err){
log.error('MongoDB error: ' + err);
+ res.status(500);
res.json({success: false, msg: ['Um erro ocorreu no banco de dados.']});
}
if(count){
- res.json({success: false, msg: ['O email informado já está cadastrado.']});
+ res.status(400);
+ res.json({success: false, msg: 'O email informado já está cadastrado.'});
} else {
next();
}
@@ -48,10 +51,13 @@ userApp.post('/', (req, res, next) => {
User.count({'cpf': req.body.cpf}, function(err, count){
if (err){
log.error('MongoDB error: ' + err);
+ res.status(500);
res.json({success: false, msg: ['Um erro ocorreu no banco de dados.']});
}
if(count){
- res.json({success: false, msg: ['O CPF informado já está cadastrado.']});
+ res.status(400);
+ res.json({success: false, msg: 'O CPF informado já está cadastrado.'});
+
} else {
next();
}
@@ -81,10 +87,11 @@ userApp.post('/', (req, res, next) => {
for (var e in err.errors) {
errArray.push(err.errors[`${e}`].message);
}
-
+ res.status(400);
res.json({success: false, msg: errArray});
}
else {
+ res.status(201);
res.json({success: true, msg: 'Usuário cadastrado com sucesso!'});
}
});
@@ -92,14 +99,16 @@ userApp.post('/', (req, res, next) => {
userApp.post('/authenticate', (req, res, next) => {
if (!req.body.email) {
- res.json({success: false, msg: ['O campo Email é obrigatório.']});
+ res.status(400);
+ res.json({success: false, msg: 'O campo Email é obrigatório.'});
} else {
next();
}
}, (req, res, next) => {
if (!req.body.password) {
- res.json({success: false, msg: ['O campo Senha é obrigatório.']});
+ res.status(400);
+ res.json({success: false, msg: 'O campo Senha é obrigatório.'});
} else {
next();
}
@@ -111,6 +120,7 @@ userApp.post('/authenticate', (req, res, next) => {
if (err) throw err;
if(!user){
+ res.status(400);
res.json({success: false, msg: ['O Email informado não está cadastrado.']});
}
else {
@@ -125,6 +135,7 @@ userApp.post('/authenticate', (req, res, next) => {
res.json({success: true, token: 'JWT ' + token, msg: 'Usuário autenticado com sucesso'});
}
else {
+ res.status(400);
res.json({success: false, msg: ['A Senha informada é inválida.']});
}
});
diff --git a/src/test/user.js b/src/test/user.js
index 90f2bb45b64916f7d7a45918ca6a7f1d2c28378a..9d6228284509f56682e7a7c6d781e79d6abb6d72 100644
--- a/src/test/user.js
+++ b/src/test/user.js
@@ -55,7 +55,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(201);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(true);
@@ -85,7 +85,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -114,7 +114,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -144,12 +144,12 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
res.body.should.have.property('msg');
- res.body.msg[0].should.be.equal('O email informado é inválido.');
+ res.body.msg.should.be.equal('O email informado é inválido.');
done();
});
});
@@ -173,7 +173,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -202,7 +202,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -231,7 +231,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -260,7 +260,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -289,7 +289,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -318,7 +318,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -347,7 +347,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -376,7 +376,7 @@ describe('Saves a user', () => {
.set('x-apicache-bypass', 'true')
.send(newUser)
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -464,7 +464,7 @@ describe('Authenticates a user', () => {
.send({email: 'lorem@ipsum.com',
password: 'umasenhaerrada'})
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -502,7 +502,7 @@ describe('Authenticates a user', () => {
.send({email: 'dolor@ipsum.com',
password: '123mudar'})
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
@@ -539,12 +539,12 @@ describe('Authenticates a user', () => {
.set('x-apicache-bypass', 'true')
.send({password: '123mudar'})
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
res.body.should.have.property('msg');
- res.body.msg[0].should.equal('O campo Email é obrigatório.')
+ res.body.msg.should.equal('O campo Email é obrigatório.')
done();
});
});
@@ -576,12 +576,12 @@ describe('Authenticates a user', () => {
.set('x-apicache-bypass', 'true')
.send({email:'lorem@ipsum.com'})
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);
res.body.should.have.property('msg');
- res.body.msg[0].should.equal('O campo Senha é obrigatório.')
+ res.body.msg.should.equal('O campo Senha é obrigatório.')
done();
});
});
@@ -613,7 +613,7 @@ describe('Authenticates a user', () => {
.set('x-apicache-bypass', 'true')
.send({email:'lorem@ipsum.com', password: '123'})
.end((err, res) => {
- res.should.have.status(200);
+ res.should.have.status(400);
res.should.be.json;
res.body.should.have.property('success');
res.body.success.should.equal(false);