6569
57%
master: 97%

Ran 05 Jun 2015 02:49PM UTC

Jobs 1

Files 63

Run time 4s

Badge

Embed ▾

pending completion

Build # 6569

Build Type

push

travis-ci

Committed by

nickstenning

Commit Message

Support reporting top-level form validation errors

When we call `form.validate()` and the validation fails, the raised
exception is a `deform.ValidationFailure`. This object has an `error`
attribute which represents the underlying validation error object for
the entire form.

Previously, if the validation failure were a result of top-level
validation errors (such as the fact that an unactivated user is trying
to log in), this error would be lost, because we only reported errors
for the forms fields, or "children".

This commit changes `h.accounts.views.validate_form` so that it converts
the entire `colander.Invalid` object into a dictionary using its
`asdict()` instance method. By doing this, we get two immediate
benefits:

- top-level validation errors are reported in the '' (empty string)
  field
- we avoid the need to aggregate form field errors by hand in ajax_form

In addition, we need to deal with this case on the frontend, so this
commit also changes the formRespond directive so that if no overall
"reason" is provided for failure, then an empty-string member of the
"errors" object can set the overall validation status (and error
message) for the form.

Run Details

1546 of 2763 relevant lines covered (55.95%)

0.56 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	6569.1	05 Jun 2015 02:49PM UTC	0	55.95	Travis Job 6569.1

Source Files on build 6569

Detailed source file information is not available for this build.

hypothesis / h / 6569
57%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6569

hypothesis / h / 6569 57% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 6569

hypothesis / h / 6569
57%
master: 97%

README BADGES
x