Opened 7 years ago

Closed 13 months ago

#6103 closed Cleanup/optimization (fixed)

tests/model_forms/tests.py could use some rewriting

Reported by: mtredinnick Owned by: claudep
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 7 years ago by mtredinnick

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Accepted

comment:2 Changed 4 years ago by gabrielhurley

  • Severity set to Normal
  • Type set to Cleanup/optimization

comment:3 Changed 3 years ago by aaugustin

  • UI/UX unset

Change UI/UX from NULL to False.

comment:4 Changed 3 years ago by aaugustin

  • Easy pickings unset

Change Easy pickings from NULL to False.

comment:5 Changed 2 years ago by bharadwajdes

  • Owner changed from nobody to bharadwajdes
  • Status changed from new to assigned

comment:6 Changed 2 years ago by bharadwajdes

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

comment:7 Changed 2 years ago by bharadwajdes

  • Owner bharadwajdes deleted
  • Status changed from assigned to new

comment:8 Changed 21 months ago by timo

  • Summary changed from modeltests/model_forms/models.py could do with some rewriting to tests/model_forms/tests.py could use some rewriting

comment:9 Changed 13 months ago by claudep

  • Owner set to claudep
  • Status changed from new to assigned

comment:10 Changed 13 months ago by Claude Paroz <claude@…>

  • Resolution set to fixed
  • Status changed from assigned to closed

In 72cfbdc10ab53ee9a3a9f253f38e9919728c0813:

Fixed #6103 -- Splitted tests in model_forms tests

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