assertFieldOutput fails if field has customized required message
|Reported by:||dpifke||Owned by:||nobody|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
The string u'This field is required.' is hard-coded into the field checks in SimpleTestCase.assertFieldOutput(). This causes tests using this method to always fail for fields which have customized default_error_messages.
Proposed fix would be to use the string from the field's error_messages dictionary instead; one-line patch to do so forthcoming.
Change History (5)
Changed 2 years ago by dpifke
comment:1 Changed 2 years ago by julien
- Needs documentation unset
- Needs tests set
- Patch needs improvement unset
- Severity changed from Normal to Release blocker
- Triage Stage changed from Unreviewed to Accepted