Skip to content
Snippets Groups Projects
Commit 7b3a4ce4 authored by Vytor Calixto's avatar Vytor Calixto :space_invader:
Browse files

Merge branch 'user_route_response_code_fix' into development

parents 5bb705ba 29c95d3e
No related branches found
No related tags found
1 merge request!116Release v1.0.0
Pipeline #
Hide whitespace changes
Inline Side-by-side
src/libs/routes/user.js
+ 17
− 6
View file @ 7b3a4ce4
......@@ -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.']});
}
});
......
src/test/user.js
+ 20
− 20
View file @ 7b3a4ce4
......@@ -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);
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment