Opened 9 years ago

Closed 3 years ago

#6103 closed Cleanup/optimization (fixed)

tests/model_forms/tests.py could use some rewriting

Reported by: Malcolm Tredinnick Owned by: Claude Paroz
Component: Forms Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

As noted at the top of this test file, the ModelForms implementation mostly ported the old form_for_* tests. It would be good if somebody could rewrite those tests to be a little more relevant (and illustrative) of model forms' capabilities. Some of them currently don't quite make sense.

Change History (10)

comment:1 Changed 9 years ago by Malcolm Tredinnick

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

comment:2 Changed 5 years ago by Gabriel Hurley

Severity: Normal
Type: Cleanup/optimization

comment:3 Changed 5 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:4 Changed 5 years ago by Aymeric Augustin

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:5 Changed 4 years ago by bharadwajdes

Owner: changed from nobody to bharadwajdes
Status: newassigned

comment:6 Changed 4 years ago by bharadwajdes

Please let me know any specific changes to be addressed here.

comment:7 Changed 3 years ago by bharadwajdes

Owner: bharadwajdes deleted
Status: assignednew

comment:8 Changed 3 years ago by Tim Graham

Summary: modeltests/model_forms/models.py could do with some rewritingtests/model_forms/tests.py could use some rewriting

comment:9 Changed 3 years ago by Claude Paroz

Owner: set to Claude Paroz
Status: newassigned

comment:10 Changed 3 years ago by Claude Paroz <claude@…>

Resolution: fixed
Status: assignedclosed

In 72cfbdc10ab53ee9a3a9f253f38e9919728c0813:

Fixed #6103 -- Splitted tests in model_forms tests

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