Why getValidationResult doesn't reject the Promise when it has an error?

I can’t understand the purpose of doing:

req.getValidationResult.then((results) => {
  if (!results.isEmpty()) {
    // Error
  }
});

Why this method exists (.isEmpty)? Isn’t better to reject the promise, then I can get the error at the .catch? .then for me means that everything occurs okay.

[rdsedmundo]

Well, that’s my particular opinion. I’d like if we don’t need this verification statement. That’s very useful if we’re using async/await too.

A good name for a new method would be: ‘isValid’. So: ‘Is valid? Then do this’.

[gustavohenke]

BTW, you could do something like the following, which will allow you to deal with the errors in the catch block of the promise:

req.getValidationResult().then( result => result.throw() ).then( () => {
  req.send( "success!" );
}, e => {
  req.status( 400 ).json( e.array() );
})