Opened 6 years ago

Closed 6 years ago

Last modified 4 years ago

#12378 closed (fixed)

Remove redundant test.

Reported by: anonymous Owned by: jcd
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:


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

This patch removes that test and urlconf file.

Attachments (1)

12378.diff (1.0 KB) - added by jcd 6 years ago.
Patch to remove redundant test.

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by jcd

Patch to remove redundant test.

comment:1 Changed 6 years ago by jcd

  • Needs documentation unset
  • Needs tests unset
  • Owner changed from nobody to jcd
  • Patch needs improvement unset

Oops. Forgot to log in. I reported this.

comment:2 Changed 6 years ago by jcd

  • Component changed from Uncategorized to Testing framework
  • Has patch set

comment:3 Changed 6 years ago by ericholscher

  • Triage Stage changed from Unreviewed to Ready for checkin

comment:4 Changed 6 years ago by russellm

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

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

comment:5 Changed 4 years ago by jacob

  • milestone 1.2 deleted

Milestone 1.2 deleted

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