Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#12378 closed (fixed)

Remove redundant test.

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

Description

r11861 fixes #12367, but leaves a test that is now 100% redundant. OldAdminForeignKeyWidgetChangeList duplicates AdminForeignKeyWidgetChangeList, and urls2.py is only used by OldAdminForeignKeyWidgetChangeList.

This patch removes that test and urlconf file.

Attachments (1)

12378.diff (1.0 KB) - added by Cliff Dyer 7 years ago.
Patch to remove redundant test.

Download all attachments as: .zip

Change History (6)

Changed 7 years ago by Cliff Dyer

Attachment: 12378.diff added

Patch to remove redundant test.

comment:1 Changed 7 years ago by Cliff Dyer

Needs documentation: unset
Needs tests: unset
Owner: changed from nobody to Cliff Dyer
Patch needs improvement: unset

Oops. Forgot to log in. I reported this.

comment:2 Changed 7 years ago by Cliff Dyer

Component: UncategorizedTesting framework
Has patch: set

comment:3 Changed 7 years ago by Eric Holscher

Triage Stage: UnreviewedReady for checkin

comment:4 Changed 7 years ago by Russell Keith-Magee

Resolution: fixed
Status: newclosed

(In [12418]) Fixed #12378 -- Removed some tests made redundant by feature deprecation. Thanks to jcd for the report.

comment:5 Changed 5 years ago by Jacob

milestone: 1.2

Milestone 1.2 deleted

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