Closed Bug 788416 Opened 12 years ago Closed 12 years ago

Remove ancient and unused tests in toolkit\mozapps\extensions\test\unit

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla18

People

(Reporter: Unfocused, Assigned: Unfocused)

Details

Attachments

(1 file)

Sooo... turns out toolkit\mozapps\extensions\test\unit still exists, and contains tests for the old Add-ons Manager. ie, before The Great Rewrite(tm). This may come as a shock, but I propose removing that directory.
Attached patch Patch v1 (deleted) — Splinter Review
Attachment #658397 - Flags: review?(dtownsend+bugmail)
Comment on attachment 658397 [details] [diff] [review] Patch v1 Review of attachment 658397 [details] [diff] [review]: ----------------------------------------------------------------- So I originally left these in place because I thought they were issues that still needed testing even in the new world order. I think that's probably not true for bug 356370 but the other two I think it'd be nice to be tested. Will we ever remember to add tests if we remove these files? Probably not. Will we ever remember to add tests if we leave these files here? Probably not.
Attachment #658397 - Flags: review?(dtownsend+bugmail) → review+
Indeed. At the very least, removing them from the tree saves time/effort, as people are currently still trying to update them (bug 758950, bug 759642, etc). https://hg.mozilla.org/integration/fx-team/rev/8d83eefa874f
Whiteboard: [fixed-in-fx-team]
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: