Opened 4 months ago

Closed 4 months ago

#29275 closed Cleanup/optimization (fixed)

Emit "(actual errors: none)" instead of "(actual errors: )" in assertFormError

Reported by: Chris Lamb Owned by: nobody
Component: Testing framework Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by Chris Lamb)

The suffix in the error output of assertFormError (eg.)

AssertionError: False is not true : The form 'form' in context 0 does not contain the non-field error 'Please enter a correct email and password. Note that both fields may be case-sensitive.' (actual errors: )

is kinda weird. We should print "none" instead, otherwise it looks like there is an error in the error.

Change History (3)

comment:1 Changed 4 months ago by Chris Lamb

Description: modified (diff)
Has patch: set
Summary: Emit "(actual errors: none)" instead of "(actual errors: )"Emit "(actual errors: none)" instead of "(actual errors: )" in assertFormError

comment:2 Changed 4 months ago by Claude Paroz

Triage Stage: UnreviewedReady for checkin
Version: 2.0master

comment:3 Changed 4 months ago by Tim Graham <timograham@…>

Resolution: fixed
Status: newclosed

In 84ad721:

Fixed #29275 -- Made assertFormError() output "(actual errors: none)" instead of "(actual errors: )".

Note: See TracTickets for help on using tickets.
Back to Top