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

Merge branch 'fix_signup' into 'development' 

fix error message on user signup when there are multiple errors

See merge request !52
parents 5235c70a 40019139
No related branches found
No related tags found
2 merge requests!116Release v1.0.0,!52fix error message on user signup when there are multiple errors
Pipeline #
Hide whitespace changes
Inline Side-by-side
src/libs/routes/user.js
+ 11
− 18
View file @ 2bed8a71
...@@ -23,7 +23,7 @@ function emailSyntax(email) { ...@@ -23,7 +23,7 @@ function emailSyntax(email) {
userApp.post('/', (req, res, next) => { userApp.post('/', (req, res, next) => {
if(req.body.email){ if(req.body.email){
if(!emailSyntax(req.body.email)){ if(!emailSyntax(req.body.email)){
res.json({success: false, msg: 'O email informado é inválido.'}); res.json({success: false, msg: ['O email informado é inválido.']});
} else { } else {
next(); next();
} }
...@@ -35,10 +35,10 @@ userApp.post('/', (req, res, next) => { ...@@ -35,10 +35,10 @@ userApp.post('/', (req, res, next) => {
User.count({'email': req.body.email}, function(err, count){ User.count({'email': req.body.email}, function(err, count){
if (err){ if (err){
log.error('MongoDB error: ' + err); log.error('MongoDB error: ' + err);
res.json({success: false, msg: 'Um erro ocorreu no banco de dados.'}); res.json({success: false, msg: ['Um erro ocorreu no banco de dados.']});
} }
if(count){ if(count){
res.json({success: false, msg: 'O email informado já está cadastrado.'}); res.json({success: false, msg: ['O email informado já está cadastrado.']});
} else { } else {
next(); next();
} }
...@@ -48,10 +48,10 @@ userApp.post('/', (req, res, next) => { ...@@ -48,10 +48,10 @@ userApp.post('/', (req, res, next) => {
User.count({'cpf': req.body.cpf}, function(err, count){ User.count({'cpf': req.body.cpf}, function(err, count){
if (err){ if (err){
log.error('MongoDB error: ' + err); log.error('MongoDB error: ' + err);
res.json({success: false, msg: 'Um erro ocorreu no banco de dados.'}); res.json({success: false, msg: ['Um erro ocorreu no banco de dados.']});
} }
if(count){ if(count){
res.json({success: false, msg: 'O CPF informado já está cadastrado.'}); res.json({success: false, msg: ['O CPF informado já está cadastrado.']});
} else { } else {
next(); next();
} }
...@@ -81,15 +81,8 @@ userApp.post('/', (req, res, next) => { ...@@ -81,15 +81,8 @@ userApp.post('/', (req, res, next) => {
for (var e in err.errors) { for (var e in err.errors) {
errArray.push(err.errors[`${e}`].message); errArray.push(err.errors[`${e}`].message);
} }
for (var i = 0; i < errArray.length; i++) {
if(i > 0){ res.json({success: false, msg: errArray});
errMsg = errMsg + '\n' + errArray[i];
}
else{
errMsg = errMsg + errArray[i];
}
}
res.json({success: false, msg: errMsg});
} }
else { else {
res.json({success: true, msg: 'Usuário cadastrado com sucesso!'}); res.json({success: true, msg: 'Usuário cadastrado com sucesso!'});
...@@ -99,14 +92,14 @@ userApp.post('/', (req, res, next) => { ...@@ -99,14 +92,14 @@ userApp.post('/', (req, res, next) => {
userApp.post('/authenticate', (req, res, next) => { userApp.post('/authenticate', (req, res, next) => {
if (!req.body.email) { if (!req.body.email) {
res.json({success: false, msg: 'O campo Email é obrigatório.'}); res.json({success: false, msg: ['O campo Email é obrigatório.']});
} else { } else {
next(); next();
} }
}, (req, res, next) => { }, (req, res, next) => {
if (!req.body.password) { if (!req.body.password) {
res.json({success: false, msg: 'O campo Senha é obrigatório.'}); res.json({success: false, msg: ['O campo Senha é obrigatório.']});
} else { } else {
next(); next();
} }
...@@ -118,7 +111,7 @@ userApp.post('/authenticate', (req, res, next) => { ...@@ -118,7 +111,7 @@ userApp.post('/authenticate', (req, res, next) => {
if (err) throw err; if (err) throw err;
if(!user){ if(!user){
res.json({success: false, msg: 'O Email informado não está cadastrado.'}); res.json({success: false, msg: ['O Email informado não está cadastrado.']});
} }
else { else {
user.comparePassword(req.body.password, (err, isMatch) => { user.comparePassword(req.body.password, (err, isMatch) => {
...@@ -132,7 +125,7 @@ userApp.post('/authenticate', (req, res, next) => { ...@@ -132,7 +125,7 @@ userApp.post('/authenticate', (req, res, next) => {
res.json({success: true, token: 'JWT ' + token, msg: 'Usuário autenticado com sucesso'}); res.json({success: true, token: 'JWT ' + token, msg: 'Usuário autenticado com sucesso'});
} }
else { else {
res.json({success: false, msg: 'A Senha informada é inválida.'}); res.json({success: false, msg: ['A Senha informada é inválida.']});
} }
}); });
} }
......
src/test/user.js
+ 16
− 16
View file @ 2bed8a71
...@@ -90,7 +90,7 @@ describe('Saves a user', () => { ...@@ -90,7 +90,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Email é obrigatório.'); res.body.msg[0].should.be.equal('O campo Email é obrigatório.');
done(); done();
}); });
}); });
...@@ -119,7 +119,7 @@ describe('Saves a user', () => { ...@@ -119,7 +119,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Senha é obrigatório.'); res.body.msg[0].should.be.equal('O campo Senha é obrigatório.');
done(); done();
}); });
}); });
...@@ -149,7 +149,7 @@ describe('Saves a user', () => { ...@@ -149,7 +149,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O email informado é inválido.'); res.body.msg[0].should.be.equal('O email informado é inválido.');
done(); done();
}); });
}); });
...@@ -178,7 +178,7 @@ describe('Saves a user', () => { ...@@ -178,7 +178,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Nome é obrigatório.'); res.body.msg[0].should.be.equal('O campo Nome é obrigatório.');
done(); done();
}); });
}); });
...@@ -207,7 +207,7 @@ describe('Saves a user', () => { ...@@ -207,7 +207,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo CPF é obrigatório.'); res.body.msg[0].should.be.equal('O campo CPF é obrigatório.');
done(); done();
}); });
}); });
...@@ -236,7 +236,7 @@ describe('Saves a user', () => { ...@@ -236,7 +236,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Escolaridade é obrigatório.'); res.body.msg[0].should.be.equal('O campo Escolaridade é obrigatório.');
done(); done();
}); });
}); });
...@@ -265,7 +265,7 @@ describe('Saves a user', () => { ...@@ -265,7 +265,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Segmento é obrigatório.'); res.body.msg[0].should.be.equal('O campo Segmento é obrigatório.');
done(); done();
}); });
}); });
...@@ -294,7 +294,7 @@ describe('Saves a user', () => { ...@@ -294,7 +294,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Função é obrigatório.'); res.body.msg[0].should.be.equal('O campo Função é obrigatório.');
done(); done();
}); });
}); });
...@@ -323,7 +323,7 @@ describe('Saves a user', () => { ...@@ -323,7 +323,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Instituição em que trabalha é obrigatório.'); res.body.msg[0].should.be.equal('O campo Instituição em que trabalha é obrigatório.');
done(); done();
}); });
}); });
...@@ -352,7 +352,7 @@ describe('Saves a user', () => { ...@@ -352,7 +352,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Estado é obrigatório.'); res.body.msg[0].should.be.equal('O campo Estado é obrigatório.');
done(); done();
}); });
}); });
...@@ -381,7 +381,7 @@ describe('Saves a user', () => { ...@@ -381,7 +381,7 @@ describe('Saves a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.be.equal('O campo Cidade é obrigatório.'); res.body.msg[0].should.be.equal('O campo Cidade é obrigatório.');
done(); done();
}); });
}); });
...@@ -469,7 +469,7 @@ describe('Authenticates a user', () => { ...@@ -469,7 +469,7 @@ describe('Authenticates a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.equal('A Senha informada é inválida.') res.body.msg[0].should.equal('A Senha informada é inválida.')
done(); done();
}); });
}); });
...@@ -507,7 +507,7 @@ describe('Authenticates a user', () => { ...@@ -507,7 +507,7 @@ describe('Authenticates a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.equal('O Email informado não está cadastrado.') res.body.msg[0].should.equal('O Email informado não está cadastrado.')
done(); done();
}); });
}); });
...@@ -544,7 +544,7 @@ describe('Authenticates a user', () => { ...@@ -544,7 +544,7 @@ describe('Authenticates a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.equal('O campo Email é obrigatório.') res.body.msg[0].should.equal('O campo Email é obrigatório.')
done(); done();
}); });
}); });
...@@ -581,7 +581,7 @@ describe('Authenticates a user', () => { ...@@ -581,7 +581,7 @@ describe('Authenticates a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.equal('O campo Senha é obrigatório.') res.body.msg[0].should.equal('O campo Senha é obrigatório.')
done(); done();
}); });
}); });
...@@ -618,7 +618,7 @@ describe('Authenticates a user', () => { ...@@ -618,7 +618,7 @@ describe('Authenticates a user', () => {
res.body.should.have.property('success'); res.body.should.have.property('success');
res.body.success.should.equal(false); res.body.success.should.equal(false);
res.body.should.have.property('msg'); res.body.should.have.property('msg');
res.body.msg.should.equal('A Senha informada é inválida.') res.body.msg[0].should.equal('A Senha informada é inválida.')
done(); done();
}); });
}); });
......
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