Opened 6 years ago

Closed 5 years ago

#15837 closed Cleanup/optimization (fixed)

Consolidate localflavor tests

Reported by: Julien Phalip Owned by: Julien Phalip
Component: contrib.localflavor Version: 1.3
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no


Localflavor tests are spread out between two locations: source:django/trunk/tests/regressiontests/forms/localflavor and source:django/trunk/tests/regressiontests/localflavor

For clarity's sake, the tests in the former location should be moved to the latter one.

Change History (5)

comment:1 Changed 6 years ago by Aymeric Augustin

Easy pickings: set
Triage Stage: UnreviewedAccepted
Version: 1.21.3

comment:2 Changed 5 years ago by rodriguealcazar

Owner: changed from nobody to rodriguealcazar
Status: newassigned

comment:3 Changed 5 years ago by Julien Phalip

UI/UX: unset

Also, in the process of consolidating the tests, it would be nice to avoid having to repeat imports as currently in source:django/trunk/tests/regressiontests/forms/ and source:django/trunk/tests/regressiontests/forms/tests/ That particular cleanup issue was raised in #16365, closed as dupe in favour of this one.

comment:4 Changed 5 years ago by Julien Phalip

Owner: changed from rodriguealcazar to Julien Phalip
Status: assignednew

No update in 3 months. I'll take a look at it. Let me know if you've got any WIP.

comment:5 Changed 5 years ago by Julien Phalip

Resolution: fixed
Status: newclosed

In [16680]:

Fixed #15837. Consolidated all the locaflavor tests into a single, logical, place (regressiontests/localflavor/).

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